ARM: sunxi: Support PSCI for sun6i and sun8i

PATCH New Chen-Yu Tsai 3 participants 19 messages May 28, 2015 – May 30, 2015

Subsystems (2)

Arm (32-bit) Arm/Allwinner sunXi SoC support

lore

Share & tools

lore

Chen-Yu Tsai

author

2015-05-28 13:25 UTC #0 lore raw

cover ARM: sunxi: Support PSCI for sun6i and sun8i

Hi everyone,

I finally got around to rebasing and fixing up my PSCI stuff. This series adds PSCI support for sun6i and sun8i. This is based on sun7i PSCI support by Marc, and the SMP support routines found in both mainline Linux and Allwinner's Android kernel.

I've tested this on my Hummingbird A31 board and Ippo q8h v5 tablet. Given the A23 SMP routines work on the A33 as well, I see no reason it would not work, so A33 support is included as well, but more testing is always welcome.

Patch 1 and 2 address a build failur on sunxi/next I ran into, which is related to the new NAND SPL support.

Patch 3 adds some comments to the sunxi PSCI code, for easier reading.

Patch 4 moves sun7i PSCI code aside.

Patch 5 adds sun6i PSCI code. Differences from sun7i are the separate PRCM block for power gating/clamps, and support for more than 1 secondary CPU core.

Patch 6 enables PSCI for sun6i.

Patch 7 lets sun8i share PSCI code with sun6i. The routines are the same, except sun8i does not have power clamps.

Patch 8 enables PSCI for sun8i (A23/A33).

Wonder if we still need the SMP support in the kernel?

Regards
ChenYu

Chen-Yu Tsai (8):
sunxi: Only compile board_nand_init() if CONFIG_NAND is set
sunxi: Add extra NAND pins for sun6i
ARM: sunxi: Document registers in PSCI code
ARM: sunxi: Make PSCI code sun7i specific
ARM: sunxi: Add sun6i specific PSCI implementation
ARM: sunxi: Enable PSCI for sun6i
ARM: sunxi: Share sun6i PSCI backend with sun8i
ARM: sunxi: Enable PSCI for sun8i

 arch/arm/cpu/armv7/sunxi/Makefile                 |  4 +-
 arch/arm/cpu/armv7/sunxi/{psci.S => psci_sun6i.S} | 81 +++++++++++++++--------
 arch/arm/cpu/armv7/sunxi/{psci.S => psci_sun7i.S} | 14 ++--
 board/sunxi/Kconfig                               |  9 +++
 board/sunxi/board.c                               |  2 +
 include/configs/sun6i.h                           |  7 ++
 include/configs/sun8i.h                           |  6 ++
 7 files changed, 87 insertions(+), 36 deletions(-)
 copy arch/arm/cpu/armv7/sunxi/{psci.S => psci_sun6i.S} (74%)
 rename arch/arm/cpu/armv7/sunxi/{psci.S => psci_sun7i.S} (94%)

Signature · 1 line

2.1.4

Chen-Yu Tsai

author

2015-05-28 13:25 UTC #1 lore raw

PATCH 1/8 sunxi: Only compile board_nand_init() if CONFIG_NAND is set

There's no need to init NAND if we're not building with NAND support. Wrap board_nand_init() in #ifdefs like we do for board_mmc_init(). This fixes build breaks for sun6i and sun9i. Both platforms do not have CONFIG_NAND_SUNXI_GPC_PORTS defined.

Fixes fbf13d684dcb ("sunxi/nand: Add support to the SPL for loading u-boot

from internal NAND memory")

Signed-off-by Chen-Yu Tsai <[email protected]>

---
 board/sunxi/board.c | 2 ++
 1 file changed, 2 insertions(+)

board/sunxi/board.c +2 -0

index 3f23f26..f27967b 100644
@@ -318,6 +318,7 @@ int board_mmc_init(bd_t *bis)
 }
 #endif
 
+#ifdef CONFIG_NAND
 void board_nand_init(void)
 {
 	unsigned int pin;

@@ -330,6 +331,7 @@ void board_nand_init(void)
 	for (pin = 0; pin < ARRAY_SIZE(ports); pin++)
 		sunxi_gpio_set_cfgpin(SUNXI_GPC(ports[pin]), SUNXI_GPC_NAND);
 }
