All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: renesas: r8a77965: use r8a77965-sysc binding definitions
@ 2018-04-26 19:34 Niklas Söderlund
  2018-04-27  7:19 ` jacopo mondi
  2018-05-02  7:08 ` Simon Horman
  0 siblings, 2 replies; 3+ messages in thread
From: Niklas Söderlund @ 2018-04-26 19:34 UTC (permalink / raw)
  To: Simon Horman, Geert Uytterhoeven; +Cc: linux-renesas-soc, Niklas Söderlund

Replace the hardcoded power domain indices by R8A77965_PD_* symbols.

Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
---
 arch/arm64/boot/dts/renesas/r8a77965.dtsi | 95 ++++++++++++-----------
 1 file changed, 48 insertions(+), 47 deletions(-)

Hi Simon,

This is based on top of your trees devel branch from earlier today.

// Nikls

diff --git a/arch/arm64/boot/dts/renesas/r8a77965.dtsi b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
index b12f41755aeafea1..054ac6be43fac6f3 100644
--- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
@@ -10,6 +10,7 @@
 
 #include <dt-bindings/clock/renesas-cpg-mssr.h>
 #include <dt-bindings/interrupt-controller/arm-gic.h>
+#include <dt-bindings/power/r8a77965-sysc.h>
 
 #define CPG_AUDIO_CLK_I		10
 
@@ -60,7 +61,7 @@
 			compatible = "arm,cortex-a57", "arm,armv8";
 			reg = <0x0>;
 			device_type = "cpu";
-			power-domains = <&sysc 0>;
+			power-domains = <&sysc R8A77965_PD_CA57_CPU0>;
 			next-level-cache = <&L2_CA57>;
 			enable-method = "psci";
 		};
@@ -69,14 +70,14 @@
 			compatible = "arm,cortex-a57","arm,armv8";
 			reg = <0x1>;
 			device_type = "cpu";
-			power-domains = <&sysc 1>;
+			power-domains = <&sysc R8A77965_PD_CA57_CPU1>;
 			next-level-cache = <&L2_CA57>;
 			enable-method = "psci";
 		};
 
 		L2_CA57: cache-controller-0 {
 			compatible = "cache";
-			power-domains = <&sysc 12>;
+			power-domains = <&sysc R8A77965_PD_CA57_SCU>;
 			cache-unified;
 			cache-level = <2>;
 		};
@@ -146,7 +147,7 @@
 			#interrupt-cells = <2>;
 			interrupt-controller;
 			clocks = <&cpg CPG_MOD 912>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 912>;
 		};
 
@@ -161,7 +162,7 @@
 			#interrupt-cells = <2>;
 			interrupt-controller;
 			clocks = <&cpg CPG_MOD 911>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 911>;
 		};
 
@@ -176,7 +177,7 @@
 			#interrupt-cells = <2>;
 			interrupt-controller;
 			clocks = <&cpg CPG_MOD 910>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 910>;
 		};
 
@@ -191,7 +192,7 @@
 			#interrupt-cells = <2>;
 			interrupt-controller;
 			clocks = <&cpg CPG_MOD 909>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 909>;
 		};
 
@@ -206,7 +207,7 @@
 			#interrupt-cells = <2>;
 			interrupt-controller;
 			clocks = <&cpg CPG_MOD 908>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 908>;
 		};
 
@@ -221,7 +222,7 @@
 			#interrupt-cells = <2>;
 			interrupt-controller;
 			clocks = <&cpg CPG_MOD 907>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 907>;
 		};
 
@@ -236,7 +237,7 @@
 			#interrupt-cells = <2>;
 			interrupt-controller;
 			clocks = <&cpg CPG_MOD 906>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 906>;
 		};
 
@@ -251,7 +252,7 @@
 			#interrupt-cells = <2>;
 			interrupt-controller;
 			clocks = <&cpg CPG_MOD 905>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 905>;
 		};
 
@@ -293,7 +294,7 @@
 				      GIC_SPI 18 IRQ_TYPE_LEVEL_HIGH
 				      GIC_SPI 161 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&cpg CPG_MOD 407>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 407>;
 		};
 
@@ -347,7 +348,7 @@
 			reg = <0 0xe60b0000 0 0x425>;
 			interrupts = <GIC_SPI 173 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&cpg CPG_MOD 926>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 926>;
 			dmas = <&dmac0 0x11>, <&dmac0 0x10>;
 			dma-names = "tx", "rx";
@@ -366,7 +367,7 @@
 			renesas,buswait = <11>;
 			phys = <&usb2_phy0>;
 			phy-names = "usb";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 704>;
 			status = "disabled";
 		};
@@ -379,7 +380,7 @@
 				      GIC_SPI 109 IRQ_TYPE_LEVEL_HIGH>;
 			interrupt-names = "ch0", "ch1";
 			clocks = <&cpg CPG_MOD 330>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 330>;
 			#dma-cells = <1>;
 			dma-channels = <2>;
@@ -393,7 +394,7 @@
 				      GIC_SPI 110 IRQ_TYPE_LEVEL_HIGH>;
 			interrupt-names = "ch0", "ch1";
 			clocks = <&cpg CPG_MOD 331>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 331>;
 			#dma-cells = <1>;
 			dma-channels = <2>;
@@ -406,7 +407,7 @@
 			clocks = <&cpg CPG_MOD 328>, <&usb3s0_clk>,
 				 <&usb_extal_clk>;
 			clock-names = "usb3-if", "usb3s_clk", "usb_extal";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 328>;
 			#phy-cells = <0>;
 			status = "disabled";
@@ -440,7 +441,7 @@
 					"ch12", "ch13", "ch14", "ch15";
 			clocks = <&cpg CPG_MOD 219>;
 			clock-names = "fck";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 219>;
 			#dma-cells = <1>;
 			dma-channels = <16>;
@@ -474,7 +475,7 @@
 					"ch12", "ch13", "ch14", "ch15";
 			clocks = <&cpg CPG_MOD 218>;
 			clock-names = "fck";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 218>;
 			#dma-cells = <1>;
 			dma-channels = <16>;
@@ -508,7 +509,7 @@
 					"ch12", "ch13", "ch14", "ch15";
 			clocks = <&cpg CPG_MOD 217>;
 			clock-names = "fck";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 217>;
 			#dma-cells = <1>;
 			dma-channels = <16>;
@@ -551,7 +552,7 @@
 					  "ch20", "ch21", "ch22", "ch23",
 					  "ch24";
 			clocks = <&cpg CPG_MOD 812>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 812>;
 			phy-mode = "rgmii";
 			#address-cells = <1>;
@@ -565,7 +566,7 @@
 			#pwm-cells = <2>;
 			clocks = <&cpg CPG_MOD 523>;
 			resets = <&cpg 523>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			status = "disabled";
 		};
 
@@ -575,7 +576,7 @@
 			#pwm-cells = <2>;
 			clocks = <&cpg CPG_MOD 523>;
 			resets = <&cpg 523>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			status = "disabled";
 		};
 
@@ -585,7 +586,7 @@
 			#pwm-cells = <2>;
 			clocks = <&cpg CPG_MOD 523>;
 			resets = <&cpg 523>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			status = "disabled";
 		};
 
@@ -595,7 +596,7 @@
 			#pwm-cells = <2>;
 			clocks = <&cpg CPG_MOD 523>;
 			resets = <&cpg 523>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			status = "disabled";
 		};
 
@@ -605,7 +606,7 @@
 			#pwm-cells = <2>;
 			clocks = <&cpg CPG_MOD 523>;
 			resets = <&cpg 523>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			status = "disabled";
 		};
 
@@ -615,7 +616,7 @@
 			#pwm-cells = <2>;
 			clocks = <&cpg CPG_MOD 523>;
 			resets = <&cpg 523>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			status = "disabled";
 		};
 
@@ -625,7 +626,7 @@
 			#pwm-cells = <2>;
 			clocks = <&cpg CPG_MOD 523>;
 			resets = <&cpg 523>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			status = "disabled";
 		};
 
@@ -641,7 +642,7 @@
 			dmas = <&dmac1 0x51>, <&dmac1 0x50>,
 			       <&dmac2 0x51>, <&dmac2 0x50>;
 			dma-names = "tx", "rx", "tx", "rx";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 207>;
 			status = "disabled";
 		};
