[PATCH v3 09/11] ARM: dts: STi: Update sound related nodes location

2018-02-13 Thread patrice.chotard 
From: Patrice Chotard 

Move sound node outside soc node and sti_sasg_codec node
as child of syscfg_core node.
This allows to fix the following warning when compiling
dtb with W=1 option:

arch/arm/boot/dts/stih407-b2120.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@0 has a unit name, but no reg 
property
arch/arm/boot/dts/stih407-b2120.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@1 has a unit name, but no reg 
property
arch/arm/boot/dts/stih407-b2120.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@2 has a unit name, but no reg 
property
arch/arm/boot/dts/stih410-b2120.dtb: Warning (simple_bus_reg):
Node /soc/sti-sasg-codec missing or empty reg/ranges property

arch/arm/boot/dts/stih410-b2120.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@0 has a unit name, but no reg 
property
arch/arm/boot/dts/stih410-b2120.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@1 has a unit name, but no reg 
property
arch/arm/boot/dts/stih410-b2120.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@2 has a unit name, but no reg 
property
arch/arm/boot/dts/stih407-b2120.dtb: Warning (simple_bus_reg):
Node /soc/sound missing or empty reg/ranges property
arch/arm/boot/dts/stih410-b2120.dtb: Warning (simple_bus_reg):
Node /soc/sti-sasg-codec missing or empty reg/ranges property

arch/arm/boot/dts/stih410-b2260.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@0 has a unit name, but no reg 
property
arch/arm/boot/dts/stih410-b2260.dtb: Warning (simple_bus_reg):
Node /soc/sti-sasg-codec missing or empty reg/ranges property
arch/arm/boot/dts/stih410-b2260.dtb: Warning (simple_bus_reg):
Node /soc/sound missing or empty reg/ranges property

arch/arm/boot/dts/stih418-b2199.dtb: Warning (simple_bus_reg):
Node /soc/sti-sasg-codec missing or empty reg/ranges property

Signed-off-by: Patrice Chotard 
---

v3: _ update location of sound related node and remove the fake reg property
v2: _ add a fake reg property to node without reg property.

 arch/arm/boot/dts/stih407-family.dtsi | 14 ++---
 arch/arm/boot/dts/stih410-b2260.dts   | 38 +++---
 arch/arm/boot/dts/stihxxx-b2120.dtsi  | 98 ++-
 3 files changed, 77 insertions(+), 73 deletions(-)

diff --git a/arch/arm/boot/dts/stih407-family.dtsi 
b/arch/arm/boot/dts/stih407-family.dtsi
index 5df827b00eb6..f7362c31de29 100644
--- a/arch/arm/boot/dts/stih407-family.dtsi
+++ b/arch/arm/boot/dts/stih407-family.dtsi
@@ -178,6 +178,13 @@
syscfg_core: core-syscfg@92b {
compatible = "st,stih407-core-syscfg", "syscon";
reg = <0x92b 0x1000>;
+
+   sti_sasg_codec: sti-sasg-codec {
+   compatible = "st,stih407-sas-codec";
+   #sound-dai-cells = <1>;
+   status = "disabled";
+   st,syscfg = <_core>;
+   };
};
 
syscfg_lpm: lpm-syscfg@94b5100 {
@@ -894,13 +901,6 @@
status = "disabled";
};
 
-   sti_sasg_codec: sti-sasg-codec {
-   compatible = "st,stih407-sas-codec";
-   #sound-dai-cells = <1>;
-   status = "disabled";
-   st,syscfg = <_core>;
-   };
-
sti_uni_player0: sti-uni-player@8d8 {
compatible = "st,stih407-uni-player-hdmi";
#sound-dai-cells = <0>;
diff --git a/arch/arm/boot/dts/stih410-b2260.dts 
b/arch/arm/boot/dts/stih410-b2260.dts
index 8bcd58118dba..5ccc6bb050df 100644
--- a/arch/arm/boot/dts/stih410-b2260.dts
+++ b/arch/arm/boot/dts/stih410-b2260.dts
@@ -57,6 +57,25 @@
};
};
 
