linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/9] ARM: dts: stm32: merge spi and i2s nodes
@ 2021-11-19 14:45 Olivier Moysan
  2021-11-19 14:45 ` [PATCH 1/9] ASoC: dt-bindings: stm32: i2s: update example Olivier Moysan
                   ` (8 more replies)
  0 siblings, 9 replies; 12+ messages in thread
From: Olivier Moysan @ 2021-11-19 14:45 UTC (permalink / raw)
  To: Alexandre Torgue, Maxime Coquelin, Rob Herring, arnaud.pouliquen,
	fabrice.gasnier
  Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32, Olivier Moysan

When a STM32 SPI instance offers I2S feature, two nodes are defined
in STM32 SoC device tree to support both SPI and I2S.
Merge SPI node and I2S nodes into a single node, to avoid
hardware description duplication and compilation warnings.

Olivier Moysan (9):
  ASoC: dt-bindings: stm32: i2s: update example
  ASoC: dt-bindings: stm32: i2s: add audio-graph-card port
  ASoC: dt-bindings: stm32: i2s: allow additional properties.
  ARM: dts: stm32: merge spi and i2s nodes
  ARM: dts: stm32: rename i2s node on stm32mp15xx-dkx boards
  ARM: dts: stm32: rename spi node on stm32mp15xx-dhcor-avenger96 boards
  ARM: dts: stm32: rename spi node on stm32mp15xx-dhcom-drc02 boards
  ARM: dts: stm32: rename spi node on stm32mp157c-ev1 board
  ARM: dts: stm32: adapt i2s node to spi binding on stm32mp15xx-dk

 .../bindings/sound/st,stm32-i2s.yaml          |  9 ++++-
 arch/arm/boot/dts/stm32mp151.dtsi             | 39 ++-----------------
 arch/arm/boot/dts/stm32mp157c-ev1.dts         |  2 +-
 .../arm/boot/dts/stm32mp15xx-dhcom-drc02.dtsi |  2 +-
 .../boot/dts/stm32mp15xx-dhcor-avenger96.dtsi |  2 +-
 arch/arm/boot/dts/stm32mp15xx-dkx.dtsi        |  7 +++-
 6 files changed, 18 insertions(+), 43 deletions(-)

-- 
2.17.1


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

