On 17/11/2022 13:39, Michal Orzel wrote:
Hi Ayan,

On 11/11/2022 15:17, Ayan Kumar Halder wrote:
Refer ARM DDI 0487I.a ID081822, G8-9817, G8.2.169
Affinity level 3 is not present in AArch32.
Also, refer ARM DDI 0406C.d ID040418, B4-1644, B4.1.106,
Affinity level 3 is not present in Armv7 (ie arm32).
Thus, any access to affinity level 3 needs to be guarded within
"ifdef CONFIG_ARM_64".

Signed-off-by: Ayan Kumar Halder <[email protected]>
Reviewed-by: Michal Orzel <[email protected]>

although, IMO the commit msg does not reflect the change (i.e. you do nothing
related to accessing MPIDR, but instead you are just not taking the Aff3 into 
account for AArch32).
Also, I'm not sure why you used VMPIDR and not MPIDR.

+1. Can one of you propose an improved commit message/title?

I would be happy to update the patch on commit if there are nothing else to update in this series.

Cheers,

--
Julien Grall

Reply via email to