@@ -658,7 +659,7 @@
 			dmas = <&dmac1 0x53>, <&dmac1 0x52>,
 			       <&dmac2 0x53>, <&dmac2 0x52>;
 			dma-names = "tx", "rx", "tx", "rx";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 206>;
 			status = "disabled";
 		};
@@ -672,7 +673,7 @@
 				 <&cpg CPG_CORE 20>,
 				 <&scif_clk>;
 			clock-names = "fck", "brg_int", "scif_clk";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 310>;
 			status = "disabled";
 		};
@@ -688,7 +689,7 @@
 			clock-names = "fck", "brg_int", "scif_clk";
 			dmas = <&dmac0 0x57>, <&dmac0 0x56>;
 			dma-names = "tx", "rx";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 204>;
 			status = "disabled";
 		};
@@ -704,7 +705,7 @@
 			clock-names = "fck", "brg_int", "scif_clk";
 			dmas = <&dmac0 0x59>, <&dmac0 0x58>;
 			dma-names = "tx", "rx";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 203>;
 			status = "disabled";
 		};
@@ -721,7 +722,7 @@
 			dmas = <&dmac1 0x5b>, <&dmac1 0x5a>,
 			       <&dmac2 0x5b>, <&dmac2 0x5a>;
 			dma-names = "tx", "rx", "tx", "rx";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 202>;
 			status = "disabled";
 		};
@@ -735,7 +736,7 @@
 			dmas = <&dmac1 0x41>, <&dmac1 0x40>,
 			       <&dmac2 0x41>, <&dmac2 0x40>;
 			dma-names = "tx", "rx", "tx", "rx";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 211>;
 			#address-cells = <1>;
 			#size-cells = <0>;
@@ -751,7 +752,7 @@
 			dmas = <&dmac1 0x43>, <&dmac1 0x42>,
 			       <&dmac2 0x43>, <&dmac2 0x42>;
 			dma-names = "tx", "rx", "tx", "rx";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 210>;
 			#address-cells = <1>;
 			#size-cells = <0>;
@@ -766,7 +767,7 @@
 			clocks = <&cpg CPG_MOD 209>;
 			dmas = <&dmac0 0x45>, <&dmac0 0x44>;
 			dma-names = "tx", "rx";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 209>;
 			#address-cells = <1>;
 			#size-cells = <0>;
@@ -781,7 +782,7 @@
 			clocks = <&cpg CPG_MOD 208>;
 			dmas = <&dmac0 0x47>, <&dmac0 0x46>;
 			dma-names = "tx", "rx";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 208>;
 			#address-cells = <1>;
 			#size-cells = <0>;
@@ -864,7 +865,7 @@
 			reg = <0 0xee000000 0 0xc00>;
 			interrupts = <GIC_SPI 102 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&cpg CPG_MOD 328>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 328>;
 			status = "disabled";
 		};
@@ -875,7 +876,7 @@
 			reg = <0 0xee020000 0 0x400>;
 			interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&cpg CPG_MOD 328>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 328>;
 			status = "disabled";
 		};
@@ -887,7 +888,7 @@
 			clocks = <&cpg CPG_MOD 703>;
 			phys = <&usb2_phy0>;
 			phy-names = "usb";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 703>;
 			status = "disabled";
 		};
@@ -899,7 +900,7 @@
 			clocks = <&cpg CPG_MOD 702>;
 			phys = <&usb2_phy1>;
 			phy-names = "usb";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 702>;
 			status = "disabled";
 		};
@@ -912,7 +913,7 @@
 			phys = <&usb2_phy0>;
 			phy-names = "usb";
 			companion = <&ohci0>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 703>;
 			status = "disabled";
 		};
@@ -925,7 +926,7 @@
 			phys = <&usb2_phy1>;
 			phy-names = "usb";
 			companion = <&ohci1>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 702>;
 			status = "disabled";
 		};
@@ -936,7 +937,7 @@
 			reg = <0 0xee080200 0 0x700>;
 			interrupts = <GIC_SPI 108 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&cpg CPG_MOD 703>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 703>;
 			#phy-cells = <0>;
 			status = "disabled";
