linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 04/13] ARM: dts: r8a7743: Add missing clock for secondary CA15 CPU core
       [not found] <1507800916-11509-1-git-send-email-geert+renesas@glider.be>
@ 2017-10-12  9:35 ` Geert Uytterhoeven
  2017-10-13  6:23   ` Chris Paterson
  2017-10-12  9:35 ` [PATCH 12/13] ARM: dts: r8a7794: Add missing clock for secondary CA7 " Geert Uytterhoeven
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 6+ messages in thread
From: Geert Uytterhoeven @ 2017-10-12  9:35 UTC (permalink / raw)
  To: linux-arm-kernel

Currently only the primary CPU in the CA15 cluster has a clocks
property, while the secondary CPU core is driven by the same clock.
Add the missing clocks property to fix this.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/boot/dts/r8a7743.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/r8a7743.dtsi b/arch/arm/boot/dts/r8a7743.dtsi
index d541fd9ffafb34ce..e09517bce30db2e8 100644
--- a/arch/arm/boot/dts/r8a7743.dtsi
+++ b/arch/arm/boot/dts/r8a7743.dtsi
@@ -63,6 +63,7 @@
 			compatible = "arm,cortex-a15";
 			reg = <1>;
 			clock-frequency = <1500000000>;
+			clocks = <&cpg CPG_CORE R8A7743_CLK_Z>;
 			power-domains = <&sysc R8A7743_PD_CA15_CPU1>;
 			next-level-cache = <&L2_CA15>;
 		};
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 12/13] ARM: dts: r8a7794: Add missing clock for secondary CA7 CPU core
       [not found] <1507800916-11509-1-git-send-email-geert+renesas@glider.be>
  2017-10-12  9:35 ` [PATCH 04/13] ARM: dts: r8a7743: Add missing clock for secondary CA15 CPU core Geert Uytterhoeven
@ 2017-10-12  9:35 ` Geert Uytterhoeven
       [not found] ` <1507800916-11509-8-git-send-email-geert+renesas@glider.be>
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Geert Uytterhoeven @ 2017-10-12  9:35 UTC (permalink / raw)
  To: linux-arm-kernel

Currently only the primary CPU in the CA7 cluster has a clocks property,
while the secondary CPU core is driven by the same clock.
Add the missing clocks property to fix this.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/boot/dts/r8a7794.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/r8a7794.dtsi b/arch/arm/boot/dts/r8a7794.dtsi
index 4fd92e989e227270..b588029ae9db8957 100644
--- a/arch/arm/boot/dts/r8a7794.dtsi
+++ b/arch/arm/boot/dts/r8a7794.dtsi
@@ -54,6 +54,7 @@
 			compatible = "arm,cortex-a7";
 			reg = <1>;
 			clock-frequency = <1000000000>;
+			clocks = <&cpg CPG_CORE R8A7794_CLK_Z2>;
 			power-domains = <&sysc R8A7794_PD_CA7_CPU1>;
 			next-level-cache = <&L2_CA7>;
 		};
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 04/13] ARM: dts: r8a7743: Add missing clock for secondary CA15 CPU core
  2017-10-12  9:35 ` [PATCH 04/13] ARM: dts: r8a7743: Add missing clock for secondary CA15 CPU core Geert Uytterhoeven
@ 2017-10-13  6:23   ` Chris Paterson
  0 siblings, 0 replies; 6+ messages in thread
From: Chris Paterson @ 2017-10-13  6:23 UTC (permalink / raw)
  To: linux-arm-kernel

Hello Geert,

> From: linux-renesas-soc-owner at vger.kernel.org [mailto:linux-renesas-soc-
> owner at vger.kernel.org] On Behalf Of Geert Uytterhoeven
> Sent: 12 October 2017 10:35
> 
> Currently only the primary CPU in the CA15 cluster has a clocks property,
> while the secondary CPU core is driven by the same clock.
> Add the missing clocks property to fix this.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Looks okay to me, thanks.

Reviewed-by: Chris Paterson <chris.paterson2@renesas.com

> ---
>  arch/arm/boot/dts/r8a7743.dtsi | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/boot/dts/r8a7743.dtsi
> b/arch/arm/boot/dts/r8a7743.dtsi index
> d541fd9ffafb34ce..e09517bce30db2e8 100644
> --- a/arch/arm/boot/dts/r8a7743.dtsi
> +++ b/arch/arm/boot/dts/r8a7743.dtsi
> @@ -63,6 +63,7 @@
>  			compatible = "arm,cortex-a15";
>  			reg = <1>;
>  			clock-frequency = <1500000000>;
> +			clocks = <&cpg CPG_CORE R8A7743_CLK_Z>;
>  			power-domains = <&sysc R8A7743_PD_CA15_CPU1>;
>  			next-level-cache = <&L2_CA15>;
>  		};
> --
> 2.7.4

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 07/13] ARM: dts: r8a7790: Add missing clocks for secondary CA15 CPU cores
       [not found] ` <1507800916-11509-8-git-send-email-geert+renesas@glider.be>
@ 2017-10-16  9:42   ` Simon Horman
  0 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2017-10-16  9:42 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Oct 12, 2017 at 11:35:10AM +0200, Geert Uytterhoeven wrote:
> Currently only the primary CPU in the CA15 cluster has a clocks
> property, while the secondary CPU cores are driven by the same clock.
> Add the missing clocks properties to fix this.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Tested-by: Simon Horman <horms+renesas@verge.net.au>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 08/13] ARM: dts: r8a7790: Add clocks for CA7 CPU cores
       [not found] ` <1507800916-11509-9-git-send-email-geert+renesas@glider.be>
@ 2017-10-16  9:42   ` Simon Horman
  0 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2017-10-16  9:42 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Oct 12, 2017 at 11:35:11AM +0200, Geert Uytterhoeven wrote:
