[PATCH 00/13] arm: dts: renesas: Tidy-up status

[PATCH 00/13] arm: dts: renesas: Tidy-up status

[Geert Uytterhoeven]

	Hi Simon, Magnus,

This patch series performs some cleanups on the Renesas DT source files:
  - Patch 1 replaces "ok" by "okay" in statuses, as the latter is the
    recommended form,
  - Patches 2-13 drop superfluous status updates when overriding the
    clock frequencies of fixed-frequency clocks, as they have already
    been enabled in the SoC-specific DTS file.

Thanks!

Geert Uytterhoeven (13):
  ARM: dts: porter: Always use status "okay" to enable devices
  ARM: dts: bockw: Drop superfluous status update for frequency override
  ARM: dts: marzen: Drop superfluous status update for frequency
    override
  ARM: dts: lager: Drop superfluous status update for frequency override
  ARM: dts: koelsch: Drop superfluous status updates for frequency
    overrides
  ARM: dts: porter: Drop superfluous status update for frequency
    override
  ARM: dts: gose: Drop superfluous status update for frequency override
  ARM: dts: alt: Drop superfluous status update for frequency override
  ARM: dts: silk: Drop superfluous status update for frequency override
  arm64: dts: h3ulcb: Drop superfluous status update for frequency
    override
  arm64: dts: r8a7795: salvator-x: Drop superfluous status updates for
    frequency overrides
  arm64: dts: m3ulcb: Drop superfluous status update for frequency
    override
  arm64: dts: r8a7796: salvator-x: Drop superfluous status update for
    frequency override

 arch/arm/boot/dts/r8a7778-bockw.dts                | 1 -
 arch/arm/boot/dts/r8a7779-marzen.dts               | 1 -
 arch/arm/boot/dts/r8a7790-lager.dts                | 1 -
 arch/arm/boot/dts/r8a7791-koelsch.dts              | 2 --
 arch/arm/boot/dts/r8a7791-porter.dts               | 5 ++---
 arch/arm/boot/dts/r8a7793-gose.dts                 | 1 -
 arch/arm/boot/dts/r8a7794-alt.dts                  | 1 -
 arch/arm/boot/dts/r8a7794-silk.dts                 | 1 -
 arch/arm64/boot/dts/renesas/r8a7795-h3ulcb.dts     | 1 -
 arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 2 --
 arch/arm64/boot/dts/renesas/r8a7796-m3ulcb.dts     | 1 -
 arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts | 1 -
 12 files changed, 2 insertions(+), 16 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 01/13] ARM: dts: porter: Always use status "okay" to enable devices

[Geert Uytterhoeven]

While status "ok" does work, the canonical form is "okay", so update the
few places that used the former.

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

diff --git a/arch/arm/boot/dts/r8a7791-porter.dts b/arch/arm/boot/dts/r8a7791-porter.dts
index 6761d11d3f9e1317..d9aa2cd6d6256776 100644
--- a/arch/arm/boot/dts/r8a7791-porter.dts
+++ b/arch/arm/boot/dts/r8a7791-porter.dts
@@ -226,7 +226,7 @@
 
 	phy-handle = <&phy1>;
 	renesas,ether-link-active-low;
-	status = "ok";
+	status = "okay";
 
 	phy1: ethernet-phy@1 {
 		reg = <1>;
@@ -359,7 +359,7 @@
 
 /* composite video input */
 &vin0 {
-	status = "ok";
+	status = "okay";
 	pinctrl-0 = <&vin0_pins>;
 	pinctrl-names = "default";
 
-- 
2.7.4

[PATCH 02/13] ARM: dts: bockw: Drop superfluous status update for frequency override

[Geert Uytterhoeven]

The scif_clk device node is already enabled in r8a7778.dtsi, so there is
no need to update its status again.

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

diff --git a/arch/arm/boot/dts/r8a7778-bockw.dts b/arch/arm/boot/dts/r8a7778-bockw.dts
index 211d239d9041a886..c79d55eb43c53fde 100644
--- a/arch/arm/boot/dts/r8a7778-bockw.dts
+++ b/arch/arm/boot/dts/r8a7778-bockw.dts
@@ -229,5 +229,4 @@
 
 &scif_clk {
 	clock-frequency = <14745600>;
-	status = "okay";
 };
-- 
2.7.4

[PATCH 03/13] ARM: dts: marzen: Drop superfluous status update for frequency override

[Geert Uytterhoeven]

The scif_clk device node is already enabled in r8a7779.dtsi, so there is
no need to update its status again.

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

diff --git a/arch/arm/boot/dts/r8a7779-marzen.dts b/arch/arm/boot/dts/r8a7779-marzen.dts
index 89c5b24a3d03f049..9412a86f9b300fa7 100644
--- a/arch/arm/boot/dts/r8a7779-marzen.dts
+++ b/arch/arm/boot/dts/r8a7779-marzen.dts
@@ -236,7 +236,6 @@
 
 &scif_clk {
 	clock-frequency = <14745600>;
-	status = "okay";
 };
 
 &sdhi0 {
-- 
2.7.4

[PATCH 04/13] ARM: dts: lager: Drop superfluous status update for frequency override

[Geert Uytterhoeven]

The scif_clk device node is already enabled in r8a7790.dtsi, so there is
no need to update its status again.

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

diff --git a/arch/arm/boot/dts/r8a7790-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts
index bd512c86e8522233..ba100a6f67ca554c 100644
--- a/arch/arm/boot/dts/r8a7790-lager.dts
+++ b/arch/arm/boot/dts/r8a7790-lager.dts
@@ -581,7 +581,6 @@
 
 &scif_clk {
 	clock-frequency = <14745600>;
-	status = "okay";
 };
 
 &msiof1 {
-- 
2.7.4

[PATCH 05/13] ARM: dts: koelsch: Drop superfluous status updates for frequency overrides

[Geert Uytterhoeven]

The scif_clk and pcie_bus_clk device nodes are already enabled in
r8a7791.dtsi, so there is no need to update their statuses again.

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

diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts b/arch/arm/boot/dts/r8a7791-koelsch.dts
index 5405d337d7448665..59beb8402a36007b 100644
--- a/arch/arm/boot/dts/r8a7791-koelsch.dts
+++ b/arch/arm/boot/dts/r8a7791-koelsch.dts
@@ -516,7 +516,6 @@
 
 &scif_clk {
 	clock-frequency = <14745600>;
-	status = "okay";
 };
 
 &sdhi0 {
@@ -767,7 +766,6 @@
 
 &pcie_bus_clk {
 	clock-frequency = <100000000>;
-	status = "okay";
 };
 
 &pciec {
-- 
2.7.4

[PATCH 06/13] ARM: dts: porter: Drop superfluous status update for frequency override

[Geert Uytterhoeven]

The pcie_bus_clk device node is already enabled in r8a7791.dtsi, so
there is no need to update its status again.

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

diff --git a/arch/arm/boot/dts/r8a7791-porter.dts b/arch/arm/boot/dts/r8a7791-porter.dts
index d9aa2cd6d6256776..95da5cb9d37ab8e5 100644
--- a/arch/arm/boot/dts/r8a7791-porter.dts
+++ b/arch/arm/boot/dts/r8a7791-porter.dts
@@ -401,7 +401,6 @@
 
 &pcie_bus_clk {
 	clock-frequency = <100000000>;
-	status = "okay";
 };
 
 &pciec {
-- 
2.7.4

[PATCH 07/13] ARM: dts: gose: Drop superfluous status update for frequency override

[Geert Uytterhoeven]

The scif_clk device node is already enabled in r8a7793.dtsi, so there is
no need to update its status again.

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

diff --git a/arch/arm/boot/dts/r8a7793-gose.dts b/arch/arm/boot/dts/r8a7793-gose.dts
index 92fff07c5e2bfe13..806c93f6ae8b9fda 100644
--- a/arch/arm/boot/dts/r8a7793-gose.dts
+++ b/arch/arm/boot/dts/r8a7793-gose.dts
@@ -412,7 +412,6 @@
 
 &scif_clk {
 	clock-frequency = <14745600>;
-	status = "okay";
 };
 
 &sdhi0 {
-- 
2.7.4

[PATCH 08/13] ARM: dts: alt: Drop superfluous status update for frequency override

[Geert Uytterhoeven]

The scif_clk device node is already enabled in r8a7794.dtsi, so there is
no need to update its status again.

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

diff --git a/arch/arm/boot/dts/r8a7794-alt.dts b/arch/arm/boot/dts/r8a7794-alt.dts
index 569e3f0e97a51804..3fcf76b8e923be86 100644
--- a/arch/arm/boot/dts/r8a7794-alt.dts
+++ b/arch/arm/boot/dts/r8a7794-alt.dts
@@ -375,7 +375,6 @@
 
 &scif_clk {
 	clock-frequency = <14745600>;
-	status = "okay";
 };
 
 &qspi {
-- 
2.7.4

[PATCH 09/13] ARM: dts: silk: Drop superfluous status update for frequency override

[Geert Uytterhoeven]

The scif_clk device node is already enabled in r8a7794.dtsi, so there is
no need to update its status again.

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

diff --git a/arch/arm/boot/dts/r8a7794-silk.dts b/arch/arm/boot/dts/r8a7794-silk.dts
index cf880ac06f4b795f..c742d80d6dca8548 100644
--- a/arch/arm/boot/dts/r8a7794-silk.dts
+++ b/arch/arm/boot/dts/r8a7794-silk.dts
@@ -248,7 +248,6 @@
 
 &scif_clk {
 	clock-frequency = <14745600>;
-	status = "okay";
 };
 
 &ether {
-- 
2.7.4

[PATCH 10/13] arm64: dts: h3ulcb: Drop superfluous status update for frequency override

[Geert Uytterhoeven]

The scif_clk device node is already enabled in r8a7795.dtsi, so there is
no need to update its status again.

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

diff --git a/arch/arm64/boot/dts/renesas/r8a7795-h3ulcb.dts b/arch/arm64/boot/dts/renesas/r8a7795-h3ulcb.dts
index 69c623faf80c1b2b..ab352159de6572c9 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795-h3ulcb.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7795-h3ulcb.dts
@@ -228,7 +228,6 @@
 
 &scif_clk {
 	clock-frequency = <14745600>;
-	status = "okay";
 };
 
 &i2c2 {
-- 
2.7.4

[PATCH 11/13] arm64: dts: r8a7795: salvator-x: Drop superfluous status updates for frequency overrides

[Geert Uytterhoeven]

The scif_clk and pcie_bus_clk device nodes are already enabled in
r8a7795.dtsi, so there is no need to update their statuses again.

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

diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
index d004eb7271578c83..3a44bd6b3455deb1 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
@@ -443,7 +443,6 @@
 
 &scif_clk {
 	clock-frequency = <14745600>;
-	status = "okay";
 };
 
 &i2c2 {
@@ -669,7 +668,6 @@
 
 &pcie_bus_clk {
 	clock-frequency = <100000000>;
-	status = "okay";
 };
 
 &pciec0 {
-- 
2.7.4

[PATCH 12/13] arm64: dts: m3ulcb: Drop superfluous status update for frequency override

[Geert Uytterhoeven]

The scif_clk device node is already enabled in r8a7796.dtsi, so there is
no need to update its status again.

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

diff --git a/arch/arm64/boot/dts/renesas/r8a7796-m3ulcb.dts b/arch/arm64/boot/dts/renesas/r8a7796-m3ulcb.dts
index c3f064ac2cb44971..372b2a9447163b24 100644
--- a/arch/arm64/boot/dts/renesas/r8a7796-m3ulcb.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7796-m3ulcb.dts
@@ -180,7 +180,6 @@
 
 &scif_clk {
 	clock-frequency = <14745600>;
-	status = "okay";
 };
 
 &wdt0 {
-- 
2.7.4

[PATCH 13/13] arm64: dts: r8a7796: salvator-x: Drop superfluous status update for frequency override

[Geert Uytterhoeven]

The scif_clk device node is already enabled in r8a7796.dtsi, so there is
no need to update its status again.

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

diff --git a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
index 74b8c653c9fed62a..c9f59b6ce33f693f 100644
--- a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
@@ -250,7 +250,6 @@
 
 &scif_clk {
 	clock-frequency = <14745600>;
-	status = "okay";
 };
 
 &i2c2 {
-- 
2.7.4

Re: [PATCH 00/13] arm: dts: renesas: Tidy-up status

[Simon Horman]

On Fri, Mar 10, 2017 at 02:19:03PM +0100, Geert Uytterhoeven wrote:
> 	Hi Simon, Magnus,
> 
> This patch series performs some cleanups on the Renesas DT source files:
>   - Patch 1 replaces "ok" by "okay" in statuses, as the latter is the
>     recommended form,
>   - Patches 2-13 drop superfluous status updates when overriding the
>     clock frequencies of fixed-frequency clocks, as they have already
>     been enabled in the SoC-specific DTS file.
> 
> Thanks!

Thanks, applied.