@@ -947,7 +948,7 @@
 				     "renesas,rcar-gen3-usb2-phy";
 			reg = <0 0xee0a0200 0 0x700>;
 			clocks = <&cpg CPG_MOD 703>;
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 703>;
 			#phy-cells = <0>;
 			status = "disabled";
@@ -986,7 +987,7 @@
 					(GIC_CPU_MASK_SIMPLE(2) | IRQ_TYPE_LEVEL_HIGH)>;
 			clocks = <&cpg CPG_MOD 408>;
 			clock-names = "clk";
-			power-domains = <&sysc 32>;
+			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
 			resets = <&cpg 408>;
 		};
 
-- 
2.17.0

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

* Re: [PATCH] arm64: dts: renesas: r8a77965: use r8a77965-sysc binding definitions
  2018-04-26 19:34 [PATCH] arm64: dts: renesas: r8a77965: use r8a77965-sysc binding definitions Niklas Söderlund
@ 2018-04-27  7:19 ` jacopo mondi
  2018-05-02  7:08 ` Simon Horman
  1 sibling, 0 replies; 3+ messages in thread
From: jacopo mondi @ 2018-04-27  7:19 UTC (permalink / raw)
  To: Niklas Söderlund; +Cc: Simon Horman, Geert Uytterhoeven, linux-renesas-soc

[-- Attachment #1: Type: text/plain, Size: 14170 bytes --]

Hi Niklas,

On Thu, Apr 26, 2018 at 09:34:47PM +0200, Niklas Söderlund wrote:
> Replace the hardcoded power domain indices by R8A77965_PD_* symbols.
>

Reviewed-by: Jacopo Mondi <jacopo@jmondi.org>

Thanks
   j

> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> ---
>  arch/arm64/boot/dts/renesas/r8a77965.dtsi | 95 ++++++++++++-----------
>  1 file changed, 48 insertions(+), 47 deletions(-)
>
> Hi Simon,
>
> This is based on top of your trees devel branch from earlier today.
>
> // Nikls
>
> diff --git a/arch/arm64/boot/dts/renesas/r8a77965.dtsi b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
> index b12f41755aeafea1..054ac6be43fac6f3 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi
> @@ -10,6 +10,7 @@
>
>  #include <dt-bindings/clock/renesas-cpg-mssr.h>
>  #include <dt-bindings/interrupt-controller/arm-gic.h>
> +#include <dt-bindings/power/r8a77965-sysc.h>
>
>  #define CPG_AUDIO_CLK_I		10
>
> @@ -60,7 +61,7 @@
>  			compatible = "arm,cortex-a57", "arm,armv8";
>  			reg = <0x0>;
>  			device_type = "cpu";
> -			power-domains = <&sysc 0>;
> +			power-domains = <&sysc R8A77965_PD_CA57_CPU0>;
>  			next-level-cache = <&L2_CA57>;
>  			enable-method = "psci";
>  		};
> @@ -69,14 +70,14 @@
>  			compatible = "arm,cortex-a57","arm,armv8";
>  			reg = <0x1>;
>  			device_type = "cpu";
> -			power-domains = <&sysc 1>;
> +			power-domains = <&sysc R8A77965_PD_CA57_CPU1>;
>  			next-level-cache = <&L2_CA57>;
>  			enable-method = "psci";
>  		};
>
>  		L2_CA57: cache-controller-0 {
>  			compatible = "cache";
> -			power-domains = <&sysc 12>;
> +			power-domains = <&sysc R8A77965_PD_CA57_SCU>;
>  			cache-unified;
>  			cache-level = <2>;
>  		};
> @@ -146,7 +147,7 @@
>  			#interrupt-cells = <2>;
>  			interrupt-controller;
>  			clocks = <&cpg CPG_MOD 912>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 912>;
>  		};
>
> @@ -161,7 +162,7 @@
>  			#interrupt-cells = <2>;
>  			interrupt-controller;
>  			clocks = <&cpg CPG_MOD 911>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 911>;
>  		};
>
> @@ -176,7 +177,7 @@
>  			#interrupt-cells = <2>;
>  			interrupt-controller;
>  			clocks = <&cpg CPG_MOD 910>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 910>;
>  		};
>
> @@ -191,7 +192,7 @@
>  			#interrupt-cells = <2>;
>  			interrupt-controller;
>  			clocks = <&cpg CPG_MOD 909>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 909>;
>  		};
>
> @@ -206,7 +207,7 @@
>  			#interrupt-cells = <2>;
>  			interrupt-controller;
>  			clocks = <&cpg CPG_MOD 908>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 908>;
>  		};
>
> @@ -221,7 +222,7 @@
>  			#interrupt-cells = <2>;
>  			interrupt-controller;
>  			clocks = <&cpg CPG_MOD 907>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 907>;
>  		};
>
> @@ -236,7 +237,7 @@
>  			#interrupt-cells = <2>;
>  			interrupt-controller;
>  			clocks = <&cpg CPG_MOD 906>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 906>;
>  		};
>
> @@ -251,7 +252,7 @@
>  			#interrupt-cells = <2>;
>  			interrupt-controller;
>  			clocks = <&cpg CPG_MOD 905>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 905>;
>  		};
>
> @@ -293,7 +294,7 @@
>  				      GIC_SPI 18 IRQ_TYPE_LEVEL_HIGH
>  				      GIC_SPI 161 IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&cpg CPG_MOD 407>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 407>;
>  		};
>
> @@ -347,7 +348,7 @@
>  			reg = <0 0xe60b0000 0 0x425>;
>  			interrupts = <GIC_SPI 173 IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&cpg CPG_MOD 926>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 926>;
>  			dmas = <&dmac0 0x11>, <&dmac0 0x10>;
>  			dma-names = "tx", "rx";
> @@ -366,7 +367,7 @@
>  			renesas,buswait = <11>;
>  			phys = <&usb2_phy0>;
>  			phy-names = "usb";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 704>;
>  			status = "disabled";
>  		};
> @@ -379,7 +380,7 @@
>  				      GIC_SPI 109 IRQ_TYPE_LEVEL_HIGH>;
>  			interrupt-names = "ch0", "ch1";
>  			clocks = <&cpg CPG_MOD 330>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 330>;
>  			#dma-cells = <1>;
>  			dma-channels = <2>;
> @@ -393,7 +394,7 @@
>  				      GIC_SPI 110 IRQ_TYPE_LEVEL_HIGH>;
>  			interrupt-names = "ch0", "ch1";
>  			clocks = <&cpg CPG_MOD 331>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 331>;
>  			#dma-cells = <1>;
>  			dma-channels = <2>;
> @@ -406,7 +407,7 @@
>  			clocks = <&cpg CPG_MOD 328>, <&usb3s0_clk>,
>  				 <&usb_extal_clk>;
>  			clock-names = "usb3-if", "usb3s_clk", "usb_extal";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 328>;
>  			#phy-cells = <0>;
>  			status = "disabled";
> @@ -440,7 +441,7 @@
>  					"ch12", "ch13", "ch14", "ch15";
>  			clocks = <&cpg CPG_MOD 219>;
>  			clock-names = "fck";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 219>;
>  			#dma-cells = <1>;
>  			dma-channels = <16>;
> @@ -474,7 +475,7 @@
>  					"ch12", "ch13", "ch14", "ch15";
>  			clocks = <&cpg CPG_MOD 218>;
>  			clock-names = "fck";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 218>;
>  			#dma-cells = <1>;
>  			dma-channels = <16>;
> @@ -508,7 +509,7 @@
>  					"ch12", "ch13", "ch14", "ch15";
>  			clocks = <&cpg CPG_MOD 217>;
>  			clock-names = "fck";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 217>;
>  			#dma-cells = <1>;
>  			dma-channels = <16>;
> @@ -551,7 +552,7 @@
>  					  "ch20", "ch21", "ch22", "ch23",
>  					  "ch24";
>  			clocks = <&cpg CPG_MOD 812>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 812>;
>  			phy-mode = "rgmii";
>  			#address-cells = <1>;
> @@ -565,7 +566,7 @@
>  			#pwm-cells = <2>;
>  			clocks = <&cpg CPG_MOD 523>;
>  			resets = <&cpg 523>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			status = "disabled";
>  		};
>
> @@ -575,7 +576,7 @@
>  			#pwm-cells = <2>;
>  			clocks = <&cpg CPG_MOD 523>;
>  			resets = <&cpg 523>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			status = "disabled";
>  		};
>
> @@ -585,7 +586,7 @@
>  			#pwm-cells = <2>;
>  			clocks = <&cpg CPG_MOD 523>;
>  			resets = <&cpg 523>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			status = "disabled";
>  		};
>
> @@ -595,7 +596,7 @@
>  			#pwm-cells = <2>;
>  			clocks = <&cpg CPG_MOD 523>;
>  			resets = <&cpg 523>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			status = "disabled";
>  		};
>
> @@ -605,7 +606,7 @@
>  			#pwm-cells = <2>;
>  			clocks = <&cpg CPG_MOD 523>;
>  			resets = <&cpg 523>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			status = "disabled";
>  		};
>
> @@ -615,7 +616,7 @@
>  			#pwm-cells = <2>;
>  			clocks = <&cpg CPG_MOD 523>;
>  			resets = <&cpg 523>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			status = "disabled";
>  		};
>
> @@ -625,7 +626,7 @@
>  			#pwm-cells = <2>;
>  			clocks = <&cpg CPG_MOD 523>;
>  			resets = <&cpg 523>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			status = "disabled";
>  		};
>
> @@ -641,7 +642,7 @@
>  			dmas = <&dmac1 0x51>, <&dmac1 0x50>,
>  			       <&dmac2 0x51>, <&dmac2 0x50>;
>  			dma-names = "tx", "rx", "tx", "rx";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 207>;
>  			status = "disabled";
>  		};
> @@ -658,7 +659,7 @@
>  			dmas = <&dmac1 0x53>, <&dmac1 0x52>,
>  			       <&dmac2 0x53>, <&dmac2 0x52>;
>  			dma-names = "tx", "rx", "tx", "rx";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 206>;
>  			status = "disabled";
>  		};
> @@ -672,7 +673,7 @@
>  				 <&cpg CPG_CORE 20>,
>  				 <&scif_clk>;
>  			clock-names = "fck", "brg_int", "scif_clk";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 310>;
>  			status = "disabled";
>  		};
> @@ -688,7 +689,7 @@
>  			clock-names = "fck", "brg_int", "scif_clk";
>  			dmas = <&dmac0 0x57>, <&dmac0 0x56>;
>  			dma-names = "tx", "rx";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 204>;
>  			status = "disabled";
>  		};
> @@ -704,7 +705,7 @@
>  			clock-names = "fck", "brg_int", "scif_clk";
>  			dmas = <&dmac0 0x59>, <&dmac0 0x58>;
>  			dma-names = "tx", "rx";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 203>;
>  			status = "disabled";
>  		};
> @@ -721,7 +722,7 @@
>  			dmas = <&dmac1 0x5b>, <&dmac1 0x5a>,
>  			       <&dmac2 0x5b>, <&dmac2 0x5a>;
>  			dma-names = "tx", "rx", "tx", "rx";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 202>;
>  			status = "disabled";
>  		};
> @@ -735,7 +736,7 @@
>  			dmas = <&dmac1 0x41>, <&dmac1 0x40>,
>  			       <&dmac2 0x41>, <&dmac2 0x40>;
>  			dma-names = "tx", "rx", "tx", "rx";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 211>;
>  			#address-cells = <1>;
>  			#size-cells = <0>;
> @@ -751,7 +752,7 @@
>  			dmas = <&dmac1 0x43>, <&dmac1 0x42>,
>  			       <&dmac2 0x43>, <&dmac2 0x42>;
>  			dma-names = "tx", "rx", "tx", "rx";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 210>;
>  			#address-cells = <1>;
>  			#size-cells = <0>;
> @@ -766,7 +767,7 @@
>  			clocks = <&cpg CPG_MOD 209>;
>  			dmas = <&dmac0 0x45>, <&dmac0 0x44>;
>  			dma-names = "tx", "rx";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 209>;
>  			#address-cells = <1>;
>  			#size-cells = <0>;
> @@ -781,7 +782,7 @@
>  			clocks = <&cpg CPG_MOD 208>;
>  			dmas = <&dmac0 0x47>, <&dmac0 0x46>;
>  			dma-names = "tx", "rx";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 208>;
>  			#address-cells = <1>;
>  			#size-cells = <0>;
> @@ -864,7 +865,7 @@
>  			reg = <0 0xee000000 0 0xc00>;
>  			interrupts = <GIC_SPI 102 IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&cpg CPG_MOD 328>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 328>;
>  			status = "disabled";
>  		};
> @@ -875,7 +876,7 @@
>  			reg = <0 0xee020000 0 0x400>;
>  			interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&cpg CPG_MOD 328>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 328>;
>  			status = "disabled";
>  		};
> @@ -887,7 +888,7 @@
>  			clocks = <&cpg CPG_MOD 703>;
>  			phys = <&usb2_phy0>;
>  			phy-names = "usb";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 703>;
>  			status = "disabled";
>  		};
> @@ -899,7 +900,7 @@
>  			clocks = <&cpg CPG_MOD 702>;
>  			phys = <&usb2_phy1>;
>  			phy-names = "usb";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 702>;
>  			status = "disabled";
>  		};
> @@ -912,7 +913,7 @@
>  			phys = <&usb2_phy0>;
>  			phy-names = "usb";
>  			companion = <&ohci0>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 703>;
>  			status = "disabled";
>  		};
> @@ -925,7 +926,7 @@
>  			phys = <&usb2_phy1>;
>  			phy-names = "usb";
>  			companion = <&ohci1>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 702>;
>  			status = "disabled";
>  		};
> @@ -936,7 +937,7 @@
>  			reg = <0 0xee080200 0 0x700>;
>  			interrupts = <GIC_SPI 108 IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&cpg CPG_MOD 703>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 703>;
>  			#phy-cells = <0>;
>  			status = "disabled";
> @@ -947,7 +948,7 @@
>  				     "renesas,rcar-gen3-usb2-phy";
>  			reg = <0 0xee0a0200 0 0x700>;
>  			clocks = <&cpg CPG_MOD 703>;
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 703>;
>  			#phy-cells = <0>;
>  			status = "disabled";
> @@ -986,7 +987,7 @@
>  					(GIC_CPU_MASK_SIMPLE(2) | IRQ_TYPE_LEVEL_HIGH)>;
>  			clocks = <&cpg CPG_MOD 408>;
>  			clock-names = "clk";
> -			power-domains = <&sysc 32>;
> +			power-domains = <&sysc R8A77965_PD_ALWAYS_ON>;
>  			resets = <&cpg 408>;
>  		};
>
> --
> 2.17.0
>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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

