> Subject: Re: [PATCH 08/10] arm64: sysregs: correct SCTLR_EL1_RES > > On 11.08.20 10:07, Peng Fan wrote: > >> Subject: Re: [PATCH 08/10] arm64: sysregs: correct SCTLR_EL1_RES > >> > >> On 11.08.20 09:49, Peng Fan wrote: > >>>> Subject: Re: [PATCH 08/10] arm64: sysregs: correct SCTLR_EL1_RES > >>>> > >>>> On 07.08.20 05:06, peng....@nxp.com wrote: > >>>>> From: Peng Fan <peng....@nxp.com> > >>>>> > >>>>> bit23 is SPAN, not reserved bit. > >>>>> > >>>> > >>>> This is not correct, see D7.2.81 in the ARMv8 manual. > >>>> > >>>> Where did you get this from? What is it trying to fix? > >>> > >>> DDI0487D_a_armv8_arm, > >>> " D12.2.100 SCTLR_EL1, System Control Register (EL1)" > >>> > >>> This is just to align with Linux Kernel, since the head.S in the > >>> bootloader needs this. > >> > >> OK, this is ARMv8.1+, I was looking at 8.0. Are you sure the reset > >> value changed from 8.0 to 8.1? We definitely need to remain compatible to > 8.0. > > > > See > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Felix > > > ir.bootlin.com%2Flinux%2Flatest%2Fsource%2Farch%2Farm64%2Finclude%2F > as > > > m%2Fsysreg.h%23L577&data=02%7C01%7Cpeng.fan%40nxp.com%7C0 > ef784761c > > > d1429e247808d83dcdd7bd%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0 > %7C0%7C63 > > > 7327301580538181&sdata=3GaVwiR0l7hzblbxYpcxJ5IUun%2BcnYb%2F > P0RNlE7 > > zz0Y%3D&reserved=0 > > > > It not have bit23 reserved. > > > > ...but set. That is broken with your patch.
Overlooked that. Then drop this change. Regards, Peng. > > Jan > > -- > Siemens AG, Corporate Technology, CT RDA IOT SES-DE Corporate > Competence Center Embedded Linux -- You received this message because you are subscribed to the Google Groups "Jailhouse" group. To unsubscribe from this group and stop receiving emails from it, send an email to jailhouse-dev+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/jailhouse-dev/AM5PR0402MB2756D6D83FC2F6DD253C968C88450%40AM5PR0402MB2756.eurprd04.prod.outlook.com.