[PATCH 00/04] ARM: shmobile: Use shmobile_init_delay() on r8a7740

[PATCH 00/04] ARM: shmobile: Use shmobile_init_delay() on r8a7740

[Magnus Damm]

ARM: shmobile: Use shmobile_init_delay() on r8a7740

[PATCH 01/04] ARM: shmobile: Add r8a7740 Maximum CPU Frequency to DTS
[PATCH 02/04] ARM: shmobile: Use r8a7740 DT CPU Frequency in common case	
[PATCH 03/04] ARM: shmobile: Use r8a7740 DT CPU Frequency for Armadillo DT Ref
[PATCH 04/04] ARM: shmobile: Remove unused r8a7740_init_delay()

Convert r8a7740 to rely on shmobile_init_delay() instead of using a per-SoC
delay setup function. This is a cleanup towards enabling Multiplatform support
for the r8a7740 SoC.

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

 Written against renesas-devel-v3.15-rc3-20140502

 arch/arm/boot/dts/r8a7740.dtsi                           |    1 +
 arch/arm/mach-shmobile/board-armadillo800eva-reference.c |    2 +-
 arch/arm/mach-shmobile/include/mach/r8a7740.h            |    1 -
 arch/arm/mach-shmobile/setup-r8a7740.c                   |    7 +------
 4 files changed, 3 insertions(+), 8 deletions(-)

[PATCH 01/04] ARM: shmobile: Add r8a7740 Maximum CPU Frequency to DTS

[Magnus Damm]

From: Magnus Damm <damm+renesas@opensource.se>

Add 800 MHz to the r8a7740 DTS to describe the maximum CPU frequency.

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

 arch/arm/boot/dts/r8a7740.dtsi |    1 +
 1 file changed, 1 insertion(+)

--- 0001/arch/arm/boot/dts/r8a7740.dtsi
+++ work/arch/arm/boot/dts/r8a7740.dtsi	2014-04-24 17:50:40.000000000 +0900
@@ -22,6 +22,7 @@
 			compatible = "arm,cortex-a9";
 			device_type = "cpu";
 			reg = <0x0>;
+			clock-frequency = <800000000>;
 		};
 	};
 

[PATCH 02/04] ARM: shmobile: Use r8a7740 DT CPU Frequency in common case

[Magnus Damm]

From: Magnus Damm <damm+renesas@opensource.se>

Convert the common C-code-less r8a7740 DT board support
to use shmobile_init_delay() to be able to migrate away
from per-SoC delay setup functions.

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

 arch/arm/mach-shmobile/setup-r8a7740.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- 0001/arch/arm/mach-shmobile/setup-r8a7740.c
+++ work/arch/arm/mach-shmobile/setup-r8a7740.c	2014-04-24 17:53:29.000000000 +0900
@@ -924,7 +924,7 @@ static const char *r8a7740_boards_compat
 
 DT_MACHINE_START(R8A7740_DT, "Generic R8A7740 (Flattened Device Tree)")
 	.map_io		= r8a7740_map_io,
-	.init_early	= r8a7740_init_delay,
+	.init_early	= shmobile_init_delay,
 	.init_irq	= r8a7740_init_irq_of,
 	.init_machine	= r8a7740_generic_init,
 	.dt_compat	= r8a7740_boards_compat_dt,

[PATCH 03/04] ARM: shmobile: Use r8a7740 DT CPU Frequency for Armadillo DT Ref

[Magnus Damm]

From: Magnus Damm <damm+renesas@opensource.se>

Convert the Armadillo r8a7740 DT reference board support
to use shmobile_init_delay() to be able to migrate away
from per-SoC delay setup functions.

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

 arch/arm/mach-shmobile/board-armadillo800eva-reference.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- 0001/arch/arm/mach-shmobile/board-armadillo800eva-reference.c