+   sound: sound {
+   compatible = "simple-audio-card";
+   simple-audio-card,name = "STI-B2260";
+   status = "okay";
+
+   simple-audio-card,dai-link0 {
+   /* DAC */
+   format = "i2s";
+   mclk-fs = <128>;
+   cpu {
+   sound-dai = <_uni_player0>;
+   };
+
+   codec {
+   sound-dai = <_hdmi>;
+   };
+   };
+   };
+
soc {
/* Low speed expansion connector */
uart0: serial@983 {
@@ -181,25 +200,6 @@
status = "okay";
};
 
-   sound {
-   compatible = "simple-audio-card";
-   simple-audio-card,name = "STI-B2260";
-   status =

[PATCH v3 09/11] ARM: dts: STi: Update sound related nodes location

2018-02-13 Thread patrice.chotard 
From: Patrice Chotard 

Move sound node outside soc node and sti_sasg_codec node
as child of syscfg_core node.
This allows to fix the following warning when compiling
dtb with W=1 option:

arch/arm/boot/dts/stih407-b2120.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@0 has a unit name, but no reg 
property
arch/arm/boot/dts/stih407-b2120.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@1 has a unit name, but no reg 
property
arch/arm/boot/dts/stih407-b2120.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@2 has a unit name, but no reg 
property
arch/arm/boot/dts/stih410-b2120.dtb: Warning (simple_bus_reg):
Node /soc/sti-sasg-codec missing or empty reg/ranges property

arch/arm/boot/dts/stih410-b2120.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@0 has a unit name, but no reg 
property
arch/arm/boot/dts/stih410-b2120.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@1 has a unit name, but no reg 
property
arch/arm/boot/dts/stih410-b2120.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@2 has a unit name, but no reg 
property
arch/arm/boot/dts/stih407-b2120.dtb: Warning (simple_bus_reg):
Node /soc/sound missing or empty reg/ranges property
arch/arm/boot/dts/stih410-b2120.dtb: Warning (simple_bus_reg):
Node /soc/sti-sasg-codec missing or empty reg/ranges property

arch/arm/boot/dts/stih410-b2260.dtb: Warning (unit_address_vs_reg):
Node /soc/sound/simple-audio-card,dai-link@0 has a unit name, but no reg 
property
arch/arm/boot/dts/stih410-b2260.dtb: Warning (simple_bus_reg):
Node /soc/sti-sasg-codec missing or empty reg/ranges property
arch/arm/boot/dts/stih410-b2260.dtb: Warning (simple_bus_reg):
Node /soc/sound missing or empty reg/ranges property

arch/arm/boot/dts/stih418-b2199.dtb: Warning (simple_bus_reg):
Node /soc/sti-sasg-codec missing or empty reg/ranges property

Signed-off-by: Patrice Chotard 
---

v3: _ update location of sound related node and remove the fake reg property
v2: _ add a fake reg property to node without reg property.

 arch/arm/boot/dts/stih407-family.dtsi | 14 ++---
 arch/arm/boot/dts/stih410-b2260.dts   | 38 +++---
 arch/arm/boot/dts/stihxxx-b2120.dtsi  | 98 ++-
 3 files changed, 77 insertions(+), 73 deletions(-)

diff --git a/arch/arm/boot/dts/stih407-family.dtsi 
b/arch/arm/boot/dts/stih407-family.dtsi
index 5df827b00eb6..f7362c31de29 100644
--- a/arch/arm/boot/dts/stih407-family.dtsi
+++ b/arch/arm/boot/dts/stih407-family.dtsi
@@ -178,6 +178,13 @@
syscfg_core: core-syscfg@92b {
compatible = "st,stih407-core-syscfg", "syscon";
reg = <0x92b 0x1000>;
+
+   sti_sasg_codec: sti-sasg-codec {
+   compatible = "st,stih407-sas-codec";
+   #sound-dai-cells = <1>;
+   status = "disabled";
+   st,syscfg = <_core>;
+   };
};
 
syscfg_lpm: lpm-syscfg@94b5100 {
@@ -894,13 +901,6 @@
status = "disabled";
};
 
-   sti_sasg_codec: sti-sasg-codec {
-   compatible = "st,stih407-sas-codec";
-   #sound-dai-cells = <1>;
-   status = "disabled";
-   st,syscfg = <_core>;
-   };
-
sti_uni_player0: sti-uni-player@8d8 {
compatible = "st,stih407-uni-player-hdmi";
#sound-dai-cells = <0>;
diff --git a/arch/arm/boot/dts/stih410-b2260.dts 
b/arch/arm/boot/dts/stih410-b2260.dts
index 8bcd58118dba..5ccc6bb050df 100644
--- a/arch/arm/boot/dts/stih410-b2260.dts
+++ b/arch/arm/boot/dts/stih410-b2260.dts
@@ -57,6 +57,25 @@
};
};
 
+   sound: sound {
+   compatible = "simple-audio-card";
+   simple-audio-card,name = "STI-B2260";
+   status = "okay";
+
+   simple-audio-card,dai-link0 {
+   /* DAC */
+   format = "i2s";
+   mclk-fs = <128>;
+   cpu {
+   sound-dai = <_uni_player0>;
+   };
+
+   codec {
+   sound-dai = <_hdmi>;
+   };
+   };
+   };
+
soc {
/* Low speed expansion connector */
uart0: serial@983 {
@@ -181,25 +200,6 @@
status = "okay";
};
 
-   sound {
-   compatible = "simple-audio-card";
-   simple-audio-card,name = "STI-B2260";
-   status = "okay";
-
-