[PATCH 00/06] ARM: shmobile: SMP virt_to_phys() conversion

[PATCH 00/06] ARM: shmobile: SMP virt_to_phys() conversion

[Magnus Damm]

ARM: shmobile: SMP virt_to_phys() conversion

[PATCH 01/06] ARM: shmobile: Add SMP boot function and argument
[PATCH 02/06] ARM: shmobile: Add SCU boot function using argument
[PATCH 03/06] ARM: shmobile: r8a7779 SMP with SCU boot fn and args
[PATCH 04/06] ARM: shmobile: sh73a0 SMP with SCU boot fn and args
[PATCH 05/06] ARM: shmobile: EMEV2 SMP with SCU boot fn and args
[PATCH 06/06] ARM: shmobile: Remove old SCU boot code

This series replaces the inline virtual to physical address conversion code
in the mach-shmobile assembly files with C code making use of virt_to_phys().

This is one of several series of code that rework code not to rely on
CONFIG_MEMORY_START/SIZE which in turn is needed for ARCH_MULTIPLATFORM.

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 Written against renesas.git renesas-next-20130607

 arch/arm/mach-shmobile/headsmp-scu.S         |   35 ++++++++++----------------
 arch/arm/mach-shmobile/headsmp.S             |   13 +++++++--
 arch/arm/mach-shmobile/include/mach/common.h |    6 ++--
 arch/arm/mach-shmobile/smp-emev2.c           |    6 ++--
 arch/arm/mach-shmobile/smp-r8a7779.c         |    6 ++--
 arch/arm/mach-shmobile/smp-sh73a0.c          |    6 ++--
 6 files changed, 40 insertions(+), 32 deletions(-)

[PATCH 01/06] ARM: shmobile: Add SMP boot function and argument

[Magnus Damm]

From: Magnus Damm <damm@opensource.se>

Add code for mach-shmobile to allow specifying boot function
and argument. Will initially be used for SMP together with SCU
but may in the future also be used for deep sleep resume. This
patch removes one inline virtual to physical address conversion.

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 arch/arm/mach-shmobile/headsmp.S             |   13 ++++++++++---
 arch/arm/mach-shmobile/include/mach/common.h |    4 +++-
 2 files changed, 13 insertions(+), 4 deletions(-)

--- 0001/arch/arm/mach-shmobile/headsmp.S
+++ work/arch/arm/mach-shmobile/headsmp.S	2013-06-05 13:21:14.000000000 +0900
@@ -27,7 +27,14 @@ ENDPROC(shmobile_invalidate_start)
  * We need _long_ jump to the physical address.
  */
 	.align  12
-ENTRY(shmobile_secondary_vector)
+ENTRY(shmobile_boot_vector)
+	ldr     r0, 2f
 	ldr     pc, 1f
-1:	.long   shmobile_invalidate_start - PAGE_OFFSET + PLAT_PHYS_OFFSET
-ENDPROC(shmobile_secondary_vector)
+ENDPROC(shmobile_boot_vector)
+
+	.globl	shmobile_boot_fn
+shmobile_boot_fn:
+1:	.space	4
+	.globl	shmobile_boot_arg
+shmobile_boot_arg:
+2:	.space	4
--- 0016/arch/arm/mach-shmobile/include/mach/common.h
+++ work/arch/arm/mach-shmobile/include/mach/common.h	2013-06-05 13:19:52.000000000 +0900
@@ -7,7 +7,9 @@ extern void shmobile_setup_delay(unsigne
 			 unsigned int mult, unsigned int div);
 struct twd_local_timer;
 extern void shmobile_setup_console(void);
-extern void shmobile_secondary_vector(void);
+extern void shmobile_boot_vector(void);
+extern unsigned long shmobile_boot_fn;
+extern unsigned long shmobile_boot_arg;
 extern void shmobile_secondary_vector_scu(void);
 struct clk;
 extern int shmobile_clk_init(void);

[PATCH 02/06] ARM: shmobile: Add SCU boot function using argument

[Magnus Damm]

From: Magnus Damm <damm@opensource.se>

Add a shmoible_boot_scu function that assumes that the base address
of the SCU is passed in r0. This code is free from inline virtual
to physical address conversion.

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 arch/arm/mach-shmobile/headsmp-scu.S         |   13 +++++++++++++
 arch/arm/mach-shmobile/include/mach/common.h |    1 +
 2 files changed, 14 insertions(+)