+++ work/arch/arm/mach-shmobile/board-armadillo800eva-reference.c	2014-04-24 17:59:55.000000000 +0900
@@ -187,7 +187,7 @@ static const char *eva_boards_compat_dt[
 
 DT_MACHINE_START(ARMADILLO800EVA_DT, "armadillo800eva-reference")
 	.map_io		= r8a7740_map_io,
-	.init_early	= r8a7740_init_delay,
+	.init_early	= shmobile_init_delay,
 	.init_irq	= r8a7740_init_irq_of,
 	.init_machine	= eva_init,
 	.init_late	= shmobile_init_late,

[PATCH 04/04] ARM: shmobile: Remove unused r8a7740_init_delay()

[Magnus Damm]

From: Magnus Damm <damm+renesas@opensource.se>

Remove the now unused r8a7740_init_delay() function.

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

 arch/arm/mach-shmobile/include/mach/r8a7740.h |    1 -
 arch/arm/mach-shmobile/setup-r8a7740.c        |    5 -----
 2 files changed, 6 deletions(-)

--- 0001/arch/arm/mach-shmobile/include/mach/r8a7740.h
+++ work/arch/arm/mach-shmobile/include/mach/r8a7740.h	2014-04-24 18:02:10.000000000 +0900
@@ -47,7 +47,6 @@ enum {
 };
 
 extern void r8a7740_meram_workaround(void);
-extern void r8a7740_init_delay(void);
 extern void r8a7740_init_irq_of(void);
 extern void r8a7740_map_io(void);
 extern void r8a7740_add_early_devices(void);
--- 0004/arch/arm/mach-shmobile/setup-r8a7740.c
+++ work/arch/arm/mach-shmobile/setup-r8a7740.c	2014-04-24 18:01:54.000000000 +0900
@@ -876,11 +876,6 @@ void __init r8a7740_add_standard_devices
 	of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
 }
 
-void __init r8a7740_init_delay(void)
-{
-	shmobile_setup_delay(800, 1, 3); /* Cortex-A9 @ 800MHz */
-};
-
 void __init r8a7740_init_irq_of(void)
 {
 	void __iomem *intc_prio_base = ioremap_nocache(0xe6900010, 0x10);

Re: [PATCH 00/04] ARM: shmobile: Use shmobile_init_delay() on r8a7740

[Geert Uytterhoeven]

Hi Magnus,

On Thu, May 8, 2014 at 1:32 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
> ARM: shmobile: Use shmobile_init_delay() on r8a7740
>
> [PATCH 01/04] ARM: shmobile: Add r8a7740 Maximum CPU Frequency to DTS
> [PATCH 02/04] ARM: shmobile: Use r8a7740 DT CPU Frequency in common case
> [PATCH 03/04] ARM: shmobile: Use r8a7740 DT CPU Frequency for Armadillo DT Ref
> [PATCH 04/04] ARM: shmobile: Remove unused r8a7740_init_delay()
>
> Convert r8a7740 to rely on shmobile_init_delay() instead of using a per-SoC
> delay setup function. This is a cleanup towards enabling Multiplatform support
> for the r8a7740 SoC.

Thanks!

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

Tested-by: Geert Uytterhoeven <geert@glider.be>

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: [PATCH 00/04] ARM: shmobile: Use shmobile_init_delay() on r8a7740

[Simon Horman]

On Fri, May 09, 2014 at 09:54:58AM +0200, Geert Uytterhoeven wrote:
> Hi Magnus,
> 
> On Thu, May 8, 2014 at 1:32 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
> > ARM: shmobile: Use shmobile_init_delay() on r8a7740
> >
> > [PATCH 01/04] ARM: shmobile: Add r8a7740 Maximum CPU Frequency to DTS
> > [PATCH 02/04] ARM: shmobile: Use r8a7740 DT CPU Frequency in common case
> > [PATCH 03/04] ARM: shmobile: Use r8a7740 DT CPU Frequency for Armadillo DT Ref
> > [PATCH 04/04] ARM: shmobile: Remove unused r8a7740_init_delay()
> >
> > Convert r8a7740 to rely on shmobile_init_delay() instead of using a per-SoC
> > delay setup function. This is a cleanup towards enabling Multiplatform support
> > for the r8a7740 SoC.
> 
> Thanks!
> 
> > Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> 
> Tested-by: Geert Uytterhoeven <geert@glider.be>

Thanks, I have queued these up.