On Wed, May 18, 2016 at 6:07 PM, Andre Przywara <andre.przyw...@arm.com> wrote: > Hi, > > On 18/05/16 10:10, macro.wav...@gmail.com wrote: >> From: Wang Dongsheng <dongsheng.w...@nxp.com> >> >> According to PSCI specification v1.0, the PSCI functions should start from >> 0x84000000 for SMC32, this patch changes this base value as well as other >> function offset values. > > Out of curiosity: Do you plan to ever support ARMv8 boards with PSCI > services in U-Boot as well? > Then we would need to think about adjusting the function IDs, because > they use a "64-bit" flag for some functions. > My workmate Hongtao Jia <hongtao....@nxp.com> is working for ARMv8 PSCI. That should be in another separate file, so same or different function names can both be used freely I think.
>> Signed-off-by: Wang Dongsheng <dongsheng.w...@nxp.com> >> Signed-off-by: Hongbo Zhang <hongbo.zh...@nxp.com> > > Otherwise: > Reviewed-by: Andre Przywara <andre.przyw...@arm.com> > Thanks for review. >> --- >> arch/arm/include/asm/psci.h | 10 +++++----- >> 1 file changed, 5 insertions(+), 5 deletions(-) >> >> diff --git a/arch/arm/include/asm/psci.h b/arch/arm/include/asm/psci.h >> index 128a606..a4a19e3 100644 >> --- a/arch/arm/include/asm/psci.h >> +++ b/arch/arm/include/asm/psci.h >> @@ -19,13 +19,13 @@ >> #define __ARM_PSCI_H__ >> >> /* PSCI interface */ >> -#define ARM_PSCI_FN_BASE 0x95c1ba5e >> +#define ARM_PSCI_FN_BASE 0x84000000 >> #define ARM_PSCI_FN(n) (ARM_PSCI_FN_BASE + (n)) >> >> -#define ARM_PSCI_FN_CPU_SUSPEND ARM_PSCI_FN(0) >> -#define ARM_PSCI_FN_CPU_OFF ARM_PSCI_FN(1) >> -#define ARM_PSCI_FN_CPU_ON ARM_PSCI_FN(2) >> -#define ARM_PSCI_FN_MIGRATE ARM_PSCI_FN(3) >> +#define ARM_PSCI_FN_CPU_SUSPEND ARM_PSCI_FN(1) >> +#define ARM_PSCI_FN_CPU_OFF ARM_PSCI_FN(2) >> +#define ARM_PSCI_FN_CPU_ON ARM_PSCI_FN(3) >> +#define ARM_PSCI_FN_MIGRATE ARM_PSCI_FN(5) >> >> #define ARM_PSCI_RET_SUCCESS 0 >> #define ARM_PSCI_RET_NI (-1) >> _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot