Re: [Qemu-devel] [Qemu-trivial] DBGAUTHSTATUS register

Re: [Qemu-devel] [Qemu-trivial] DBGAUTHSTATUS register

[Michael Tokarev]

29.05.2014 21:32, Aleksandar Ristovski wrote:
> Hello,
> 
> 
> This is not a big issue: DBGAUTHSTATUS, according to the docs, should exist. According to "ARM Architecture Reference Manual, ARMv7-A and ARMv7-R edition" (DDI0406C_b_arm_architecture_reference_manual.pdf), DBGAUTHSTATUS is required in all implementations. Reading it should be permitted, even if as RAZ only (implying no features are implemented/enabled).
> 
> This trivial patch defines cp14 register DBGAUTHSTATUS.

While the patch appears to be trivial, I had to grab the above mentioned ARM manual
and read a bit from there.  Unfortunately, since I don't have any prior knowlege
about this stuff, at all, I still have to accept for granted that it is the right
thing to do.  Maybe it is better to route this to the ARM maintainer instead
(Cc'ed) ?

Thanks,

/mjt

Re: [Qemu-devel] [Qemu-trivial] DBGAUTHSTATUS register

[Michael Tokarev]

10.06.2014 19:50, Michael Tokarev wrote:
> 29.05.2014 21:32, Aleksandar Ristovski wrote:
>> Hello,
>>
>> This is not a big issue: DBGAUTHSTATUS, according to the docs, should exist. According to "ARM Architecture Reference Manual, ARMv7-A and ARMv7-R edition" (DDI0406C_b_arm_architecture_reference_manual.pdf), DBGAUTHSTATUS is required in all implementations. Reading it should be permitted, even if as RAZ only (implying no features are implemented/enabled).
>>
>> This trivial patch defines cp14 register DBGAUTHSTATUS.
> 
> While the patch appears to be trivial, I had to grab the above mentioned ARM manual
> and read a bit from there.  Unfortunately, since I don't have any prior knowlege
> about this stuff, at all, I still have to accept for granted that it is the right
> thing to do.  Maybe it is better to route this to the ARM maintainer instead
> (Cc'ed) ?

Also, this patch does not have the Signed-off-by line.

Thanks,

/mjt