* [PATCH 1/9] ASoC: dt-bindings: stm32: i2s: update example
  2021-11-19 14:45 [PATCH 0/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
@ 2021-11-19 14:45 ` Olivier Moysan
  2021-11-23 16:34   ` Rob Herring
  2021-11-19 14:45 ` [PATCH 2/9] ASoC: dt-bindings: stm32: i2s: add audio-graph-card port Olivier Moysan
                   ` (7 subsequent siblings)
  8 siblings, 1 reply; 12+ messages in thread
From: Olivier Moysan @ 2021-11-19 14:45 UTC (permalink / raw)
  To: Alexandre Torgue, Maxime Coquelin, Rob Herring, arnaud.pouliquen,
	fabrice.gasnier
  Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32, Olivier Moysan

Some STM32 SPI peripheral instances support I2S for audio.
SPI and I2S features were initially described through two separated
nodes in the SoC Device Tree. In the next SoC Device Trees
a single node is used to describe SPI peripheral, leading
to a change in node name for I2S.
Change example in STM32 DT binding example to match this change.

Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
---
 Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml b/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml
index 6feb5a09c184..ae41fa80bdb7 100644
--- a/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml
+++ b/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml
@@ -74,7 +74,7 @@ examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
     #include <dt-bindings/clock/stm32mp1-clks.h>
-    i2s2: audio-controller@4000b000 {
+    spi2s2: spi@4000b000 {
         compatible = "st,stm32h7-i2s";
         #sound-dai-cells = <0>;
         reg = <0x4000b000 0x400>;
-- 
2.17.1


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

* [PATCH 2/9] ASoC: dt-bindings: stm32: i2s: add audio-graph-card port
  2021-11-19 14:45 [PATCH 0/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
  2021-11-19 14:45 ` [PATCH 1/9] ASoC: dt-bindings: stm32: i2s: update example Olivier Moysan
@ 2021-11-19 14:45 ` Olivier Moysan
  2021-11-23 16:34   ` Rob Herring
  2021-11-19 14:45 ` [PATCH 3/9] ASoC: dt-bindings: stm32: i2s: allow additional properties Olivier Moysan
                   ` (6 subsequent siblings)
  8 siblings, 1 reply; 12+ messages in thread
From: Olivier Moysan @ 2021-11-19 14:45 UTC (permalink / raw)
  To: Alexandre Torgue, Maxime Coquelin, Rob Herring, arnaud.pouliquen,
	fabrice.gasnier
  Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32, Olivier Moysan

The STM2 I2S DAI can be connected via the audio-graph-card.
Add port entry into the bindings.

Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
---
 Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml b/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml
index ae41fa80bdb7..52e3368d34dc 100644
--- a/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml
+++ b/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml
@@ -58,6 +58,11 @@ properties:
     description: Configure the I2S device as MCLK clock provider.
     const: 0
 
+patternProperties:
+  '^port@[0-9]':
+    $ref: audio-graph-port.yaml#
+    unevaluatedProperties: false
+
 required:
   - compatible
   - "#sound-dai-cells"
-- 
2.17.1


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

* [PATCH 3/9] ASoC: dt-bindings: stm32: i2s: allow additional properties.
  2021-11-19 14:45 [PATCH 0/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
  2021-11-19 14:45 ` [PATCH 1/9] ASoC: dt-bindings: stm32: i2s: update example Olivier Moysan
  2021-11-19 14:45 ` [PATCH 2/9] ASoC: dt-bindings: stm32: i2s: add audio-graph-card port Olivier Moysan
@ 2021-11-19 14:45 ` Olivier Moysan
  2021-11-19 14:45 ` [PATCH 4/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
                   ` (5 subsequent siblings)
  8 siblings, 0 replies; 12+ messages in thread
From: Olivier Moysan @ 2021-11-19 14:45 UTC (permalink / raw)
  To: Alexandre Torgue, Maxime Coquelin, Rob Herring, arnaud.pouliquen,
	fabrice.gasnier
  Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32, Olivier Moysan

The STM32 SPI peripheral supports both SPI and I2S protocols.
In the SoC device tree the node describes the peripheral as an
SPI peripheral by default. This default configuration can be
overwritten in board device tree to use the IP as an I2S peripheral.
In this case the address-cells and size-cells properties from
SoC DT SPI node should not be checked against STM32 I2S bindings.
Set additionalProperties to "true" to allow these extra properties.

Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
---
 Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml b/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml
index 52e3368d34dc..591f9c941f54 100644
--- a/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml
+++ b/Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml
@@ -73,7 +73,7 @@ required:
   - dmas
   - dma-names
 
-additionalProperties: false
+additionalProperties: true
 
 examples:
   - |
-- 
2.17.1


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

* [PATCH 4/9] ARM: dts: stm32: merge spi and i2s nodes
  2021-11-19 14:45 [PATCH 0/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
                   ` (2 preceding siblings ...)
  2021-11-19 14:45 ` [PATCH 3/9] ASoC: dt-bindings: stm32: i2s: allow additional properties Olivier Moysan
@ 2021-11-19 14:45 ` Olivier Moysan
  2021-11-19 14:45 ` [PATCH 5/9] ARM: dts: stm32: rename i2s node on stm32mp15xx-dkx boards Olivier Moysan
                   ` (4 subsequent siblings)
  8 siblings, 0 replies; 12+ messages in thread
From: Olivier Moysan @ 2021-11-19 14:45 UTC (permalink / raw)
  To: Alexandre Torgue, Maxime Coquelin, Rob Herring, arnaud.pouliquen,
	fabrice.gasnier
  Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32, Olivier Moysan

When a SPI instance offers I2S feature, two nodes are defined
in SoC device tree to support both SPI and I2S.
Merge SPI node and I2S nodes into a single node, to avoid
hardware description duplication and compilation warnings.
spi2sx label is used to identify the SPI instances which
are supporting I2S feature.

Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
---
 arch/arm/boot/dts/stm32mp151.dtsi | 39 +++----------------------------
 1 file changed, 3 insertions(+), 36 deletions(-)

diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi
index bd289bf5d269..3649a4eb425f 100644
--- a/arch/arm/boot/dts/stm32mp151.dtsi
+++ b/arch/arm/boot/dts/stm32mp151.dtsi
@@ -386,7 +386,7 @@
 			};
 		};
 
-		spi2: spi@4000b000 {
+		spi2s2: spi@4000b000 {
 			#address-cells = <1>;
 			#size-cells = <0>;
 			compatible = "st,stm32h7-spi";
@@ -400,18 +400,7 @@
 			status = "disabled";
 		};
 
-		i2s2: audio-controller@4000b000 {
-			compatible = "st,stm32h7-i2s";
-			#sound-dai-cells = <0>;
-			reg = <0x4000b000 0x400>;
-			interrupts = <GIC_SPI 36 IRQ_TYPE_LEVEL_HIGH>;
-			dmas = <&dmamux1 39 0x400 0x01>,
-			       <&dmamux1 40 0x400 0x01>;
-			dma-names = "rx", "tx";
-			status = "disabled";
-		};
-
-		spi3: spi@4000c000 {
+		spi2s3: spi@4000c000 {
 			#address-cells = <1>;
 			#size-cells = <0>;
 			compatible = "st,stm32h7-spi";
@@ -425,17 +414,6 @@
 			status = "disabled";
 		};
 
-		i2s3: audio-controller@4000c000 {
-			compatible = "st,stm32h7-i2s";
-			#sound-dai-cells = <0>;
-			reg = <0x4000c000 0x400>;
-			interrupts = <GIC_SPI 51 IRQ_TYPE_LEVEL_HIGH>;
-			dmas = <&dmamux1 61 0x400 0x01>,
-			       <&dmamux1 62 0x400 0x01>;
-			dma-names = "rx", "tx";
-			status = "disabled";
-		};
-
 		spdifrx: audio-controller@4000d000 {
 			compatible = "st,stm32h7-spdifrx";
 			#sound-dai-cells = <0>;
@@ -681,7 +659,7 @@
 			status = "disabled";
 		};
 
-		spi1: spi@44004000 {
+		spi2s1: spi@44004000 {
 			#address-cells = <1>;
 			#size-cells = <0>;
 			compatible = "st,stm32h7-spi";
@@ -695,17 +673,6 @@
 			status = "disabled";
 		};
 
-		i2s1: audio-controller@44004000 {
-			compatible = "st,stm32h7-i2s";
-			#sound-dai-cells = <0>;
-			reg = <0x44004000 0x400>;
-			interrupts = <GIC_SPI 35 IRQ_TYPE_LEVEL_HIGH>;
-			dmas = <&dmamux1 37 0x400 0x01>,
-			       <&dmamux1 38 0x400 0x01>;
-			dma-names = "rx", "tx";
-			status = "disabled";
-		};
-
 		spi4: spi@44005000 {
 			#address-cells = <1>;
 			#size-cells = <0>;
-- 
2.17.1


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

* [PATCH 5/9] ARM: dts: stm32: rename i2s node on stm32mp15xx-dkx boards
  2021-11-19 14:45 [PATCH 0/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
                   ` (3 preceding siblings ...)
  2021-11-19 14:45 ` [PATCH 4/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
@ 2021-11-19 14:45 ` Olivier Moysan
  2021-11-19 14:45 ` [PATCH 6/9] ARM: dts: stm32: rename spi node on stm32mp15xx-dhcor-avenger96 boards Olivier Moysan
                   ` (3 subsequent siblings)
  8 siblings, 0 replies; 12+ messages in thread
From: Olivier Moysan @ 2021-11-19 14:45 UTC (permalink / raw)
  To: Alexandre Torgue, Maxime Coquelin, Rob Herring, arnaud.pouliquen,
	fabrice.gasnier
  Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32, Olivier Moysan

Rename i2s2 node on STMP32MP15xx-DKx boards, to match new
labels of SPI/I2S nodes in the SoC device tree.

Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
---
 arch/arm/boot/dts/stm32mp15xx-dkx.dtsi | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi b/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi
index 899bfe04aeb9..ff7dabfeb322 100644
--- a/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi
+++ b/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi
@@ -427,7 +427,9 @@
 	status = "disabled";
 };
 
-&i2s2 {
+&spi2s2 {
+	compatible = "st,stm32h7-i2s";
+	#sound-dai-cells = <0>;
 	clocks = <&rcc SPI2>, <&rcc SPI2_K>, <&rcc PLL3_Q>, <&rcc PLL3_R>;
 	clock-names = "pclk", "i2sclk", "x8k", "x11k";
 	pinctrl-names = "default", "sleep";
-- 
2.17.1


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

* [PATCH 6/9] ARM: dts: stm32: rename spi node on stm32mp15xx-dhcor-avenger96 boards
  2021-11-19 14:45 [PATCH 0/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
                   ` (4 preceding siblings ...)
  2021-11-19 14:45 ` [PATCH 5/9] ARM: dts: stm32: rename i2s node on stm32mp15xx-dkx boards Olivier Moysan
@ 2021-11-19 14:45 ` Olivier Moysan
  2021-11-19 14:45 ` [PATCH 7/9] ARM: dts: stm32: rename spi node on stm32mp15xx-dhcom-drc02 boards Olivier Moysan
                   ` (2 subsequent siblings)
  8 siblings, 0 replies; 12+ messages in thread
From: Olivier Moysan @ 2021-11-19 14:45 UTC (permalink / raw)
  To: Alexandre Torgue, Maxime Coquelin, Rob Herring, arnaud.pouliquen,
	fabrice.gasnier
  Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32, Olivier Moysan

Rename spi2 node on STMP32MP15xx-dhcor-avenger96 boards, to match new
labels of SPI/I2S nodes in the SoC device tree.

Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
---
 arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi b/arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi
index 6885948f3024..0dce9b118318 100644
--- a/arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi
+++ b/arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi
@@ -362,7 +362,7 @@
 	};
 };
 
-&spi2 {
+&spi2s2 {
 	pinctrl-names = "default";
 	pinctrl-0 = <&spi2_pins_a>;
 	cs-gpios = <&gpioi 0 0>;
-- 
2.17.1


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

* [PATCH 7/9] ARM: dts: stm32: rename spi node on stm32mp15xx-dhcom-drc02 boards
  2021-11-19 14:45 [PATCH 0/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
                   ` (5 preceding siblings ...)
  2021-11-19 14:45 ` [PATCH 6/9] ARM: dts: stm32: rename spi node on stm32mp15xx-dhcor-avenger96 boards Olivier Moysan
@ 2021-11-19 14:45 ` Olivier Moysan
  2021-11-19 14:45 ` [PATCH 8/9] ARM: dts: stm32: rename spi node on stm32mp157c-ev1 board Olivier Moysan
  2021-11-19 14:45 ` [PATCH 9/9] ARM: dts: stm32: adapt i2s node to spi binding on stm32mp15xx-dk Olivier Moysan
  8 siblings, 0 replies; 12+ messages in thread
From: Olivier Moysan @ 2021-11-19 14:45 UTC (permalink / raw)
  To: Alexandre Torgue, Maxime Coquelin, Rob Herring, arnaud.pouliquen,
	fabrice.gasnier
  Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32, Olivier Moysan

Rename spi1 node on STMP32MP15xx-dhcom-drc02 boards, to match new
labels of SPI/I2S nodes in the SoC device tree.

Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
---
 arch/arm/boot/dts/stm32mp15xx-dhcom-drc02.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcom-drc02.dtsi b/arch/arm/boot/dts/stm32mp15xx-dhcom-drc02.dtsi
index 4b10b013ffd5..29f18382d962 100644
--- a/arch/arm/boot/dts/stm32mp15xx-dhcom-drc02.dtsi
+++ b/arch/arm/boot/dts/stm32mp15xx-dhcom-drc02.dtsi
@@ -114,7 +114,7 @@
 	disable-wp;
 };
 
-&spi1 {
+&spi2s1 {
 	pinctrl-names = "default";
 	pinctrl-0 = <&spi1_pins_a>;
 	cs-gpios = <&gpioz 3 0>;
-- 
2.17.1


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

* [PATCH 8/9] ARM: dts: stm32: rename spi node on stm32mp157c-ev1 board
  2021-11-19 14:45 [PATCH 0/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
                   ` (6 preceding siblings ...)
  2021-11-19 14:45 ` [PATCH 7/9] ARM: dts: stm32: rename spi node on stm32mp15xx-dhcom-drc02 boards Olivier Moysan
@ 2021-11-19 14:45 ` Olivier Moysan
  2021-11-19 14:45 ` [PATCH 9/9] ARM: dts: stm32: adapt i2s node to spi binding on stm32mp15xx-dk Olivier Moysan
  8 siblings, 0 replies; 12+ messages in thread
From: Olivier Moysan @ 2021-11-19 14:45 UTC (permalink / raw)
  To: Alexandre Torgue, Maxime Coquelin, Rob Herring, arnaud.pouliquen,
	fabrice.gasnier
  Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32, Olivier Moysan

Rename spi1 node on STMP32MP157C-EV1 boards, to match new
labels of SPI/I2S nodes in the SoC device tree.

Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
---
 arch/arm/boot/dts/stm32mp157c-ev1.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/stm32mp157c-ev1.dts b/arch/arm/boot/dts/stm32mp157c-ev1.dts
index 5c5b1ddf7bfd..c836b4a1dbe2 100644
--- a/arch/arm/boot/dts/stm32mp157c-ev1.dts
+++ b/arch/arm/boot/dts/stm32mp157c-ev1.dts
@@ -293,7 +293,7 @@
 	status = "disabled";
 };
 
-&spi1 {
+&spi2s1 {
 	pinctrl-names = "default";
 	pinctrl-0 = <&spi1_pins_a>;
 	status = "disabled";
-- 
2.17.1


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

* [PATCH 9/9] ARM: dts: stm32: adapt i2s node to spi binding on stm32mp15xx-dk
  2021-11-19 14:45 [PATCH 0/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
                   ` (7 preceding siblings ...)
  2021-11-19 14:45 ` [PATCH 8/9] ARM: dts: stm32: rename spi node on stm32mp157c-ev1 board Olivier Moysan
@ 2021-11-19 14:45 ` Olivier Moysan
  8 siblings, 0 replies; 12+ messages in thread
From: Olivier Moysan @ 2021-11-19 14:45 UTC (permalink / raw)
  To: Alexandre Torgue, Maxime Coquelin, Rob Herring, arnaud.pouliquen,
	fabrice.gasnier
  Cc: devicetree, linux-arm-kernel, linux-kernel, linux-stm32, Olivier Moysan

In DT check utility, the spi2s2 node is identified as an spi node.
The check_spi_bus_reg() function issues a warning "missing or empty
reg property" if reg property is not defined in child nodes.
Add reg property to STM32 I2S port node on STM32MP15XX-DK board
to match this requirement and add related unit-address in node name.

Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
---
 arch/arm/boot/dts/stm32mp15xx-dkx.dtsi | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi b/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi
index ff7dabfeb322..36187089c073 100644
--- a/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi
+++ b/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi
@@ -437,7 +437,8 @@
 	pinctrl-1 = <&i2s2_sleep_pins_a>;
 	status = "okay";
 
-	i2s2_port: port {
+	i2s2_port: port@0 {
+		reg = <0>;
 		i2s2_endpoint: endpoint {
 			remote-endpoint = <&sii9022_tx_endpoint>;
 			format = "i2s";
-- 
2.17.1


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

* Re: [PATCH 1/9] ASoC: dt-bindings: stm32: i2s: update example
  2021-11-19 14:45 ` [PATCH 1/9] ASoC: dt-bindings: stm32: i2s: update example Olivier Moysan
@ 2021-11-23 16:34   ` Rob Herring
  0 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2021-11-23 16:34 UTC (permalink / raw)
  To: Olivier Moysan
  Cc: linux-kernel, linux-arm-kernel, linux-stm32, fabrice.gasnier,
	devicetree, arnaud.pouliquen, Rob Herring, Alexandre Torgue,
	Maxime Coquelin

On Fri, 19 Nov 2021 15:45:43 +0100, Olivier Moysan wrote:
> Some STM32 SPI peripheral instances support I2S for audio.
> SPI and I2S features were initially described through two separated
> nodes in the SoC Device Tree. In the next SoC Device Trees
> a single node is used to describe SPI peripheral, leading
> to a change in node name for I2S.
> Change example in STM32 DT binding example to match this change.
> 
> Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
> ---
>  Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.

Full log is available here: https://patchwork.ozlabs.org/patch/1557188


audio-controller@4000b000: 'port' does not match any of the regexes: 'pinctrl-[0-9]+'
	arch/arm/boot/dts/stm32mp157a-dk1.dt.yaml
	arch/arm/boot/dts/stm32mp157c-dk2.dt.yaml


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

* Re: [PATCH 2/9] ASoC: dt-bindings: stm32: i2s: add audio-graph-card port
  2021-11-19 14:45 ` [PATCH 2/9] ASoC: dt-bindings: stm32: i2s: add audio-graph-card port Olivier Moysan
@ 2021-11-23 16:34   ` Rob Herring
  0 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2021-11-23 16:34 UTC (permalink / raw)
  To: Olivier Moysan
  Cc: fabrice.gasnier, linux-arm-kernel, devicetree, linux-kernel,
	Alexandre Torgue, linux-stm32, arnaud.pouliquen, Maxime Coquelin,
	Rob Herring

On Fri, 19 Nov 2021 15:45:44 +0100, Olivier Moysan wrote:
> The STM2 I2S DAI can be connected via the audio-graph-card.
> Add port entry into the bindings.
> 
> Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
> ---
>  Documentation/devicetree/bindings/sound/st,stm32-i2s.yaml | 5 +++++
>  1 file changed, 5 insertions(+)
> 

Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.

Full log is available here: https://patchwork.ozlabs.org/patch/1557189


audio-controller@4000b000: 'port' does not match any of the regexes: '^port@[0-9]', 'pinctrl-[0-9]+'
	arch/arm/boot/dts/stm32mp157a-dk1.dt.yaml
	arch/arm/boot/dts/stm32mp157c-dk2.dt.yaml


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

end of thread, other threads:[~2021-11-23 16:36 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-19 14:45 [PATCH 0/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
2021-11-19 14:45 ` [PATCH 1/9] ASoC: dt-bindings: stm32: i2s: update example Olivier Moysan
2021-11-23 16:34   ` Rob Herring
2021-11-19 14:45 ` [PATCH 2/9] ASoC: dt-bindings: stm32: i2s: add audio-graph-card port Olivier Moysan
2021-11-23 16:34   ` Rob Herring
2021-11-19 14:45 ` [PATCH 3/9] ASoC: dt-bindings: stm32: i2s: allow additional properties Olivier Moysan
2021-11-19 14:45 ` [PATCH 4/9] ARM: dts: stm32: merge spi and i2s nodes Olivier Moysan
2021-11-19 14:45 ` [PATCH 5/9] ARM: dts: stm32: rename i2s node on stm32mp15xx-dkx boards Olivier Moysan
2021-11-19 14:45 ` [PATCH 6/9] ARM: dts: stm32: rename spi node on stm32mp15xx-dhcor-avenger96 boards Olivier Moysan
2021-11-19 14:45 ` [PATCH 7/9] ARM: dts: stm32: rename spi node on stm32mp15xx-dhcom-drc02 boards Olivier Moysan
2021-11-19 14:45 ` [PATCH 8/9] ARM: dts: stm32: rename spi node on stm32mp157c-ev1 board Olivier Moysan
2021-11-19 14:45 ` [PATCH 9/9] ARM: dts: stm32: adapt i2s node to spi binding on stm32mp15xx-dk Olivier Moysan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).