Hi Julien, >> Just to be sure to be on the same page, are you suggesting these changes on >> the original file? > > Yes with one tweak. > > > > diff --git a/docs/misc/arm/booting.txt b/docs/misc/arm/booting.txt >> index 21ae74837dcc..c00c651805d7 100644 >> --- a/docs/misc/arm/booting.txt >> +++ b/docs/misc/arm/booting.txt >> @@ -58,10 +58,14 @@ Firmware/bootloader requirements >> Xen relies on some settings the firmware has to configure in EL3 before >> starting Xen. > > I think you want to update this sentence to remove the reference to EL3. Even > on A-profile EL3 is not mandatory (I vaguely remember one of the platform I > worked on had no EL3). > >> -* Xen must be entered in NS EL2 mode >> +* Xen must be entered in: >> + * Non-Secure EL2 mode for Armv8-A Arm64 and Arm32, Armv8-R Arm32. >> + * Secure EL2 mode for Armv8-R Arm64. >> * The bit SCR_EL3.HCE (resp. SCR.HCE for 32-bit ARM) must be set to 1. > > And then here: "When EL3 is supported, ...". This would also cover the > R-profile change. >
Thanks for the clarification, @Michal, @Ayan, are you ok if I retain your R-by with these changes or should I drop it? > Cheers, > >> +* Xen must be entered with MMU/MPU off and data cache disabled >> (SCTLR_EL2.M bit >> + and SCTLR_EL2.C set to 0). >> [1] linux/Documentation/arm/booting.rst >> Latest version: >> http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/arch/arm/booting.rst Cheers, Luca
