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.

> 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>

> ---
>  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

Reply via email to