From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Rutland Date: Mon, 23 May 2016 16:54:19 +0100 Subject: [U-Boot] [PATCH v3 01/11] ARM: PSCI: change PSCI function IDs base and offsets In-Reply-To: <1463562634-16723-2-git-send-email-hongbo.zhang@nxp.com> References: <1463562634-16723-1-git-send-email-hongbo.zhang@nxp.com> <1463562634-16723-2-git-send-email-hongbo.zhang@nxp.com> Message-ID: <20160523155419.GC4976@leverpostej> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, May 18, 2016 at 05:10:24PM +0800, macro.wave.z at gmail.com wrote: > From: Wang Dongsheng > > 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. I agree that these are the correct valeus for PSCI 0.2, and we must use those IDs for PSCI 0.2+. However, this code is also used on platforms using PSCI 0.1, which did not have well-defined IDs, and relied on them being described in the DT. I fear that this may have the unintended sonequence of breaking those. Does U-Boot patch the DT with the correct IDs per the PSCI 0.1 binding? If so, then things are fine. Thanks, Mark. > Signed-off-by: Wang Dongsheng > Signed-off-by: Hongbo Zhang > --- > 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) > -- > 2.1.4 >