* Re: [PATCH] arm64: dts: renesas: r8a77965: use r8a77965-sysc binding definitions
  2018-04-26 19:34 [PATCH] arm64: dts: renesas: r8a77965: use r8a77965-sysc binding definitions Niklas Söderlund
  2018-04-27  7:19 ` jacopo mondi
@ 2018-05-02  7:08 ` Simon Horman
  1 sibling, 0 replies; 3+ messages in thread
From: Simon Horman @ 2018-05-02  7:08 UTC (permalink / raw)
  To: Niklas Söderlund; +Cc: Geert Uytterhoeven, linux-renesas-soc

On Thu, Apr 26, 2018 at 09:34:47PM +0200, Niklas Söderlund wrote:
> Replace the hardcoded power domain indices by R8A77965_PD_* symbols.
> 
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> ---
>  arch/arm64/boot/dts/renesas/r8a77965.dtsi | 95 ++++++++++++-----------
>  1 file changed, 48 insertions(+), 47 deletions(-)
> 
> Hi Simon,
> 
> This is based on top of your trees devel branch from earlier today.

Thanks, applied.

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

end of thread, other threads:[~2018-05-02  7:37 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-26 19:34 [PATCH] arm64: dts: renesas: r8a77965: use r8a77965-sysc binding definitions Niklas Söderlund
2018-04-27  7:19 ` jacopo mondi
2018-05-02  7:08 ` Simon Horman

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.