--- 0001/arch/arm/mach-shmobile/headsmp-scu.S
+++ work/arch/arm/mach-shmobile/headsmp-scu.S	2013-06-10 17:44:41.000000000 +0900
@@ -51,6 +51,19 @@ ENTRY(shmobile_secondary_vector_scu)
 2:	.long shmobile_scu_base - PAGE_OFFSET + PLAT_PHYS_OFFSET
 ENDPROC(shmobile_secondary_vector_scu)
 
+ENTRY(shmobile_boot_scu)
+					@ r0 = SCU base address
+	mrc     p15, 0, r1, c0, c0, 5	@ read MIPDR
+	and	r1, r1, #3		@ mask out cpu ID
+	lsl	r1, r1, #3		@ we will shift by cpu_id * 8 bits
+	ldr	r2, [r0, #8]		@ SCU Power Status Register
+	mov	r3, #3
+	bic	r2, r2, r3, lsl r1	@ Clear bits of our CPU (Run Mode)
+	str	r2, [r0, #8]		@ write back
+
+	b	shmobile_invalidate_start
+ENDPROC(shmobile_boot_scu)
+
 	.text
 	.globl	shmobile_scu_base
 shmobile_scu_base:
--- 0004/arch/arm/mach-shmobile/include/mach/common.h
+++ work/arch/arm/mach-shmobile/include/mach/common.h	2013-06-10 16:32:25.000000000 +0900
@@ -11,6 +11,7 @@ extern void shmobile_boot_vector(void);
 extern unsigned long shmobile_boot_fn;
 extern unsigned long shmobile_boot_arg;
 extern void shmobile_secondary_vector_scu(void);
+extern void shmobile_boot_scu(void);
 struct clk;
 extern int shmobile_clk_init(void);
 extern void shmobile_handle_irq_intc(struct pt_regs *);

[PATCH 03/06] ARM: shmobile: r8a7779 SMP with SCU boot fn and args

[Magnus Damm]

From: Magnus Damm <damm@opensource.se>

Let r8a7779 make use of shmobile_boot_fn and shmobile_boot_arg
together with shmobile_boot_scu and the SCU base address.

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 arch/arm/mach-shmobile/smp-r8a7779.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

--- 0001/arch/arm/mach-shmobile/smp-r8a7779.c
+++ work/arch/arm/mach-shmobile/smp-r8a7779.c	2013-06-05 14:58:21.000000000 +0900
@@ -101,8 +101,10 @@ static void __init r8a7779_smp_prepare_c
 {
 	scu_enable(shmobile_scu_base);
 
-	/* Map the reset vector (in headsmp-scu.S) */
-	__raw_writel(__pa(shmobile_secondary_vector_scu), AVECR);
+	/* Map the reset vector (in headsmp-scu.S, headsmp.S) */
+	__raw_writel(__pa(shmobile_boot_vector), AVECR);
+	shmobile_boot_fn = virt_to_phys(shmobile_boot_scu);
+	shmobile_boot_arg = (unsigned long)shmobile_scu_base;
 
 	/* enable cache coherency on booting CPU */
 	scu_power_mode(shmobile_scu_base, SCU_PM_NORMAL);

[PATCH 04/06] ARM: shmobile: sh73a0 SMP with SCU boot fn and args

[Magnus Damm]

From: Magnus Damm <damm@opensource.se>

Let sh73a0 make use of shmobile_boot_fn and shmobile_boot_arg
together with shmobile_boot_scu and the SCU base address.

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 arch/arm/mach-shmobile/smp-sh73a0.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

--- 0001/arch/arm/mach-shmobile/smp-sh73a0.c
+++ work/arch/arm/mach-shmobile/smp-sh73a0.c	2013-06-05 14:59:24.000000000 +0900
@@ -64,9 +64,11 @@ static void __init sh73a0_smp_prepare_cp
 {
 	scu_enable(shmobile_scu_base);
 
-	/* Map the reset vector (in headsmp-scu.S) */
+	/* Map the reset vector (in headsmp-scu.S, headsmp.S) */
 	__raw_writel(0, APARMBAREA);      /* 4k */
-	__raw_writel(__pa(shmobile_secondary_vector_scu), SBAR);
+	__raw_writel(__pa(shmobile_boot_vector), SBAR);
+	shmobile_boot_fn = virt_to_phys(shmobile_boot_scu);
+	shmobile_boot_arg = (unsigned long)shmobile_scu_base;
 
 	/* enable cache coherency on booting CPU */
 	scu_power_mode(shmobile_scu_base, SCU_PM_NORMAL);

[PATCH 05/06] ARM: shmobile: EMEV2 SMP with SCU boot fn and args

[Magnus Damm]

From: Magnus Damm <damm@opensource.se>

Let EMEV2 make use of shmobile_boot_fn and shmobile_boot_arg
together with shmobile_boot_scu and the SCU base address.

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 arch/arm/mach-shmobile/smp-emev2.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

--- 0001/arch/arm/mach-shmobile/smp-emev2.c
+++ work/arch/arm/mach-shmobile/smp-emev2.c	2013-06-05 15:00:28.000000000 +0900
@@ -40,8 +40,10 @@ static void __init emev2_smp_prepare_cpu
 {
 	scu_enable(shmobile_scu_base);
 
-	/* Tell ROM loader about our vector (in headsmp-scu.S) */
-	emev2_set_boot_vector(__pa(shmobile_secondary_vector_scu));
+	/* Tell ROM loader about our vector (in headsmp-scu.S, headsmp.S) */
+	emev2_set_boot_vector(__pa(shmobile_boot_vector));
+	shmobile_boot_fn = virt_to_phys(shmobile_boot_scu);
+	shmobile_boot_arg = (unsigned long)shmobile_scu_base;
 
 	/* enable cache coherency on booting CPU */
 	scu_power_mode(shmobile_scu_base, SCU_PM_NORMAL);

[PATCH 06/06] ARM: shmobile: Remove old SCU boot code

[Magnus Damm]

From: Magnus Damm <damm@opensource.se>

Remove shmobile_secondary_vector_scu now when all SCU enabled
SMP platforms instead make use of shmobile_boot_scu. This
removes two inline virtual to physical address conversions.

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 arch/arm/mach-shmobile/headsmp-scu.S         |   22 +---------------------
 arch/arm/mach-shmobile/include/mach/common.h |    1 -
 2 files changed, 1 insertion(+), 22 deletions(-)

--- 0028/arch/arm/mach-shmobile/headsmp-scu.S
+++ work/arch/arm/mach-shmobile/headsmp-scu.S	2013-06-05 14:36:41.000000000 +0900
@@ -25,32 +25,12 @@
 
 	__CPUINIT
 /*
- * Reset vector for secondary CPUs.
+ * Boot code for secondary CPUs.
  *
  * First we turn on L1 cache coherency for our CPU. Then we jump to
  * shmobile_invalidate_start that invalidates the cache and hands over control
  * to the common ARM startup code.
- * This function will be mapped to address 0 by the SBAR register.
- * A normal branch is out of range here so we need a long jump. We jump to
- * the physical address as the MMU is still turned off.
  */
-	.align	12
-ENTRY(shmobile_secondary_vector_scu)
-	mrc     p15, 0, r0, c0, c0, 5	@ read MIPDR
-	and	r0, r0, #3		@ mask out cpu ID
-	lsl	r0, r0, #3		@ we will shift by cpu_id * 8 bits
-	ldr	r1, 2f
-	ldr	r1, [r1]		@ SCU base address
-	ldr	r2, [r1, #8]		@ SCU Power Status Register
-	mov	r3, #3
-	bic	r2, r2, r3, lsl r0	@ Clear bits of our CPU (Run Mode)
-	str	r2, [r1, #8]		@ write back
-
-	ldr	pc, 1f
-1:	.long shmobile_invalidate_start - PAGE_OFFSET + PLAT_PHYS_OFFSET
-2:	.long shmobile_scu_base - PAGE_OFFSET + PLAT_PHYS_OFFSET
-ENDPROC(shmobile_secondary_vector_scu)
-
 ENTRY(shmobile_boot_scu)
 					@ r0 = SCU base address
 	mrc     p15, 0, r1, c0, c0, 5	@ read MIPDR
--- 0028/arch/arm/mach-shmobile/include/mach/common.h
+++ work/arch/arm/mach-shmobile/include/mach/common.h	2013-06-05 14:35:54.000000000 +0900
@@ -10,7 +10,6 @@ extern void shmobile_setup_console(void)
 extern void shmobile_boot_vector(void);
 extern unsigned long shmobile_boot_fn;
 extern unsigned long shmobile_boot_arg;
-extern void shmobile_secondary_vector_scu(void);
 extern void shmobile_boot_scu(void);
 struct clk;
 extern int shmobile_clk_init(void);

[PATCH 01/06] ARM: shmobile: Add SMP boot function and argument

[Simon Horman]

On Mon, Jun 10, 2013 at 06:19:36PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
> 
> Add code for mach-shmobile to allow specifying boot function
> and argument. Will initially be used for SMP together with SCU
> but may in the future also be used for deep sleep resume. This
> patch removes one inline virtual to physical address conversion.
> 
> Signed-off-by: Magnus Damm <damm@opensource.se>
> ---
> 
>  arch/arm/mach-shmobile/headsmp.S             |   13 ++++++++++---
>  arch/arm/mach-shmobile/include/mach/common.h |    4 +++-
>  2 files changed, 13 insertions(+), 4 deletions(-)

Thanks, I have queued this up for v3.11 in the (new) cleanup-boot branch.

[PATCH 02/06] ARM: shmobile: Add SCU boot function using argument

[Simon Horman]

On Mon, Jun 10, 2013 at 06:19:46PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
> 
> Add a shmoible_boot_scu function that assumes that the base address
> of the SCU is passed in r0. This code is free from inline virtual
> to physical address conversion.
> 
> Signed-off-by: Magnus Damm <damm@opensource.se>
> ---
> 
>  arch/arm/mach-shmobile/headsmp-scu.S         |   13 +++++++++++++
>  arch/arm/mach-shmobile/include/mach/common.h |    1 +
>  2 files changed, 14 insertions(+)

Thanks, I have queued this up for v3.11 in the (new) cleanup-boot branch.

[PATCH 03/06] ARM: shmobile: r8a7779 SMP with SCU boot fn and args

[Simon Horman]

On Mon, Jun 10, 2013 at 06:19:56PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
> 
> Let r8a7779 make use of shmobile_boot_fn and shmobile_boot_arg
> together with shmobile_boot_scu and the SCU base address.

Thanks, I have queued this up for v3.11 in the (new) cleanup-boot branch.

[PATCH 04/06] ARM: shmobile: sh73a0 SMP with SCU boot fn and args

[Simon Horman]

On Mon, Jun 10, 2013 at 06:20:06PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
> 
> Let sh73a0 make use of shmobile_boot_fn and shmobile_boot_arg
> together with shmobile_boot_scu and the SCU base address.
> 
> Signed-off-by: Magnus Damm <damm@opensource.se>
> ---
> 
>  arch/arm/mach-shmobile/smp-sh73a0.c |    6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)

Thanks, I have queued this up for v3.11 in the (new) cleanup-boot branch.

[PATCH 05/06] ARM: shmobile: EMEV2 SMP with SCU boot fn and args

[Simon Horman]

On Mon, Jun 10, 2013 at 06:20:15PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
> 
> Let EMEV2 make use of shmobile_boot_fn and shmobile_boot_arg
> together with shmobile_boot_scu and the SCU base address.
> 
> Signed-off-by: Magnus Damm <damm@opensource.se>
> ---
> 
>  arch/arm/mach-shmobile/smp-emev2.c |    6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)

Thanks, I have queued this up for v3.11 in the (new) cleanup-boot branch.

[PATCH 06/06] ARM: shmobile: Remove old SCU boot code

[Simon Horman]

On Mon, Jun 10, 2013 at 06:20:25PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
> 
> Remove shmobile_secondary_vector_scu now when all SCU enabled
> SMP platforms instead make use of shmobile_boot_scu. This
> removes two inline virtual to physical address conversions.
> 
> Signed-off-by: Magnus Damm <damm@opensource.se>
> ---
> 
>  arch/arm/mach-shmobile/headsmp-scu.S         |   22 +---------------------
>  arch/arm/mach-shmobile/include/mach/common.h |    1 -
>  2 files changed, 1 insertion(+), 22 deletions(-)

Thanks, I have queued this up for v3.11 in the (new) cleanup-boot branch.