target: fix ordering of arguments to mcr and mrc commands
Now matches machine code syntax and old arm11 syntax. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
This commit is contained in:
parent
4e3c2676f1
commit
051eaf7950
|
@ -722,25 +722,25 @@ static int arm_cp_check(struct target_s *target, int cpnum, uint32_t op1, uint32
|
||||||
return ERROR_FAIL;
|
return ERROR_FAIL;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (op1>7)
|
if (op1 > 7)
|
||||||
{
|
{
|
||||||
LOG_ERROR("Illegal op1");
|
LOG_ERROR("Illegal op1");
|
||||||
return ERROR_FAIL;
|
return ERROR_FAIL;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (op2>7)
|
if (op2 > 7)
|
||||||
{
|
{
|
||||||
LOG_ERROR("Illegal op2");
|
LOG_ERROR("Illegal op2");
|
||||||
return ERROR_FAIL;
|
return ERROR_FAIL;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (CRn>15)
|
if (CRn > 15)
|
||||||
{
|
{
|
||||||
LOG_ERROR("Illegal CRn");
|
LOG_ERROR("Illegal CRn");
|
||||||
return ERROR_FAIL;
|
return ERROR_FAIL;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (CRm>7)
|
if (CRm > 15)
|
||||||
{
|
{
|
||||||
LOG_ERROR("Illegal CRm");
|
LOG_ERROR("Illegal CRm");
|
||||||
return ERROR_FAIL;
|
return ERROR_FAIL;
|
||||||
|
@ -4899,19 +4899,19 @@ static int jim_mcrmrc(Jim_Interp *interp, int argc, Jim_Obj *const *argv)
|
||||||
if (e != JIM_OK) {
|
if (e != JIM_OK) {
|
||||||
return e;
|
return e;
|
||||||
}
|
}
|
||||||
op2 = l;
|
CRn = l;
|
||||||
|
|
||||||
e = Jim_GetLong(interp, argv[4], &l);
|
e = Jim_GetLong(interp, argv[4], &l);
|
||||||
if (e != JIM_OK) {
|
if (e != JIM_OK) {
|
||||||
return e;
|
return e;
|
||||||
}
|
}
|
||||||
CRn = l;
|
CRm = l;
|
||||||
|
|
||||||
e = Jim_GetLong(interp, argv[5], &l);
|
e = Jim_GetLong(interp, argv[5], &l);
|
||||||
if (e != JIM_OK) {
|
if (e != JIM_OK) {
|
||||||
return e;
|
return e;
|
||||||
}
|
}
|
||||||
CRm = l;
|
op2 = l;
|
||||||
|
|
||||||
value = 0;
|
value = 0;
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue