public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Andre Przywara <andre.przywara@arm.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 01/11] ARM: PSCI: change PSCI function IDs base and offsets
Date: Wed, 18 May 2016 11:07:35 +0100	[thread overview]
Message-ID: <573C3EE7.3060308@arm.com> (raw)
In-Reply-To: <1463562634-16723-2-git-send-email-hongbo.zhang@nxp.com>

Hi,

On 18/05/16 10:10, macro.wave.z at gmail.com wrote:
> From: Wang Dongsheng <dongsheng.wang@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.wang@nxp.com>
> Signed-off-by: Hongbo Zhang <hongbo.zhang@nxp.com>

Otherwise:
Reviewed-by: Andre Przywara <andre.przywara@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)
> 

  parent reply	other threads:[~2016-05-18 10:07 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-18  9:10 [U-Boot] [PATCH v3 00/11] ARMv7: PSCI: add PSCI v1.0 support macro.wave.z at gmail.com
2016-05-18  9:10 ` [U-Boot] [PATCH v3 01/11] ARM: PSCI: change PSCI function IDs base and offsets macro.wave.z at gmail.com
2016-05-18  9:19   ` Chen-Yu Tsai
2016-05-19  8:40     ` Hongbo Zhang
2016-05-19  8:42       ` Chen-Yu Tsai
2016-05-18 10:07   ` Andre Przywara [this message]
2016-05-19  8:45     ` Hongbo Zhang
2016-05-19  9:07       ` Andre Przywara
2016-05-20 11:26         ` Hongbo Zhang
2016-05-23 15:54   ` Mark Rutland
2016-05-24  6:21     ` Hongbo Zhang
2016-05-18  9:10 ` [U-Boot] [PATCH v3 02/11] ARM: PSCI: change PSCI related macros definition style macro.wave.z at gmail.com
2016-05-18  9:10 ` [U-Boot] [PATCH v3 03/11] ARMv7: PSCI: update function psci_get_cpu_stack_top macro.wave.z at gmail.com
2016-05-18  9:10 ` [U-Boot] [PATCH v3 04/11] ARMv7: PSCI: update the place of saving target PC macro.wave.z at gmail.com
2016-05-18  9:10 ` [U-Boot] [PATCH v3 05/11] ARMv7: PSCI: add codes to save context ID for CPU_ON macro.wave.z at gmail.com
2016-05-18  9:10 ` [U-Boot] [PATCH v3 06/11] ARMv7: PSCI: factor out reusable psci_cpu_on_common macro.wave.z at gmail.com
2016-05-18  9:10 ` [U-Boot] [PATCH v3 07/11] ARMv7: PSCI: add PSCI v1.0 functions skeleton macro.wave.z at gmail.com
2016-05-18 10:39   ` Andre Przywara
2016-05-19  8:23     ` Hongbo Zhang
2016-05-27 17:25       ` York Sun
2016-05-30  2:11         ` Hongbo Zhang
2016-05-18  9:10 ` [U-Boot] [PATCH v3 08/11] ARMv7: PSCI: ls102xa: check target CPU ID before further operations macro.wave.z at gmail.com
2016-05-18  9:23   ` Chen-Yu Tsai
2016-05-19  7:13     ` Hongbo Zhang
2016-05-18  9:10 ` [U-Boot] [PATCH v3 09/11] ARMv7: PSCI: ls102xa: check ALREADY_ON or ON_PENDING for CPU_ON macro.wave.z at gmail.com
2016-05-18  9:10 ` [U-Boot] [PATCH v3 10/11] ARMv7: PSCI: ls102xa: add more PSCI v1.0 functions implemention macro.wave.z at gmail.com
2016-05-18  9:10 ` [U-Boot] [PATCH v3 11/11] ARMv7: PSCI: ls102xa: move secure text section into OCRAM macro.wave.z at gmail.com

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=573C3EE7.3060308@arm.com \
    --to=andre.przywara@arm.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox