[PATCH 1/2] ARM: dts: r8a7792: add IMR-L[S]X3 clocks

[PATCH 1/2] ARM: dts: r8a7792: add IMR-L[S]X3 clocks

[Sergei Shtylyov]

[-- Attachment #1: ARM-dts-r8a7792-add-IMR-L-S-X3-clocks.patch --]
[-- Type: text/plain, Size: 2623 bytes --]

Add the IMR-L[S]X3 clocks to the R8A7792 device tree.

Based on the original patch by Roman Meshkevich
<roman.meshkevich-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>.

Signed-off-by: Sergei Shtylyov <sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>

---
 arch/arm/boot/dts/r8a7792.dtsi            |   15 ++++++++++++---
 include/dt-bindings/clock/r8a7792-clock.h |    7 +++++++
 2 files changed, 19 insertions(+), 3 deletions(-)

Index: renesas/arch/arm/boot/dts/r8a7792.dtsi
===================================================================
--- renesas.orig/arch/arm/boot/dts/r8a7792.dtsi
+++ renesas/arch/arm/boot/dts/r8a7792.dtsi
@@ -927,16 +927,25 @@
 				     "renesas,cpg-mstp-clocks";
 			reg = <0 0xe6150990 0 4>, <0 0xe61509a0 0 4>;
 			clocks = <&zg_clk>, <&zg_clk>, <&zg_clk>, <&zg_clk>,
-			         <&zg_clk>, <&zg_clk>, <&hp_clk>;
+				 <&zg_clk>, <&zg_clk>, <&hp_clk>, <&zg_clk>,
+				 <&zg_clk>, <&zg_clk>, <&zg_clk>, <&zg_clk>,
+				 <&zg_clk>, <&zg_clk>;
 			#clock-cells = <1>;
 			clock-indices = <
 				R8A7792_CLK_VIN5 R8A7792_CLK_VIN4
 				R8A7792_CLK_VIN3 R8A7792_CLK_VIN2
 				R8A7792_CLK_VIN1 R8A7792_CLK_VIN0
-				R8A7792_CLK_ETHERAVB
+				R8A7792_CLK_ETHERAVB R8A7792_CLK_IMR_LX3
+				R8A7792_CLK_IMR_LSX3_1 R8A7792_CLK_IMR_LSX3_0
+				R8A7792_CLK_IMR_LSX3_5 R8A7792_CLK_IMR_LSX3_4
+				R8A7792_CLK_IMR_LSX3_3 R8A7792_CLK_IMR_LSX3_2
 			>;
 			clock-output-names = "vin5", "vin4", "vin3", "vin2",
-					     "vin1", "vin0", "etheravb";
+					     "vin1", "vin0",
+					     "etheravb", "imr-lx3",
+					     "imr-lsx3-1", "imr-lsx3-0",
+					     "imr-lsx3-5", "imr-lsx3-4",
+					     "imr-lsx3-3", "imr-lsx3-2";
 		};
 		mstp9_clks: mstp9_clks@e6150994 {
 			compatible = "renesas,r8a7792-mstp-clocks",
Index: renesas/include/dt-bindings/clock/r8a7792-clock.h
===================================================================
--- renesas.orig/include/dt-bindings/clock/r8a7792-clock.h
+++ renesas/include/dt-bindings/clock/r8a7792-clock.h
@@ -70,6 +70,13 @@
 #define R8A7792_CLK_VIN1		10
 #define R8A7792_CLK_VIN0		11
 #define R8A7792_CLK_ETHERAVB		12
+#define R8A7792_CLK_IMR_LX3		21
+#define R8A7792_CLK_IMR_LSX3_1		22
+#define R8A7792_CLK_IMR_LSX3_0		23
+#define R8A7792_CLK_IMR_LSX3_5		25
+#define R8A7792_CLK_IMR_LSX3_4		26
+#define R8A7792_CLK_IMR_LSX3_3		27
+#define R8A7792_CLK_IMR_LSX3_2		28
 
 /* MSTP9 */
 #define R8A7792_CLK_GPIO7		4

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] ARM: dts: r8a7792: add IMR-L[S]X3 clocks

[Sergei Shtylyov]

Still not cover letter but I think it's better now, so I won't retry...
The patches are against Simon Horman's 'renesas.git' repo,
'renesas-devel-20170321-v4.11-rc3' tag.

MBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] ARM: dts: r8a7792: add IMR-L[S]X3 clocks

[Sergei Shtylyov]

On 03/21/2017 11:08 PM, Sergei Shtylyov wrote:

> Still not cover letter but I think it's better now, so I won't retry...
> The patches are against Simon Horman's 'renesas.git' repo,
> 'renesas-devel-20170321-v4.11-rc3' tag.

    The patches depend on the IMR driver patches adding the IMR-L[S]X3 support 
for the "compatible" values.

MBR, Sergei

Re: [PATCH 1/2] ARM: dts: r8a7792: add IMR-L[S]X3 clocks

[Simon Horman]

On Tue, Mar 21, 2017 at 11:14:43PM +0300, Sergei Shtylyov wrote:
> On 03/21/2017 11:08 PM, Sergei Shtylyov wrote:
> 
> >Still not cover letter but I think it's better now, so I won't retry...
> >The patches are against Simon Horman's 'renesas.git' repo,
> >'renesas-devel-20170321-v4.11-rc3' tag.
> 
>    The patches depend on the IMR driver patches adding the IMR-L[S]X3
> support for the "compatible" values.

I have marked these patches as deferred accordingly.

Re: [PATCH 1/2] ARM: dts: r8a7792: add IMR-L[S]X3 clocks

[Sergei Shtylyov]

Hello!

On 03/29/2017 11:04 AM, Simon Horman wrote:

>>> Still not cover letter but I think it's better now, so I won't retry...
>>> The patches are against Simon Horman's 'renesas.git' repo,
>>> 'renesas-devel-20170321-v4.11-rc3' tag.
>>
>>    The patches depend on the IMR driver patches adding the IMR-L[S]X3
>> support for the "compatible" values.
>
> I have marked these patches as deferred accordingly.

    Actually, the 1st patch in htis series doesn't depend on anything and 
should still apply (with offsets).

WBR, Sergei

Re: [PATCH 1/2] ARM: dts: r8a7792: add IMR-L[S]X3 clocks

[Simon Horman]

On Wed, Jun 28, 2017 at 11:49:01PM +0300, Sergei Shtylyov wrote:
> Hello!
> 
> On 03/29/2017 11:04 AM, Simon Horman wrote:
> 
> >>>Still not cover letter but I think it's better now, so I won't retry...
> >>>The patches are against Simon Horman's 'renesas.git' repo,
> >>>'renesas-devel-20170321-v4.11-rc3' tag.
> >>
> >>   The patches depend on the IMR driver patches adding the IMR-L[S]X3
> >>support for the "compatible" values.
> >
> >I have marked these patches as deferred accordingly.
> 
>    Actually, the 1st patch in htis series doesn't depend on anything and
> should still apply (with offsets).

I'd prefer to take both patches at the same time unless there is a strong
motivation to just take the first one.

[PATCH 1/2] ARM: dts: r8a7792: add IMR-L[S]X3 clocks

[Sergei Shtylyov]

[-- Attachment #1: ARM-dts-r8a7792-add-IMR-L-S-X3-clocks.patch --]
[-- Type: text/plain, Size: 2342 bytes --]

Add the IMR-L[S]X3 clocks to the R8A7792 device tree.

Based on the original patch by Roman Meshkevich
<roman.meshkevich@cogentembedded.com>.

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

---
 arch/arm/boot/dts/r8a7792.dtsi            |   15 ++++++++++++---
 include/dt-bindings/clock/r8a7792-clock.h |    7 +++++++
 2 files changed, 19 insertions(+), 3 deletions(-)

Index: renesas/arch/arm/boot/dts/r8a7792.dtsi
===================================================================
--- renesas.orig/arch/arm/boot/dts/r8a7792.dtsi
+++ renesas/arch/arm/boot/dts/r8a7792.dtsi
@@ -927,16 +927,25 @@
 				     "renesas,cpg-mstp-clocks";
 			reg = <0 0xe6150990 0 4>, <0 0xe61509a0 0 4>;
 			clocks = <&zg_clk>, <&zg_clk>, <&zg_clk>, <&zg_clk>,
-			         <&zg_clk>, <&zg_clk>, <&hp_clk>;
+				 <&zg_clk>, <&zg_clk>, <&hp_clk>, <&zg_clk>,
+				 <&zg_clk>, <&zg_clk>, <&zg_clk>, <&zg_clk>,
+				 <&zg_clk>, <&zg_clk>;
 			#clock-cells = <1>;
 			clock-indices = <
 				R8A7792_CLK_VIN5 R8A7792_CLK_VIN4
 				R8A7792_CLK_VIN3 R8A7792_CLK_VIN2
 				R8A7792_CLK_VIN1 R8A7792_CLK_VIN0
-				R8A7792_CLK_ETHERAVB
+				R8A7792_CLK_ETHERAVB R8A7792_CLK_IMR_LX3
+				R8A7792_CLK_IMR_LSX3_1 R8A7792_CLK_IMR_LSX3_0
+				R8A7792_CLK_IMR_LSX3_5 R8A7792_CLK_IMR_LSX3_4
+				R8A7792_CLK_IMR_LSX3_3 R8A7792_CLK_IMR_LSX3_2
 			>;
 			clock-output-names = "vin5", "vin4", "vin3", "vin2",
-					     "vin1", "vin0", "etheravb";
+					     "vin1", "vin0",
+					     "etheravb", "imr-lx3",
+					     "imr-lsx3-1", "imr-lsx3-0",
+					     "imr-lsx3-5", "imr-lsx3-4",
+					     "imr-lsx3-3", "imr-lsx3-2";
 		};
 		mstp9_clks: mstp9_clks@e6150994 {
 			compatible = "renesas,r8a7792-mstp-clocks",
Index: renesas/include/dt-bindings/clock/r8a7792-clock.h
===================================================================
--- renesas.orig/include/dt-bindings/clock/r8a7792-clock.h
+++ renesas/include/dt-bindings/clock/r8a7792-clock.h
@@ -70,6 +70,13 @@
 #define R8A7792_CLK_VIN1		10
 #define R8A7792_CLK_VIN0		11
 #define R8A7792_CLK_ETHERAVB		12
+#define R8A7792_CLK_IMR_LX3		21
+#define R8A7792_CLK_IMR_LSX3_1		22
+#define R8A7792_CLK_IMR_LSX3_0		23
+#define R8A7792_CLK_IMR_LSX3_5		25
+#define R8A7792_CLK_IMR_LSX3_4		26
+#define R8A7792_CLK_IMR_LSX3_3		27
+#define R8A7792_CLK_IMR_LSX3_2		28
 
 /* MSTP9 */
 #define R8A7792_CLK_GPIO7		4

Re: [PATCH 1/2] ARM: dts: r8a7792: add IMR-L[S]X3 clocks

[Sergei Shtylyov]

Hello!

    Sorry for the bad patch dates and no cover letter -- I'm trying to beat 
'quilt mail' into submission...

WBR, Sergei

Re: [PATCH 1/2] ARM: dts: r8a7792: add IMR-L[S]X3 clocks

[Geert Uytterhoeven]

On Thu, Jan 1, 1970 at 12:00 AM, Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
> Add the IMR-L[S]X3 clocks to the R8A7792 device tree.
>
> Based on the original patch by Roman Meshkevich
> <roman.meshkevich@cogentembedded.com>.
>
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

I can't verify the parent clocks, though.

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 1/2] ARM: dts: r8a7792: add IMR-L[S]X3 clocks

[Simon Horman]

On Thu, Jan 01, 1970 at 03:00:01AM +0300, Sergei Shtylyov wrote:
> Add the IMR-L[S]X3 clocks to the R8A7792 device tree.
> 
> Based on the original patch by Roman Meshkevich
> <roman.meshkevich-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>.
> 
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
> 
> ---
>  arch/arm/boot/dts/r8a7792.dtsi            |   15 ++++++++++++---
>  include/dt-bindings/clock/r8a7792-clock.h |    7 +++++++
>  2 files changed, 19 insertions(+), 3 deletions(-)
> 
> Index: renesas/arch/arm/boot/dts/r8a7792.dtsi
> ===================================================================
> --- renesas.orig/arch/arm/boot/dts/r8a7792.dtsi
> +++ renesas/arch/arm/boot/dts/r8a7792.dtsi
> @@ -927,16 +927,25 @@
>  				     "renesas,cpg-mstp-clocks";
>  			reg = <0 0xe6150990 0 4>, <0 0xe61509a0 0 4>;
>  			clocks = <&zg_clk>, <&zg_clk>, <&zg_clk>, <&zg_clk>,
> -			         <&zg_clk>, <&zg_clk>, <&hp_clk>;
> +				 <&zg_clk>, <&zg_clk>, <&hp_clk>, <&zg_clk>,
> +				 <&zg_clk>, <&zg_clk>, <&zg_clk>, <&zg_clk>,
> +				 <&zg_clk>, <&zg_clk>;
>  			#clock-cells = <1>;
>  			clock-indices = <
>  				R8A7792_CLK_VIN5 R8A7792_CLK_VIN4
>  				R8A7792_CLK_VIN3 R8A7792_CLK_VIN2
>  				R8A7792_CLK_VIN1 R8A7792_CLK_VIN0
> -				R8A7792_CLK_ETHERAVB
> +				R8A7792_CLK_ETHERAVB R8A7792_CLK_IMR_LX3
> +				R8A7792_CLK_IMR_LSX3_1 R8A7792_CLK_IMR_LSX3_0
> +				R8A7792_CLK_IMR_LSX3_5 R8A7792_CLK_IMR_LSX3_4
> +				R8A7792_CLK_IMR_LSX3_3 R8A7792_CLK_IMR_LSX3_2
>  			>;
>  			clock-output-names = "vin5", "vin4", "vin3", "vin2",
> -					     "vin1", "vin0", "etheravb";
> +					     "vin1", "vin0",
> +					     "etheravb", "imr-lx3",
> +					     "imr-lsx3-1", "imr-lsx3-0",
> +					     "imr-lsx3-5", "imr-lsx3-4",
> +					     "imr-lsx3-3", "imr-lsx3-2";
>  		};
>  		mstp9_clks: mstp9_clks@e6150994 {
>  			compatible = "renesas,r8a7792-mstp-clocks",
> Index: renesas/include/dt-bindings/clock/r8a7792-clock.h
> ===================================================================
> --- renesas.orig/include/dt-bindings/clock/r8a7792-clock.h
> +++ renesas/include/dt-bindings/clock/r8a7792-clock.h
> @@ -70,6 +70,13 @@
>  #define R8A7792_CLK_VIN1		10
>  #define R8A7792_CLK_VIN0		11
>  #define R8A7792_CLK_ETHERAVB		12
> +#define R8A7792_CLK_IMR_LX3		21
> +#define R8A7792_CLK_IMR_LSX3_1		22
> +#define R8A7792_CLK_IMR_LSX3_0		23
> +#define R8A7792_CLK_IMR_LSX3_5		25
> +#define R8A7792_CLK_IMR_LSX3_4		26
> +#define R8A7792_CLK_IMR_LSX3_3		27
> +#define R8A7792_CLK_IMR_LSX3_2		28
>  
>  /* MSTP9 */
>  #define R8A7792_CLK_GPIO7		4
> 

I think this patch needs to be split in two, one for each file,
to allow queuing bindings and DT changes in different branches.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] ARM: dts: r8a7792: add IMR-L[S]X3 clocks

[Geert Uytterhoeven]

On Fri, Jun 30, 2017 at 12:01 PM, Simon Horman <horms@verge.net.au> wrote:
> On Thu, Jan 01, 1970 at 03:00:01AM +0300, Sergei Shtylyov wrote:
>> Add the IMR-L[S]X3 clocks to the R8A7792 device tree.
>>
>> Based on the original patch by Roman Meshkevich
>> <roman.meshkevich@cogentembedded.com>.
>>
>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>
>> ---
>>  arch/arm/boot/dts/r8a7792.dtsi            |   15 ++++++++++++---
>>  include/dt-bindings/clock/r8a7792-clock.h |    7 +++++++
>>  2 files changed, 19 insertions(+), 3 deletions(-)
>>
>> Index: renesas/arch/arm/boot/dts/r8a7792.dtsi
>> ===================================================================
>> --- renesas.orig/arch/arm/boot/dts/r8a7792.dtsi
>> +++ renesas/arch/arm/boot/dts/r8a7792.dtsi
>> @@ -927,16 +927,25 @@
>>                                    "renesas,cpg-mstp-clocks";
>>                       reg = <0 0xe6150990 0 4>, <0 0xe61509a0 0 4>;
>>                       clocks = <&zg_clk>, <&zg_clk>, <&zg_clk>, <&zg_clk>,
>> -                              <&zg_clk>, <&zg_clk>, <&hp_clk>;
>> +                              <&zg_clk>, <&zg_clk>, <&hp_clk>, <&zg_clk>,
>> +                              <&zg_clk>, <&zg_clk>, <&zg_clk>, <&zg_clk>,
>> +                              <&zg_clk>, <&zg_clk>;
>>                       #clock-cells = <1>;
>>                       clock-indices = <
>>                               R8A7792_CLK_VIN5 R8A7792_CLK_VIN4
>>                               R8A7792_CLK_VIN3 R8A7792_CLK_VIN2
>>                               R8A7792_CLK_VIN1 R8A7792_CLK_VIN0
>> -                             R8A7792_CLK_ETHERAVB
>> +                             R8A7792_CLK_ETHERAVB R8A7792_CLK_IMR_LX3
>> +                             R8A7792_CLK_IMR_LSX3_1 R8A7792_CLK_IMR_LSX3_0
>> +                             R8A7792_CLK_IMR_LSX3_5 R8A7792_CLK_IMR_LSX3_4
>> +                             R8A7792_CLK_IMR_LSX3_3 R8A7792_CLK_IMR_LSX3_2
>>                       >;
>>                       clock-output-names = "vin5", "vin4", "vin3", "vin2",
>> -                                          "vin1", "vin0", "etheravb";
>> +                                          "vin1", "vin0",
>> +                                          "etheravb", "imr-lx3",
>> +                                          "imr-lsx3-1", "imr-lsx3-0",
>> +                                          "imr-lsx3-5", "imr-lsx3-4",
>> +                                          "imr-lsx3-3", "imr-lsx3-2";
>>               };
>>               mstp9_clks: mstp9_clks@e6150994 {
>>                       compatible = "renesas,r8a7792-mstp-clocks",
>> Index: renesas/include/dt-bindings/clock/r8a7792-clock.h
>> ===================================================================
>> --- renesas.orig/include/dt-bindings/clock/r8a7792-clock.h
>> +++ renesas/include/dt-bindings/clock/r8a7792-clock.h
>> @@ -70,6 +70,13 @@
>>  #define R8A7792_CLK_VIN1             10
>>  #define R8A7792_CLK_VIN0             11
>>  #define R8A7792_CLK_ETHERAVB         12
>> +#define R8A7792_CLK_IMR_LX3          21
>> +#define R8A7792_CLK_IMR_LSX3_1               22
>> +#define R8A7792_CLK_IMR_LSX3_0               23
>> +#define R8A7792_CLK_IMR_LSX3_5               25
>> +#define R8A7792_CLK_IMR_LSX3_4               26
>> +#define R8A7792_CLK_IMR_LSX3_3               27
>> +#define R8A7792_CLK_IMR_LSX3_2               28
>>
>>  /* MSTP9 */
>>  #define R8A7792_CLK_GPIO7            4
>>
>
> I think this patch needs to be split in two, one for each file,
> to allow queuing bindings and DT changes in different branches.

I don't know if it's worth still adding this.
About 47 years after the original patch date, my plan is to convert r8a7792.dtsi
to the new CPG/MSSR bindings, cfr. latest renesas-drivers.

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