[PATCH 0/3] arm: dts: renesas: Drop _clk suffix from clock node names

[PATCH 0/3] arm: dts: renesas: Drop _clk suffix from clock node names

[Geert Uytterhoeven]

	Hi Simon, Magnus,

The current practice is to not add _clk suffixes to clock node names in
DT, as these names are used as the actual clock names.

This patch removes the remaining offenders in the various Renesas DTSes.

Thanks!

Geert Uytterhoeven (3):
  ARM: dts: r8a7790: Drop _clk suffix from external CAN clock node name
  ARM: dts: r8a7791: Drop _clk suffix from external CAN clock node name
  arm64: dts: r8a7795: salvator-x: Drop _clk suffix from X12 clock node
    name

 arch/arm/boot/dts/r8a7790.dtsi                     | 2 +-
 arch/arm/boot/dts/r8a7791.dtsi                     | 2 +-
 arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

-- 
2.7.4

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

[PATCH 0/3] arm: dts: renesas: Drop _clk suffix from clock node names

[Geert Uytterhoeven]

	Hi Simon, Magnus,

The current practice is to not add _clk suffixes to clock node names in
DT, as these names are used as the actual clock names.

This patch removes the remaining offenders in the various Renesas DTSes.

Thanks!

Geert Uytterhoeven (3):
  ARM: dts: r8a7790: Drop _clk suffix from external CAN clock node name
  ARM: dts: r8a7791: Drop _clk suffix from external CAN clock node name
  arm64: dts: r8a7795: salvator-x: Drop _clk suffix from X12 clock node
    name

 arch/arm/boot/dts/r8a7790.dtsi                     | 2 +-
 arch/arm/boot/dts/r8a7791.dtsi                     | 2 +-
 arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

-- 
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

[PATCH 1/3] ARM: dts: r8a7790: Drop _clk suffix from external CAN clock node name

[Geert Uytterhoeven]

The current practice is to not add _clk suffixes to clock node names in
DT, as these names are used as the actual clock names.

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

diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
index fe6b8c2a2d711536..99269aaca6fc953e 100644
--- a/arch/arm/boot/dts/r8a7790.dtsi
+++ b/arch/arm/boot/dts/r8a7790.dtsi
@@ -1101,7 +1101,7 @@
 		};
 
 		/* External CAN clock */
-		can_clk: can_clk {
+		can_clk: can {
 			compatible = "fixed-clock";
 			#clock-cells = <0>;
 			/* This value must be overridden by the board. */
-- 
2.7.4

[PATCH 1/3] ARM: dts: r8a7790: Drop _clk suffix from external CAN clock node name

[Geert Uytterhoeven]

The current practice is to not add _clk suffixes to clock node names in
DT, as these names are used as the actual clock names.

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

diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
index fe6b8c2a2d711536..99269aaca6fc953e 100644
--- a/arch/arm/boot/dts/r8a7790.dtsi
+++ b/arch/arm/boot/dts/r8a7790.dtsi
@@ -1101,7 +1101,7 @@
 		};
 
 		/* External CAN clock */
-		can_clk: can_clk {
+		can_clk: can {
 			compatible = "fixed-clock";
 			#clock-cells = <0>;
 			/* This value must be overridden by the board. */
-- 
2.7.4

[PATCH 2/3] ARM: dts: r8a7791: Drop _clk suffix from external CAN clock node name

[Geert Uytterhoeven]

The current practice is to not add _clk suffixes to clock node names in
DT, as these names are used as the actual clock names.

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

diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi
index a6478ca3f4ca7554..4d0c2ce59900f811 100644
--- a/arch/arm/boot/dts/r8a7791.dtsi
+++ b/arch/arm/boot/dts/r8a7791.dtsi
@@ -1126,7 +1126,7 @@
 		};
 
 		/* External CAN clock */
-		can_clk: can_clk {
+		can_clk: can {
 			compatible = "fixed-clock";
 			#clock-cells = <0>;
 			/* This value must be overridden by the board. */
-- 
2.7.4

[PATCH 2/3] ARM: dts: r8a7791: Drop _clk suffix from external CAN clock node name

[Geert Uytterhoeven]

The current practice is to not add _clk suffixes to clock node names in
DT, as these names are used as the actual clock names.

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

diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi
index a6478ca3f4ca7554..4d0c2ce59900f811 100644
--- a/arch/arm/boot/dts/r8a7791.dtsi
+++ b/arch/arm/boot/dts/r8a7791.dtsi
@@ -1126,7 +1126,7 @@
 		};
 
 		/* External CAN clock */
-		can_clk: can_clk {
+		can_clk: can {
 			compatible = "fixed-clock";
 			#clock-cells = <0>;
 			/* This value must be overridden by the board. */
-- 
2.7.4

[PATCH 3/3] arm64: dts: r8a7795: salvator-x: Drop _clk suffix from X12 clock node name

[Geert Uytterhoeven]

The current practice is to not add _clk suffixes to clock node names in
DT, as these names are used as the actual clock names.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
index f25241921067dcef..639aa085d9966676 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
@@ -56,7 +56,7 @@
 		reg = <0x0 0x48000000 0x0 0x38000000>;
 	};
 
-	x12_clk: x12_clk {
+	x12_clk: x12 {
 		compatible = "fixed-clock";
 		#clock-cells = <0>;
 		clock-frequency = <24576000>;
-- 
2.7.4

[PATCH 3/3] arm64: dts: r8a7795: salvator-x: Drop _clk suffix from X12 clock node name

[Geert Uytterhoeven]

The current practice is to not add _clk suffixes to clock node names in
DT, as these names are used as the actual clock names.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
index f25241921067dcef..639aa085d9966676 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
@@ -56,7 +56,7 @@
 		reg = <0x0 0x48000000 0x0 0x38000000>;
 	};
 
-	x12_clk: x12_clk {
+	x12_clk: x12 {
 		compatible = "fixed-clock";
 		#clock-cells = <0>;
 		clock-frequency = <24576000>;
-- 
2.7.4

Re: [PATCH 0/3] arm: dts: renesas: Drop _clk suffix from clock node names

[Stephen Boyd]

On 04/03, Geert Uytterhoeven wrote:
> 	Hi Simon, Magnus,
> 
> The current practice is to not add _clk suffixes to clock node names in
> DT, as these names are used as the actual clock names.
> 
> This patch removes the remaining offenders in the various Renesas DTSes.
> 

This changes the names in the clk framework too though? If you're
ok with that I'm ok too.

Acked-by: Stephen Boyd <sboyd@codeaurora.org>

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

[PATCH 0/3] arm: dts: renesas: Drop _clk suffix from clock node names

[Stephen Boyd]

On 04/03, Geert Uytterhoeven wrote:
> 	Hi Simon, Magnus,
> 
> The current practice is to not add _clk suffixes to clock node names in
> DT, as these names are used as the actual clock names.
> 
> This patch removes the remaining offenders in the various Renesas DTSes.
> 

This changes the names in the clk framework too though? If you're
ok with that I'm ok too.

Acked-by: Stephen Boyd <sboyd@codeaurora.org>

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

Re: [PATCH 0/3] arm: dts: renesas: Drop _clk suffix from clock node names

[Geert Uytterhoeven]

Hi Stephen,

On Wed, Apr 5, 2017 at 3:38 AM, Stephen Boyd <sboyd@codeaurora.org> wrote:
> On 04/03, Geert Uytterhoeven wrote:
>> The current practice is to not add _clk suffixes to clock node names in
>> DT, as these names are used as the actual clock names.
>>
>> This patch removes the remaining offenders in the various Renesas DTSes.
>
> This changes the names in the clk framework too though? If you're
> ok with that I'm ok too.

Yes, that's my main motivation doing this: it's a bit silly to have some clocks
show up in /sys/kernel/debug/clk/clk_summary with a "_clk" suffix.

> Acked-by: Stephen Boyd <sboyd@codeaurora.org>

Thanks!

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

[PATCH 0/3] arm: dts: renesas: Drop _clk suffix from clock node names

[Geert Uytterhoeven]

Hi Stephen,

On Wed, Apr 5, 2017 at 3:38 AM, Stephen Boyd <sboyd@codeaurora.org> wrote:
> On 04/03, Geert Uytterhoeven wrote:
>> The current practice is to not add _clk suffixes to clock node names in
>> DT, as these names are used as the actual clock names.
>>
>> This patch removes the remaining offenders in the various Renesas DTSes.
>
> This changes the names in the clk framework too though? If you're
> ok with that I'm ok too.

Yes, that's my main motivation doing this: it's a bit silly to have some clocks
show up in /sys/kernel/debug/clk/clk_summary with a "_clk" suffix.

> Acked-by: Stephen Boyd <sboyd@codeaurora.org>

Thanks!

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

Re: [PATCH 0/3] arm: dts: renesas: Drop _clk suffix from clock node names

[Simon Horman]

On Wed, Apr 05, 2017 at 09:09:20AM +0200, Geert Uytterhoeven wrote:
> Hi Stephen,
> 
> On Wed, Apr 5, 2017 at 3:38 AM, Stephen Boyd <sboyd@codeaurora.org> wrote:
> > On 04/03, Geert Uytterhoeven wrote:
> >> The current practice is to not add _clk suffixes to clock node names in
> >> DT, as these names are used as the actual clock names.
> >>
> >> This patch removes the remaining offenders in the various Renesas DTSes.
> >
> > This changes the names in the clk framework too though? If you're
> > ok with that I'm ok too.
> 
> Yes, that's my main motivation doing this: it's a bit silly to have some clocks
> show up in /sys/kernel/debug/clk/clk_summary with a "_clk" suffix.
> 
> > Acked-by: Stephen Boyd <sboyd@codeaurora.org>
> 
> Thanks!

Thanks, I have queued these up.

Re: [PATCH 0/3] arm: dts: renesas: Drop _clk suffix from clock node names

[Simon Horman]

On Wed, Apr 05, 2017 at 09:09:20AM +0200, Geert Uytterhoeven wrote:
> Hi Stephen,
> 
> On Wed, Apr 5, 2017 at 3:38 AM, Stephen Boyd <sboyd@codeaurora.org> wrote:
> > On 04/03, Geert Uytterhoeven wrote:
> >> The current practice is to not add _clk suffixes to clock node names in
> >> DT, as these names are used as the actual clock names.
> >>
> >> This patch removes the remaining offenders in the various Renesas DTSes.
> >
> > This changes the names in the clk framework too though? If you're
> > ok with that I'm ok too.
> 
> Yes, that's my main motivation doing this: it's a bit silly to have some clocks
> show up in /sys/kernel/debug/clk/clk_summary with a "_clk" suffix.
> 
> > Acked-by: Stephen Boyd <sboyd@codeaurora.org>
> 
> Thanks!

Thanks, I have queued these up.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

[PATCH 0/3] arm: dts: renesas: Drop _clk suffix from clock node names

[Simon Horman]

On Wed, Apr 05, 2017 at 09:09:20AM +0200, Geert Uytterhoeven wrote:
> Hi Stephen,
> 
> On Wed, Apr 5, 2017 at 3:38 AM, Stephen Boyd <sboyd@codeaurora.org> wrote:
> > On 04/03, Geert Uytterhoeven wrote:
> >> The current practice is to not add _clk suffixes to clock node names in
> >> DT, as these names are used as the actual clock names.
> >>
> >> This patch removes the remaining offenders in the various Renesas DTSes.
> >
> > This changes the names in the clk framework too though? If you're
> > ok with that I'm ok too.
> 
> Yes, that's my main motivation doing this: it's a bit silly to have some clocks
> show up in /sys/kernel/debug/clk/clk_summary with a "_clk" suffix.
> 
> > Acked-by: Stephen Boyd <sboyd@codeaurora.org>
> 
> Thanks!

Thanks, I have queued these up.