> Currently only the CPU cores in the CA15 cluster have clocks properties.
> Add the missing clocks properties for the CPU cores in the CA7 cluster
> to fix this.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Tested-by: Simon Horman <horms+renesas@verge.net.au>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 00/13] ARM: dts: renesas: Add missing clocks for ARM CPU cores
       [not found] <1507800916-11509-1-git-send-email-geert+renesas@glider.be>
                   ` (3 preceding siblings ...)
       [not found] ` <1507800916-11509-9-git-send-email-geert+renesas@glider.be>
@ 2017-10-16  9:53 ` Simon Horman
  4 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2017-10-16  9:53 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Oct 12, 2017 at 11:35:03AM +0200, Geert Uytterhoeven wrote:
> 	Hi Simon, Magnus,
> 
> This series improves DT hardware descriptions for Renesas arm32 SoCs by
> adding missing clocks properties to the device nodes corresponding to
> ARM CPU cores.
> 
> Notes:
>   - This series should have no functional impact, except perhaps on
>     big.LITTLE SoCs, when using a firmware that enables both big and
>     little CPU cores (see also Sudeep Holla's comments for R-Car Gen3 in
>     "Re: [PATCH v2 2/2] arm64: dts: r8a7796: Add OPPs table for cpu
>     devices" (https://www.spinics.net/lists/arm-kernel/msg610489.html),
>   - The clocks for CA9 CPU cores on R-Mobile A1 and EMMA Mobile EV2 are
>     still missing, as these clocks are not supported by the DT bindings
>     and drivers, and cannot be referenced yet.
> 
> Tested on r8a73a4/ape6evm, r8a7791/koelsch, and sh73a0/kzm9g.
> 
> Testing on r8a7790/lager would be appreciated.
> 
> Thanks for your comments!
> 
> Geert Uytterhoeven (13):
>   dt-bindings: clk: r7s72100: Add missing I and G clocks
>   ARM: dts: r7s72100: Add clock for CA9 CPU core
>   ARM: dts: r8a73a4: Add clock for CA15 CPU0 core
>   ARM: dts: r8a7743: Add missing clock for secondary CA15 CPU core
>   ARM: dts: r8a7778: Add clock for CA9 CPU core
>   ARM: dts: r8a7779: Add clocks for CA9 CPU cores
>   ARM: dts: r8a7790: Add missing clocks for secondary CA15 CPU cores
>   ARM: dts: r8a7790: Add clocks for CA7 CPU cores
>   ARM: dts: r8a7791: Add missing clock for secondary CA15 CPU core
>   ARM: dts: r8a7792: Add missing clock for secondary CA15 CPU core
>   ARM: dts: r8a7793: Add missing clock for secondary CA15 CPU core
>   ARM: dts: r8a7794: Add missing clock for secondary CA7 CPU core
>   ARM: dts: sh73a0: Add clocks for CA9 CPU cores

Thanks,

I have applied the above for v4.15. There was some fuzz when applying the
r8a7790 patches so please check that what I have is correct after I push
the next branch of the renesas tree a little later day.

> 
>  arch/arm/boot/dts/r7s72100.dtsi            | 1 +
>  arch/arm/boot/dts/r8a73a4.dtsi             | 1 +
>  arch/arm/boot/dts/r8a7743.dtsi             | 1 +
>  arch/arm/boot/dts/r8a7778.dtsi             | 1 +
>  arch/arm/boot/dts/r8a7779.dtsi             | 4 ++++
>  arch/arm/boot/dts/r8a7790.dtsi             | 7 +++++++
>  arch/arm/boot/dts/r8a7791.dtsi             | 1 +
>  arch/arm/boot/dts/r8a7792.dtsi             | 1 +
>  arch/arm/boot/dts/r8a7793.dtsi             | 1 +
>  arch/arm/boot/dts/r8a7794.dtsi             | 1 +
>  arch/arm/boot/dts/sh73a0.dtsi              | 2 ++
>  include/dt-bindings/clock/r7s72100-clock.h | 2 ++
>  12 files changed, 23 insertions(+)
> 
> -- 
> 2.7.4
> 
> Gr{oetje,eeting}s,
> 
> 						Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-10-16  9:53 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1507800916-11509-1-git-send-email-geert+renesas@glider.be>
2017-10-12  9:35 ` [PATCH 04/13] ARM: dts: r8a7743: Add missing clock for secondary CA15 CPU core Geert Uytterhoeven
2017-10-13  6:23   ` Chris Paterson
2017-10-12  9:35 ` [PATCH 12/13] ARM: dts: r8a7794: Add missing clock for secondary CA7 " Geert Uytterhoeven
     [not found] ` <1507800916-11509-8-git-send-email-geert+renesas@glider.be>
2017-10-16  9:42   ` [PATCH 07/13] ARM: dts: r8a7790: Add missing clocks for secondary CA15 CPU cores Simon Horman
     [not found] ` <1507800916-11509-9-git-send-email-geert+renesas@glider.be>
2017-10-16  9:42   ` [PATCH 08/13] ARM: dts: r8a7790: Add clocks for CA7 " Simon Horman
2017-10-16  9:53 ` [PATCH 00/13] ARM: dts: renesas: Add missing clocks for ARM " Simon Horman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).