+#endif
 
 void i2c_init_board(void)
 {

Signature · 1 line

2.1.4

Chen-Yu Tsai

author

2015-05-28 13:25 UTC #2 lore raw

PATCH 2/8 sunxi: Add extra NAND pins for sun6i

In reply to Chen-Yu Tsai

In addition to pins PC00 ~ PC15, PC24, PC25, and PC26 are mapped to NAND0 DQS, CE2, and CE3 respectively.

Signed-off-by Chen-Yu Tsai <[email protected]>

---
 include/configs/sun6i.h | 2 ++
 1 file changed, 2 insertions(+)

include/configs/sun6i.h +2 -0

index 2c24bd2..e1263f6 100644
@@ -22,6 +22,8 @@
 
 #define CONFIG_SUNXI_USB_PHYS	3
 
+#define CONFIG_NAND_SUNXI_GPC_PORTS	{24, 25, 26}
+
 /*
  * Include common sunxi configuration where most the settings are
  */

Signature · 1 line

2.1.4

Chen-Yu Tsai

author

2015-05-28 13:25 UTC #3 lore raw

PATCH 3/8 ARM: sunxi: Document registers in PSCI code

In reply to Chen-Yu Tsai

The PSCI CPU_ON code accesses quite a few registers. Document their names to make it easier to cross reference.

Also explain "lock cpu" and "unlock cpu" as enabling/disabling debug access.

Signed-off-by Chen-Yu Tsai <[email protected]>

---
 arch/arm/cpu/armv7/sunxi/psci.S | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

arch/arm/cpu/armv7/sunxi/psci.S +7 -7

index 7ec0500..bbfeec8 100644
@@ -165,12 +165,12 @@ psci_cpu_on:
 	str	r6, [r5]	@ Reset CPU
 
 	@ l1 invalidate
-	ldr	r6, [r0, #0x184]
+	ldr	r6, [r0, #0x184] @ CPUCFG_GEN_CTRL_REG
 	bic	r6, r6, r4
 	str	r6, [r0, #0x184]
 
-	@ Lock CPU
-	ldr	r6, [r0, #0x1e4]
+	@ Lock CPU (Disable external debug access)
+	ldr	r6, [r0, #0x1e4] @ CPUCFG_DBG_CTL1_REG
 	bic	r6, r6, r4
 	str	r6, [r0, #0x1e4]

@@ -178,13 +178,13 @@ psci_cpu_on:
 	movw	r6, #0x1ff
 	movt	r6, #0
 1:	lsrs	r6, r6, #1
-	str	r6, [r0, #0x1b0]
+	str	r6, [r0, #0x1b0] @ CPU1_PWR_CLAMP
 	bne	1b
 
 	timer_wait r1, TEN_MS
 
 	@ Clear power gating
-	ldr	r6, [r0, #0x1b4]
+	ldr	r6, [r0, #0x1b4] @ CPU1_PWROFF_REG
 	bic	r6, r6, #1
 	str	r6, [r0, #0x1b4]

@@ -192,8 +192,8 @@ psci_cpu_on:
 	mov	r6, #3
 	str	r6, [r5]
 
-	@ Unlock CPU
-	ldr	r6, [r0, #0x1e4]
+	@ Unlock CPU (Enable external debug access)
+	ldr	r6, [r0, #0x1e4] @ CPUCFG_DBG_CTL1_REG
 	orr	r6, r6, r4
 	str	r6, [r0, #0x1e4]

Signature · 1 line

2.1.4

Maxime Ripard

2015-05-28 15:23 UTC #4 lore raw

PATCH 3/8 ARM: sunxi: Document registers in PSCI code

Quoted 16 lines · Chen-Yu Tsai

The PSCI CPU_ON code accesses quite a few registers. Document their names to make it easier to cross reference.
Also explain "lock cpu" and "unlock cpu" as enabling/disabling debug access.
Signed-off-by Chen-Yu Tsai <[email protected]>
---
 arch/arm/cpu/armv7/sunxi/psci.S | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)
arch/arm/cpu/armv7/sunxi/psci.S +0 -0
index 7ec0500..bbfeec8 100644

arch/arm/cpu/armv7/sunxi/psci.S psci_cpu_on: · line 168

Quoted 5 lines

arch/arm/cpu/armv7/sunxi/psci.S +1 -1

@@ -165,12 +165,12 @@ psci_cpu_on:
 	str	r6, [r5]	@ Reset CPU
 
 	@ l1 invalidate
-	ldr	r6, [r0, #0x184]
+	ldr	r6, [r0, #0x184] @ CPUCFG_GEN_CTRL_REG

Can't we use some defines instead? You seem to do that on a few registers (mostly the GIC's), why not for these?

Maxime

Signature · 10 lines

Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150528/762a5f6e/attachment.sig>

Chen-Yu Tsai

author

2015-05-29 03:18 UTC #5 lore raw

PATCH 3/8 ARM: sunxi: Document registers in PSCI code

Quoted 19 lines · Maxime Ripard

Quoted 18 lines · Chen-Yu Tsai
The PSCI CPU_ON code accesses quite a few registers. Document their names to make it easier to cross reference.
Also explain "lock cpu" and "unlock cpu" as enabling/disabling debug access.
Signed-off-by Chen-Yu Tsai <[email protected]>
---
 arch/arm/cpu/armv7/sunxi/psci.S | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)
arch/arm/cpu/armv7/sunxi/psci.S +0 -0
index 7ec0500..bbfeec8 100644
@@ -165,12 +165,12 @@ psci_cpu_on:
      str     r6, [r5]        @ Reset CPU
 

Quoted 6 lines

arch/arm/cpu/armv7/sunxi/psci.S psci_cpu_on: · line 168
Quoted 3 lines
arch/arm/cpu/armv7/sunxi/psci.S +1 -1
      @ l1 invalidate
-     ldr     r6, [r0, #0x184]
+     ldr     r6, [r0, #0x184] @ CPUCFG_GEN_CTRL_REG
Can't we use some defines instead? You seem to do that on a few registers (mostly the GIC's), why not for these?

Sure. Though to be clear GIC wasn't my work. :p

For sun6i there's a bit of offset calculating, so
we can't use defines for them all.

ChenYu

Chen-Yu Tsai

author

2015-05-28 13:25 UTC #6 lore raw

PATCH 4/8 ARM: sunxi: Make PSCI code sun7i specific

In reply to Chen-Yu Tsai

The PSCI code only works for sun7i. Rename it with _sun7i suffix, and build only if building for sun7i.

This paves the way for adding PSCI support for other platforms.

Signed-off-by Chen-Yu Tsai <[email protected]>

---
 arch/arm/cpu/armv7/sunxi/Makefile                 | 2 +-
 arch/arm/cpu/armv7/sunxi/{psci.S => psci_sun7i.S} | 0
 2 files changed, 1 insertion(+), 1 deletion(-)
 rename arch/arm/cpu/armv7/sunxi/{psci.S => psci_sun7i.S} (100%)

arch/arm/cpu/armv7/sunxi/Makefile +1 -1

index 7ef6b4c..85fbc85 100644
@@ -35,7 +35,7 @@ obj-$(CONFIG_AXP221_POWER)	+= pmic_bus.o
 
 ifndef CONFIG_SPL_BUILD
 ifdef CONFIG_ARMV7_PSCI
-obj-y	+= psci.o
+obj-$(CONFIG_MACH_SUN7I)	+= psci_sun7i.o
 endif
 endif

arch/arm/cpu/armv7/sunxi/psci_sun7i.S renamed from arch/arm/cpu/armv7/sunxi/psci.S +0 -0

similarity index 100%
rename from arch/arm/cpu/armv7/sunxi/psci.S
rename to arch/arm/cpu/armv7/sunxi/psci_sun7i.S

Signature · 1 line

2.1.4

Chen-Yu Tsai

author

2015-05-28 13:25 UTC #7 lore raw

PATCH 5/8 ARM: sunxi: Add sun6i specific PSCI implementation

In reply to Chen-Yu Tsai

This adds PSCI support for sun6i. So far it only supports the PWR_ON method.

Signed-off-by Chen-Yu Tsai <[email protected]>

---
 arch/arm/cpu/armv7/sunxi/Makefile     |   1 +
 arch/arm/cpu/armv7/sunxi/psci_sun6i.S | 276 ++++++++++++++++++++++++++++++++++
 2 files changed, 277 insertions(+)
 create mode 100644 arch/arm/cpu/armv7/sunxi/psci_sun6i.S

arch/arm/cpu/armv7/sunxi/Makefile +1 -0

index 85fbc85..4b783e0 100644
@@ -35,6 +35,7 @@ obj-$(CONFIG_AXP221_POWER)	+= pmic_bus.o
 
 ifndef CONFIG_SPL_BUILD
 ifdef CONFIG_ARMV7_PSCI
+obj-$(CONFIG_MACH_SUN6I)	+= psci_sun6i.o
 obj-$(CONFIG_MACH_SUN7I)	+= psci_sun7i.o
 endif
 endif

arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +276 -0

276 lines not on this page · raw message

Signature · 1 line

2.1.4

Maxime Ripard

2015-05-28 15:22 UTC #8 lore raw

PATCH 5/8 ARM: sunxi: Add sun6i specific PSCI implementation

Quoted 83 lines · Chen-Yu Tsai

This adds PSCI support for sun6i. So far it only supports the PWR_ON method.

Signed-off-by Chen-Yu Tsai <[email protected]>

---
 arch/arm/cpu/armv7/sunxi/Makefile     |   1 +
 arch/arm/cpu/armv7/sunxi/psci_sun6i.S | 276 ++++++++++++++++++++++++++++++++++
 2 files changed, 277 insertions(+)
 create mode 100644 arch/arm/cpu/armv7/sunxi/psci_sun6i.S

arch/arm/cpu/armv7/sunxi/Makefile +1 -0

index 85fbc85..4b783e0 100644
@@ -35,6 +35,7 @@ obj-$(CONFIG_AXP221_POWER)	+= pmic_bus.o
 
 ifndef CONFIG_SPL_BUILD
 ifdef CONFIG_ARMV7_PSCI
+obj-$(CONFIG_MACH_SUN6I)	+= psci_sun6i.o
 obj-$(CONFIG_MACH_SUN7I)	+= psci_sun7i.o
 endif
 endif

arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +55 -0

new file mode 100644
index 0000000..2516804
@@ -0,0 +1,276 @@
+/*
+ * Copyright (C) 2015 - Chen-Yu Tsai
+ * Author: Chen-Yu Tsai <[email protected]>
+ *
+ * Based on psci_sun7i.S by Marc Zyngier <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include <config.h>
+#include <asm/gic.h>
+#include <asm/macro.h>
+#include <asm/psci.h>
+#include <asm/arch/cpu.h>
+
+/*
+ * Memory layout:
+ *
+ * SECURE_RAM to text_end :
+ *	._secure_text section
+ * text_end to ALIGN_PAGE(text_end):
+ *	nothing
+ * ALIGN_PAGE(text_end) to ALIGN_PAGE(text_end) + 0x1000)
+ *	1kB of stack per CPU (4 CPUs max).
+ */
+
+	.pushsection ._secure.text, "ax"
+
+	.arch_extension sec
+
+#define	ONE_MS			(CONFIG_TIMER_CLK_FREQ / 1000)
+#define	TEN_MS			(10 * ONE_MS)
+#define	GICD_BASE		0x1c81000
+#define	GICC_BASE		0x1c82000
+
+.macro	timer_wait	reg, ticks
+	@ Program CNTP_TVAL
+	movw	\reg, #(\ticks & 0xffff)
+	movt	\reg, #(\ticks >> 16)
+	mcr	p15, 0, \reg, c14, c2, 0
+	isb
+	@ Enable physical timer, mask interrupt
+	mov	\reg, #3
+	mcr	p15, 0, \reg, c14, c2, 1
+	@ Poll physical timer until ISTATUS is on

arch/arm/cpu/armv7/sunxi/psci_sun6i.S line 65

Quoted 10 lines

arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +10 -0

+1:	isb
+	mrc	p15, 0, \reg, c14, c2, 1
+	ands	\reg, \reg, #4
+	bne	1b
+	@ Disable timer
+	mov	\reg, #0
+	mcr	p15, 0, \reg, c14, c2, 1
+	isb
+.endm
+

I think I saw some patch to factorize that out. In the Tegra K1 PSCI patches iirc.

Quoted 9 lines

arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +9 -0

+.globl	psci_fiq_enter
+psci_fiq_enter:
+	push	{r0-r12}
+
+	@ Switch to secure
+	mrc	p15, 0, r7, c1, c1, 0
+	bic	r8, r7, #1
+	mcr	p15, 0, r8, c1, c1, 0
+	isb

arch/arm/cpu/armv7/sunxi/psci_sun6i.S line 84

Quoted 10 lines

arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +10 -0

+
+	@ Validate reason based on IAR and acknowledge
+	movw	r8, #(GICC_BASE & 0xffff)
+	movt	r8, #(GICC_BASE >> 16)
+	ldr	r9, [r8, #GICC_IAR]
+	movw	r10, #0x3ff
+	movt	r10, #0
+	cmp	r9, r10			@ skip spurious interrupt 1023
+	beq	out
+	movw	r10, #0x3fe		@ ...and 1022

Maybe we could add some defines for these spurious interrupts values ?

Quoted 26 lines

arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +26 -0

+.globl	psci_arch_init
+psci_arch_init:
+	mov	r6, lr
+
+	movw	r4, #(GICD_BASE & 0xffff)
+	movt	r4, #(GICD_BASE >> 16)
+
+	ldr	r5, [r4, #GICD_IGROUPRn]
+	bic	r5, r5, #(1 << 15) 	@ SGI15 as Group-0
+	str	r5, [r4, #GICD_IGROUPRn]
+
+	mov	r5, #0			@ Set SGI15 priority to 0
+	strb	r5, [r4, #(GICD_IPRIORITYRn + 15)]
+
+	add	r4, r4, #0x1000		@ GICC address
+
+	mov	r5, #0xff
+	str	r5, [r4, #GICC_PMR]	@ Be cool with non-secure
+
+	ldr	r5, [r4, #GICC_CTLR]
+	orr	r5, r5, #(1 << 3)	@ Switch FIQEn on
+	str	r5, [r4, #GICC_CTLR]
+
+	mrc	p15, 0, r5, c1, c1, 0	@ Read SCR
+	orr	r5, r5, #4		@ Enable FIQ in monitor mode
+	bic	r5, r5, #1		@ Secure mode

arch/arm/cpu/armv7/sunxi/psci_sun6i.S line 274

Quoted 10 lines

arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +10 -0

+	mcr	p15, 0, r5, c1, c1, 0	@ Write SCR
+	isb
+
+	bl	psci_get_cpu_id		@ CPU ID => r0
+	bl	psci_get_cpu_stack_top	@ stack top => r0
+	mov	sp, r0
+
+	bx	r6
+
+	.globl psci_text_end

Isn't it exactly the same function than the A20's? Maybe that can be shared?

Thanks,
Maxime

Signature · 10 lines

Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150528/faf0f502/attachment.sig>

Chen-Yu Tsai

author

2015-05-29 03:08 UTC #9 lore raw

PATCH 5/8 ARM: sunxi: Add sun6i specific PSCI implementation

Quoted 89 lines · Maxime Ripard

Quoted 88 lines · Chen-Yu Tsai

This adds PSCI support for sun6i. So far it only supports the PWR_ON method.

Signed-off-by Chen-Yu Tsai <[email protected]>

---
 arch/arm/cpu/armv7/sunxi/Makefile     |   1 +
 arch/arm/cpu/armv7/sunxi/psci_sun6i.S | 276 ++++++++++++++++++++++++++++++++++
 2 files changed, 277 insertions(+)
 create mode 100644 arch/arm/cpu/armv7/sunxi/psci_sun6i.S

arch/arm/cpu/armv7/sunxi/Makefile +1 -0

index 85fbc85..4b783e0 100644
@@ -35,6 +35,7 @@ obj-$(CONFIG_AXP221_POWER)  += pmic_bus.o
 
 ifndef CONFIG_SPL_BUILD
 ifdef CONFIG_ARMV7_PSCI
+obj-$(CONFIG_MACH_SUN6I)     += psci_sun6i.o
 obj-$(CONFIG_MACH_SUN7I)     += psci_sun7i.o
 endif
 endif

arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +60 -0

new file mode 100644
index 0000000..2516804
@@ -0,0 +1,276 @@
+/*
+ * Copyright (C) 2015 - Chen-Yu Tsai
+ * Author: Chen-Yu Tsai <[email protected]>
+ *
+ * Based on psci_sun7i.S by Marc Zyngier <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include <config.h>
+#include <asm/gic.h>
+#include <asm/macro.h>
+#include <asm/psci.h>
+#include <asm/arch/cpu.h>
+
+/*
+ * Memory layout:
+ *
+ * SECURE_RAM to text_end :
+ *   ._secure_text section
+ * text_end to ALIGN_PAGE(text_end):
+ *   nothing
+ * ALIGN_PAGE(text_end) to ALIGN_PAGE(text_end) + 0x1000)
+ *   1kB of stack per CPU (4 CPUs max).
+ */
+
+     .pushsection ._secure.text, "ax"
+
+     .arch_extension sec
+
+#define      ONE_MS                  (CONFIG_TIMER_CLK_FREQ / 1000)
+#define      TEN_MS                  (10 * ONE_MS)
+#define      GICD_BASE               0x1c81000
+#define      GICC_BASE               0x1c82000
+
+.macro       timer_wait      reg, ticks
+     @ Program CNTP_TVAL
+     movw    \reg, #(\ticks & 0xffff)
+     movt    \reg, #(\ticks >> 16)
+     mcr     p15, 0, \reg, c14, c2, 0
+     isb
+     @ Enable physical timer, mask interrupt
+     mov     \reg, #3
+     mcr     p15, 0, \reg, c14, c2, 1
+     @ Poll physical timer until ISTATUS is on
+1:   isb
+     mrc     p15, 0, \reg, c14, c2, 1
+     ands    \reg, \reg, #4
+     bne     1b
+     @ Disable timer

Quoted 8 lines

arch/arm/cpu/armv7/sunxi/psci_sun6i.S line 65
Quoted 5 lines
arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +5 -0
+     mov     \reg, #0
+     mcr     p15, 0, \reg, c14, c2, 1
+     isb
+.endm
+
I think I saw some patch to factorize that out. In the Tegra K1 PSCI patches iirc.

Waiting for v2: https://patchwork.ozlabs.org/patch/471694/

Quoted 13 lines

arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +13 -0

+.globl       psci_fiq_enter
+psci_fiq_enter:
+     push    {r0-r12}
+
+     @ Switch to secure
+     mrc     p15, 0, r7, c1, c1, 0
+     bic     r8, r7, #1
+     mcr     p15, 0, r8, c1, c1, 0
+     isb
+
+     @ Validate reason based on IAR and acknowledge
+     movw    r8, #(GICC_BASE & 0xffff)
+     movt    r8, #(GICC_BASE >> 16)

Quoted 8 lines

arch/arm/cpu/armv7/sunxi/psci_sun6i.S line 84
Quoted 6 lines
arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +6 -0
+     ldr     r9, [r8, #GICC_IAR]
+     movw    r10, #0x3ff
+     movt    r10, #0
+     cmp     r9, r10                 @ skip spurious interrupt 1023
+     beq     out
+     movw    r10, #0x3fe             @ ...and 1022
Maybe we could add some defines for these spurious interrupts values ?

From the GIC spec:

The read returns a spurious interrupt ID of 1023 if any of the following apply:
- forwarding of interrupts by the Distributor to the CPU interface is disabled
- signaling of interrupts by the CPU interface to the connected processor is disabled
- no pending interrupt on the CPU interface has sufficient priority for the interface to signal it to the processor.

Likewise, 1022 means group 0 secure interrupts aren't forwarded to the processor.

So my understanding is that 1022/1023 just means there's nothing for the processor to process. Maybe a note referring to the spec would be enough?

Quoted 31 lines

arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +31 -0

+.globl       psci_arch_init
+psci_arch_init:
+     mov     r6, lr
+
+     movw    r4, #(GICD_BASE & 0xffff)
+     movt    r4, #(GICD_BASE >> 16)
+
+     ldr     r5, [r4, #GICD_IGROUPRn]
+     bic     r5, r5, #(1 << 15)      @ SGI15 as Group-0
+     str     r5, [r4, #GICD_IGROUPRn]
+
+     mov     r5, #0                  @ Set SGI15 priority to 0
+     strb    r5, [r4, #(GICD_IPRIORITYRn + 15)]
+
+     add     r4, r4, #0x1000         @ GICC address
+
+     mov     r5, #0xff
+     str     r5, [r4, #GICC_PMR]     @ Be cool with non-secure
+
+     ldr     r5, [r4, #GICC_CTLR]
+     orr     r5, r5, #(1 << 3)       @ Switch FIQEn on
+     str     r5, [r4, #GICC_CTLR]
+
+     mrc     p15, 0, r5, c1, c1, 0   @ Read SCR
+     orr     r5, r5, #4              @ Enable FIQ in monitor mode
+     bic     r5, r5, #1              @ Secure mode
+     mcr     p15, 0, r5, c1, c1, 0   @ Write SCR
+     isb
+
+     bl      psci_get_cpu_id         @ CPU ID => r0
+     bl      psci_get_cpu_stack_top  @ stack top => r0

Quoted 8 lines

arch/arm/cpu/armv7/sunxi/psci_sun6i.S line 274
Quoted 5 lines
arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +5 -0
+     mov     sp, r0
+
+     bx      r6
+
+     .globl psci_text_end
Isn't it exactly the same function than the A20's? Maybe that can be shared?

Good idea. This might even be shared by some other platforms. "psci_text_end" should always be linked in last though.

ChenYu

Hans de Goede

2015-05-29 07:40 UTC #10 lore raw

PATCH 5/8 ARM: sunxi: Add sun6i specific PSCI implementation

Hi,

Quoted 100 lines · Chen-Yu Tsai

Quoted 89 lines · Maxime Ripard

Quoted 88 lines · Chen-Yu Tsai

This adds PSCI support for sun6i. So far it only supports the PWR_ON method.

Signed-off-by Chen-Yu Tsai <[email protected]>

---
  arch/arm/cpu/armv7/sunxi/Makefile     |   1 +
  arch/arm/cpu/armv7/sunxi/psci_sun6i.S | 276 ++++++++++++++++++++++++++++++++++
  2 files changed, 277 insertions(+)

create mode 100644 arch/arm/cpu/armv7/sunxi/psci_sun6i.S

arch/arm/cpu/armv7/sunxi/Makefile +1 -0

index 85fbc85..4b783e0 100644
@@ -35,6 +35,7 @@ obj-$(CONFIG_AXP221_POWER)  += pmic_bus.o
 
  ifndef CONFIG_SPL_BUILD
  ifdef CONFIG_ARMV7_PSCI
+obj-$(CONFIG_MACH_SUN6I)     += psci_sun6i.o
  obj-$(CONFIG_MACH_SUN7I)     += psci_sun7i.o
  endif
  endif

arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +60 -0

new file mode 100644
index 0000000..2516804
@@ -0,0 +1,276 @@
+/*
+ * Copyright (C) 2015 - Chen-Yu Tsai
+ * Author: Chen-Yu Tsai <[email protected]>
+ *
+ * Based on psci_sun7i.S by Marc Zyngier <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include <config.h>
+#include <asm/gic.h>
+#include <asm/macro.h>
+#include <asm/psci.h>
+#include <asm/arch/cpu.h>
+
+/*
+ * Memory layout:
+ *
+ * SECURE_RAM to text_end :
+ *   ._secure_text section
+ * text_end to ALIGN_PAGE(text_end):
+ *   nothing
+ * ALIGN_PAGE(text_end) to ALIGN_PAGE(text_end) + 0x1000)
+ *   1kB of stack per CPU (4 CPUs max).
+ */
+
+     .pushsection ._secure.text, "ax"
+
+     .arch_extension sec
+
+#define      ONE_MS                  (CONFIG_TIMER_CLK_FREQ / 1000)
+#define      TEN_MS                  (10 * ONE_MS)
+#define      GICD_BASE               0x1c81000
+#define      GICC_BASE               0x1c82000
+
+.macro       timer_wait      reg, ticks
+     @ Program CNTP_TVAL
+     movw    \reg, #(\ticks & 0xffff)
+     movt    \reg, #(\ticks >> 16)
+     mcr     p15, 0, \reg, c14, c2, 0
+     isb
+     @ Enable physical timer, mask interrupt
+     mov     \reg, #3
+     mcr     p15, 0, \reg, c14, c2, 1
+     @ Poll physical timer until ISTATUS is on
+1:   isb
+     mrc     p15, 0, \reg, c14, c2, 1
+     ands    \reg, \reg, #4
+     bne     1b
+     @ Disable timer

Quoted 8 lines

arch/arm/cpu/armv7/sunxi/psci_sun6i.S line 65
Quoted 5 lines
arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +5 -0
+     mov     \reg, #0
+     mcr     p15, 0, \reg, c14, c2, 1
+     isb
+.endm
+
I think I saw some patch to factorize that out. In the Tegra K1 PSCI patches iirc.

Quoted 1 line

Waiting for v2: https://patchwork.ozlabs.org/patch/471694/

Note I plan to merge this patch-set as is. I agree that factoring out this bit into a helper is a good idea, but that can be done as a later cleanup.

Regards,

Hans

Maxime Ripard

2015-05-29 07:54 UTC #11 lore raw

PATCH 5/8 ARM: sunxi: Add sun6i specific PSCI implementation

In reply to Chen-Yu Tsai

Quoted 33 lines · Chen-Yu Tsai

Quoted 13 lines
Quoted 13 lines
+.globl psci_fiq_enter
arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +12 -0
+psci_fiq_enter:
+     push    {r0-r12}
+
+     @ Switch to secure
+     mrc     p15, 0, r7, c1, c1, 0
+     bic     r8, r7, #1
+     mcr     p15, 0, r8, c1, c1, 0
+     isb
+
+     @ Validate reason based on IAR and acknowledge
+     movw    r8, #(GICC_BASE & 0xffff)
+     movt    r8, #(GICC_BASE >> 16)
Quoted 8 lines
arch/arm/cpu/armv7/sunxi/psci_sun6i.S line 84
Quoted 6 lines
arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +6 -0
+     ldr     r9, [r8, #GICC_IAR]
+     movw    r10, #0x3ff
+     movt    r10, #0
+     cmp     r9, r10                 @ skip spurious interrupt 1023
+     beq     out
+     movw    r10, #0x3fe             @ ...and 1022
Maybe we could add some defines for these spurious interrupts values ?
From the GIC spec:
The read returns a spurious interrupt ID of 1023 if any of the following apply:
- forwarding of interrupts by the Distributor to the CPU interface is disabled
- signaling of interrupts by the CPU interface to the connected processor is disabled
- no pending interrupt on the CPU interface has sufficient priority for the interface to signal it to the processor.

Quoted 6 lines

Likewise, 1022 means group 0 secure interrupts aren't forwarded to the processor.
So my understanding is that 1022/1023 just means there's nothing for the processor to process. Maybe a note referring to the spec would be enough?

Sorry, that's not what I meant.

What I meant was to do

#define GIC_SPURIOUS_INTERRUPT 1023

movw r10, #GIC_SPURIOUS_INTERRUPT.

Quoted 41 lines

Quoted 32 lines

+.globl psci_arch_init

arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +30 -0

+psci_arch_init:
+     mov     r6, lr
+
+     movw    r4, #(GICD_BASE & 0xffff)
+     movt    r4, #(GICD_BASE >> 16)
+
+     ldr     r5, [r4, #GICD_IGROUPRn]
+     bic     r5, r5, #(1 << 15)      @ SGI15 as Group-0
+     str     r5, [r4, #GICD_IGROUPRn]
+
+     mov     r5, #0                  @ Set SGI15 priority to 0
+     strb    r5, [r4, #(GICD_IPRIORITYRn + 15)]
+
+     add     r4, r4, #0x1000         @ GICC address
+
+     mov     r5, #0xff
+     str     r5, [r4, #GICC_PMR]     @ Be cool with non-secure
+
+     ldr     r5, [r4, #GICC_CTLR]
+     orr     r5, r5, #(1 << 3)       @ Switch FIQEn on
+     str     r5, [r4, #GICC_CTLR]
+
+     mrc     p15, 0, r5, c1, c1, 0   @ Read SCR
+     orr     r5, r5, #4              @ Enable FIQ in monitor mode
+     bic     r5, r5, #1              @ Secure mode
+     mcr     p15, 0, r5, c1, c1, 0   @ Write SCR
+     isb
+
+     bl      psci_get_cpu_id         @ CPU ID => r0
+     bl      psci_get_cpu_stack_top  @ stack top => r0

Quoted 8 lines

arch/arm/cpu/armv7/sunxi/psci_sun6i.S line 274
Quoted 5 lines
arch/arm/cpu/armv7/sunxi/psci_sun6i.S new file +5 -0
+     mov     sp, r0
+
+     bx      r6
+
+     .globl psci_text_end
Isn't it exactly the same function than the A20's? Maybe that can be shared?

Quoted 2 lines

Good idea. This might even be shared by some other platforms. "psci_text_end" should always be linked in last though.

Maybe that can be generated directly by the linker script at the end of the PSCI text section?

Maxime

Signature · 10 lines

Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150529/e85400fc/attachment.sig>

Chen-Yu Tsai

author

2015-05-28 13:25 UTC #12 lore raw

PATCH 6/8 ARM: sunxi: Enable PSCI for sun6i

In reply to Chen-Yu Tsai

Now that we have a PSCI backend for sun6i, enable it.

Signed-off-by Chen-Yu Tsai <[email protected]>

---
 board/sunxi/Kconfig     | 3 +++
 include/configs/sun6i.h | 5 +++++
 2 files changed, 8 insertions(+)

board/sunxi/Kconfig +3 -0

index c6c876d..64d2af8 100644
@@ -35,8 +35,11 @@ config MACH_SUN5I
 config MACH_SUN6I
 	bool "sun6i (Allwinner A31)"
 	select CPU_V7
+	select CPU_V7_HAS_NONSEC
+	select CPU_V7_HAS_VIRT
 	select SUNXI_GEN_SUN6I
 	select SUPPORT_SPL
+	select ARMV7_BOOT_SEC_DEFAULT if OLD_SUNXI_KERNEL_COMPAT
 
 config MACH_SUN7I
 	bool "sun7i (Allwinner A20)"

include/configs/sun6i.h +5 -0

index e1263f6..a0ebc7e 100644
@@ -22,6 +22,11 @@
 
 #define CONFIG_SUNXI_USB_PHYS	3
 
+#define CONFIG_ARMV7_PSCI		1
+#define CONFIG_ARMV7_PSCI_NR_CPUS	4
+#define CONFIG_ARMV7_SECURE_BASE	SUNXI_SRAM_B_BASE
+#define CONFIG_TIMER_CLK_FREQ		24000000
+
 #define CONFIG_NAND_SUNXI_GPC_PORTS	{24, 25, 26}
 
 /*

Signature · 1 line

2.1.4

Chen-Yu Tsai

author

2015-05-28 13:25 UTC #13 lore raw

PATCH 7/8 ARM: sunxi: Share sun6i PSCI backend with sun8i

In reply to Chen-Yu Tsai

sun8i can share the PSCI backend with sun6i. Only difference is sun8i does not have CPU power clamp controls.

Signed-off-by Chen-Yu Tsai <[email protected]>

---
 arch/arm/cpu/armv7/sunxi/Makefile     | 1 +
 arch/arm/cpu/armv7/sunxi/psci_sun6i.S | 4 ++++
 2 files changed, 5 insertions(+)

arch/arm/cpu/armv7/sunxi/Makefile +1 -0

index 4b783e0..76c7e55 100644
@@ -37,6 +37,7 @@ ifndef CONFIG_SPL_BUILD
 ifdef CONFIG_ARMV7_PSCI
 obj-$(CONFIG_MACH_SUN6I)	+= psci_sun6i.o
 obj-$(CONFIG_MACH_SUN7I)	+= psci_sun7i.o
+obj-$(CONFIG_MACH_SUN8I)	+= psci_sun6i.o
 endif
 endif

arch/arm/cpu/armv7/sunxi/psci_sun6i.S +4 -0

index 2516804..d4cb51e 100644
@@ -124,11 +124,13 @@ psci_fiq_enter:
 	str	r10, [r8, #0x100]
 	timer_wait r10, ONE_MS
 
+#ifdef CONFIG_MACH_SUN6I
 	@ Activate power clamp
 	lsl	r12, r9, #2		@ x4
 	add	r12, r12, r8
 	mov	r10, #0xff
 	str	r10, [r12, #0x140]
+#endif
 
 	movw	r8, #(SUN6I_CPUCFG_BASE & 0xffff)
 	movt	r8, #(SUN6I_CPUCFG_BASE >> 16)

@@ -186,6 +188,7 @@ psci_cpu_on:
 	movw	r0, #(SUNXI_PRCM_BASE & 0xffff)
 	movt	r0, #(SUNXI_PRCM_BASE >> 16)
 
+#ifdef CONFIG_MACH_SUN6I
 	@ Release power clamp
 	lsl	r5, r1, #2	@ 1 register per CPU
 	add	r5, r5, r0	@ PRCM

@@ -194,6 +197,7 @@ psci_cpu_on:
 1:	lsrs	r6, r6, #1
 	str	r6, [r5, #0x140] @ CPUx_PWR_CLAMP
 	bne	1b
+#endif
 
 	timer_wait r6, TEN_MS

Signature · 1 line

2.1.4

Chen-Yu Tsai

author

2015-05-28 13:25 UTC #14 lore raw

PATCH 8/8 ARM: sunxi: Enable PSCI for sun8i

In reply to Chen-Yu Tsai

sun8i uses the same PSCI backend as sun6i, without power clamps. Since there is no secure SRAM, the backend is placed at the end of DRAM.

Signed-off-by Chen-Yu Tsai <[email protected]>

---
 board/sunxi/Kconfig     | 6 ++++++
 include/configs/sun8i.h | 6 ++++++
 2 files changed, 12 insertions(+)

board/sunxi/Kconfig +6 -0

index 64d2af8..8e27396 100644
@@ -53,14 +53,20 @@ config MACH_SUN7I
 config MACH_SUN8I_A23
 	bool "sun8i (Allwinner A23)"
 	select CPU_V7
+	select CPU_V7_HAS_NONSEC
+	select CPU_V7_HAS_VIRT
 	select SUNXI_GEN_SUN6I
 	select SUPPORT_SPL
+	select ARMV7_BOOT_SEC_DEFAULT if OLD_SUNXI_KERNEL_COMPAT
 
 config MACH_SUN8I_A33
 	bool "sun8i (Allwinner A33)"
 	select CPU_V7
+	select CPU_V7_HAS_NONSEC
+	select CPU_V7_HAS_VIRT
 	select SUNXI_GEN_SUN6I
 	select SUPPORT_SPL
+	select ARMV7_BOOT_SEC_DEFAULT if OLD_SUNXI_KERNEL_COMPAT
 
 config MACH_SUN9I
 	bool "sun9i (Allwinner A80)"

include/configs/sun8i.h +6 -0

index cd33758..fe8c511 100644
@@ -20,11 +20,17 @@
 
 #define CONFIG_SUNXI_USB_PHYS	2
 
+#define CONFIG_ARMV7_PSCI		1
 #if defined(CONFIG_MACH_SUN8I_A23)
+#define CONFIG_ARMV7_PSCI_NR_CPUS	2
 #define CONFIG_NAND_SUNXI_GPC_PORTS	{16, 17, 18}
 #elif defined(CONFIG_MACH_SUN8I_A33)
+#define CONFIG_ARMV7_PSCI_NR_CPUS	4
 #define CONFIG_NAND_SUNXI_GPC_PORTS	{16}
+#else
+#error Unsupported sun8i variant
 #endif
+#define CONFIG_TIMER_CLK_FREQ		24000000
 
 /*
  * Include common sunxi configuration where most the settings are

Signature · 1 line

2.1.4

Maxime Ripard

2015-05-28 15:31 UTC #15 lore raw

cover ARM: sunxi: Support PSCI for sun6i and sun8i

In reply to Chen-Yu Tsai

Hi Chen-Yu,

Quoted 7 lines · Chen-Yu Tsai

Hi everyone,
I finally got around to rebasing and fixing up my PSCI stuff. This series adds PSCI support for sun6i and sun8i. This is based on sun7i PSCI support by Marc, and the SMP support routines found in both mainline Linux and Allwinner's Android kernel.

This is great, thanks!

Quoted 4 lines

I've tested this on my Hummingbird A31 board and Ippo q8h v5 tablet. Given the A23 SMP routines work on the A33 as well, I see no reason it would not work, so A33 support is included as well, but more testing is always welcome.

Yeah, and we can expect it to work on the H3 and R16 as well.

Quoted 12 lines

Patch 1 and 2 address a build failur on sunxi/next I ran into, which is related to the new NAND SPL support.
Patch 3 adds some comments to the sunxi PSCI code, for easier reading.
Patch 4 moves sun7i PSCI code aside.
Patch 5 adds sun6i PSCI code. Differences from sun7i are the separate PRCM block for power gating/clamps, and support for more than 1 secondary CPU core.

Quoted 8 lines

Patch 6 enables PSCI for sun6i.
Patch 7 lets sun8i share PSCI code with sun6i. The routines are the same, except sun8i does not have power clamps.
Patch 8 enables PSCI for sun8i (A23/A33).
Wonder if we still need the SMP support in the kernel?

Eventually, we can remove it.

However, I think we should wait a bit before doing so. We have a few things that will be required before that:
- That these patches get merged (obviously)
- That we have a way to replace the Allwinner bootloader in NAND on these SoCs. That's especially true on the A31, which will boot on NAND by default. We don't look so far off though.
- That these patches get into enough u-boot releases that we can expect that most of the users to have been moved over (I'd say something like 2-3 releases)

Maxime

Signature · 10 lines

Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150528/f00af6ad/attachment.sig>

Chen-Yu Tsai

author

2015-05-29 03:15 UTC #16 lore raw

cover ARM: sunxi: Support PSCI for sun6i and sun8i

Hi,

Quoted 39 lines · Maxime Ripard

Hi Chen-Yu,
Quoted 7 lines · Chen-Yu Tsai
Hi everyone,
I finally got around to rebasing and fixing up my PSCI stuff. This series adds PSCI support for sun6i and sun8i. This is based on sun7i PSCI support by Marc, and the SMP support routines found in both mainline Linux and Allwinner's Android kernel.
This is great, thanks!
Quoted 4 lines
I've tested this on my Hummingbird A31 board and Ippo q8h v5 tablet. Given the A23 SMP routines work on the A33 as well, I see no reason it would not work, so A33 support is included as well, but more testing is always welcome.
Yeah, and we can expect it to work on the H3 and R16 as well.
Quoted 19 lines
Patch 1 and 2 address a build failur on sunxi/next I ran into, which is related to the new NAND SPL support.
Patch 3 adds some comments to the sunxi PSCI code, for easier reading.
Patch 4 moves sun7i PSCI code aside.
Patch 5 adds sun6i PSCI code. Differences from sun7i are the separate PRCM block for power gating/clamps, and support for more than 1 secondary CPU core.
Patch 6 enables PSCI for sun6i.
Patch 7 lets sun8i share PSCI code with sun6i. The routines are the same, except sun8i does not have power clamps.
Patch 8 enables PSCI for sun8i (A23/A33).

Quoted 7 lines

Quoted 1 line
Wonder if we still need the SMP support in the kernel?
Eventually, we can remove it.
However, I think we should wait a bit before doing so. We have a few things that will be required before that:
- That these patches get merged (obviously)

Let's see what the maintainers say. :)

Quoted 3 lines

- That we have a way to replace the Allwinner bootloader in NAND on these SoCs. That's especially true on the A31, which will boot on NAND by default. We don't look so far off though.

Hmm... Is this somehow related? AFAIK most people can use mmc0, and at least my NAND boot0 seems to check mmc0 for a valid signature, and reboot into that.

Quoted 3 lines

- That these patches get into enough u-boot releases that we can expect that most of the users to have been moved over (I'd say something like 2-3 releases)

Yeah, that seems like a safe bet.

Thanks!
ChenYu

Maxime Ripard

2015-05-29 07:49 UTC #17 lore raw

cover ARM: sunxi: Support PSCI for sun6i and sun8i

Quoted 7 lines · Chen-Yu Tsai

Quoted 3 lines
- That we have a way to replace the Allwinner bootloader in NAND on these SoCs. That's especially true on the A31, which will boot on NAND by default. We don't look so far off though.
Hmm... Is this somehow related? AFAIK most people can use mmc0, and at least my NAND boot0 seems to check mmc0 for a valid signature, and reboot into that.

I guess it depends on the A31 board then. I have a board here that will completely ignore the MMC unless you press a button.

Maxime

Signature · 10 lines

Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150529/e6e67e39/attachment.sig>

Hans de Goede

2015-05-30 09:23 UTC #18 lore raw

cover ARM: sunxi: Support PSCI for sun6i and sun8i

In reply to Chen-Yu Tsai

Hi,

Quoted 8 lines · Chen-Yu Tsai

Hi everyone,
I finally got around to rebasing and fixing up my PSCI stuff. This series adds PSCI support for sun6i and sun8i. This is based on sun7i PSCI support by Marc, and the SMP support routines found in both mainline Linux and Allwinner's Android kernel.

Quoted 4 lines

I've tested this on my Hummingbird A31 board and Ippo q8h v5 tablet. Given the A23 SMP routines work on the A33 as well, I see no reason it would not work, so A33 support is included as well, but more testing is always welcome.

Thanks for your work on this! I've merged this into u-boot-sunxi/next, and tested it on sun6i, sun7i, sun8i-a23 and sun8i-a33 and it works fine everywhere.

Quoted 2 lines

Patch 1 and 2 address a build failur on sunxi/next I ran into, which is related to the new NAND SPL support.

Thanks I've squashed these into the patch causing the build failures in the first place.

Quoted 9 lines

Patch 3 adds some comments to the sunxi PSCI code, for easier reading.
Patch 4 moves sun7i PSCI code aside.
Patch 5 adds sun6i PSCI code. Differences from sun7i are the separate PRCM block for power gating/clamps, and support for more than 1 secondary CPU core.

Quoted 6 lines

Patch 6 enables PSCI for sun6i.
Patch 7 lets sun8i share PSCI code with sun6i. The routines are the same, except sun8i does not have power clamps.
Patch 8 enables PSCI for sun8i (A23/A33).

And I've done a normal merge of the rest, and after testing pushed u-boot-sunxi/next to u-boot-sunxi/master and send a pull-req to Tom to get this included into u-boot 2015.07

Regards,

Hans