* [PATCH 1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding
@ 2023-01-05 14:41 ` Marek Vasut
0 siblings, 0 replies; 21+ messages in thread
From: Marek Vasut @ 2023-01-05 14:41 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Marek Vasut, Adam Ford, Alexander Stein, Fabio Estevam,
Joakim Zhang, Krzysztof Kozlowski, Laurent Pinchart,
Liam Girdwood, Lucas Stach, Mark Brown, Markus Niebel,
NXP Linux Team, Paul Elder, Peng Fan, Pengutronix Kernel Team,
Richard Cochran, Richard Zhu, Rob Herring, Sascha Hauer,
Shawn Guo, Shengjiu Wang, Stefan Agner, Tim Harvey, alsa-devel,
devicetree
Get rid of the vf610 sai special case, instead update the vfxxx.dtsi
DT to use the same DMA channel ordering as all the other devices. The
sai DMA channel ordering has not been aligned with other IP DMA channel
ordering in the vfxxx.dtsi anyway.
Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Adam Ford <aford173@gmail.com>
Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: Joakim Zhang <qiangqing.zhang@nxp.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Lucas Stach <l.stach@pengutronix.de>
Cc: Marek Vasut <marex@denx.de>
Cc: Mark Brown <broonie@kernel.org>
Cc: Markus Niebel <Markus.Niebel@ew.tq-group.com>
Cc: NXP Linux Team <linux-imx@nxp.com>
Cc: Paul Elder <paul.elder@ideasonboard.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
Cc: Richard Cochran <richardcochran@gmail.com>
Cc: Richard Zhu <hongxing.zhu@nxp.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Shengjiu Wang <shengjiu.wang@nxp.com>
Cc: Stefan Agner <stefan@agner.ch>
Cc: Tim Harvey <tharvey@gateworks.com>
Cc: alsa-devel@alsa-project.org
Cc: devicetree@vger.kernel.org
To: linux-arm-kernel@lists.infradead.org
---
.../devicetree/bindings/sound/fsl,sai.yaml | 38 ++++---------------
1 file changed, 8 insertions(+), 30 deletions(-)
diff --git a/Documentation/devicetree/bindings/sound/fsl,sai.yaml b/Documentation/devicetree/bindings/sound/fsl,sai.yaml
index 7e56337d8edc1..088c26b001cc0 100644
--- a/Documentation/devicetree/bindings/sound/fsl,sai.yaml
+++ b/Documentation/devicetree/bindings/sound/fsl,sai.yaml
@@ -76,10 +76,14 @@ properties:
minItems: 4
dmas:
- maxItems: 2
+ items:
+ - description: DMA controller phandle and request line for RX
+ - description: DMA controller phandle and request line for TX
dma-names:
- maxItems: 2
+ items:
+ - const: rx
+ - const: tx
interrupts:
items:
@@ -142,31 +146,6 @@ properties:
allOf:
- $ref: dai-common.yaml#
- - if:
- properties:
- compatible:
- contains:
- const: fsl,vf610-sai
- then:
- properties:
- dmas:
- items:
- - description: DMA controller phandle and request line for TX
- - description: DMA controller phandle and request line for RX
- dma-names:
- items:
- - const: tx
- - const: rx
- else:
- properties:
- dmas:
- items:
- - description: DMA controller phandle and request line for RX
- - description: DMA controller phandle and request line for TX
- dma-names:
- items:
- - const: rx
- - const: tx
- if:
required:
- fsl,sai-asynchronous
@@ -199,9 +178,8 @@ examples:
<&clks VF610_CLK_SAI2>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 0 21>,
- <&edma0 0 20>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 0 20>, <&edma0 0 21>;
big-endian;
lsb-first;
};
--
2.39.0
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding
@ 2023-01-05 14:41 ` Marek Vasut
0 siblings, 0 replies; 21+ messages in thread
From: Marek Vasut @ 2023-01-05 14:41 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Peng Fan, Alexander Stein, alsa-devel, Stefan Agner,
Liam Girdwood, Laurent Pinchart, Krzysztof Kozlowski,
Fabio Estevam, Marek Vasut, Markus Niebel, Paul Elder,
NXP Linux Team, Tim Harvey, Sascha Hauer, devicetree,
Richard Zhu, Richard Cochran, Rob Herring, Adam Ford,
Shengjiu Wang, Joakim Zhang, Mark Brown, Pengutronix Kernel Team,
Shawn Guo, Lucas Stach
Get rid of the vf610 sai special case, instead update the vfxxx.dtsi
DT to use the same DMA channel ordering as all the other devices. The
sai DMA channel ordering has not been aligned with other IP DMA channel
ordering in the vfxxx.dtsi anyway.
Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Adam Ford <aford173@gmail.com>
Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: Joakim Zhang <qiangqing.zhang@nxp.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Lucas Stach <l.stach@pengutronix.de>
Cc: Marek Vasut <marex@denx.de>
Cc: Mark Brown <broonie@kernel.org>
Cc: Markus Niebel <Markus.Niebel@ew.tq-group.com>
Cc: NXP Linux Team <linux-imx@nxp.com>
Cc: Paul Elder <paul.elder@ideasonboard.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
Cc: Richard Cochran <richardcochran@gmail.com>
Cc: Richard Zhu <hongxing.zhu@nxp.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Shengjiu Wang <shengjiu.wang@nxp.com>
Cc: Stefan Agner <stefan@agner.ch>
Cc: Tim Harvey <tharvey@gateworks.com>
Cc: alsa-devel@alsa-project.org
Cc: devicetree@vger.kernel.org
To: linux-arm-kernel@lists.infradead.org
---
.../devicetree/bindings/sound/fsl,sai.yaml | 38 ++++---------------
1 file changed, 8 insertions(+), 30 deletions(-)
diff --git a/Documentation/devicetree/bindings/sound/fsl,sai.yaml b/Documentation/devicetree/bindings/sound/fsl,sai.yaml
index 7e56337d8edc1..088c26b001cc0 100644
--- a/Documentation/devicetree/bindings/sound/fsl,sai.yaml
+++ b/Documentation/devicetree/bindings/sound/fsl,sai.yaml
@@ -76,10 +76,14 @@ properties:
minItems: 4
dmas:
- maxItems: 2
+ items:
+ - description: DMA controller phandle and request line for RX
+ - description: DMA controller phandle and request line for TX
dma-names:
- maxItems: 2
+ items:
+ - const: rx
+ - const: tx
interrupts:
items:
@@ -142,31 +146,6 @@ properties:
allOf:
- $ref: dai-common.yaml#
- - if:
- properties:
- compatible:
- contains:
- const: fsl,vf610-sai
- then:
- properties:
- dmas:
- items:
- - description: DMA controller phandle and request line for TX
- - description: DMA controller phandle and request line for RX
- dma-names:
- items:
- - const: tx
- - const: rx
- else:
- properties:
- dmas:
- items:
- - description: DMA controller phandle and request line for RX
- - description: DMA controller phandle and request line for TX
- dma-names:
- items:
- - const: rx
- - const: tx
- if:
required:
- fsl,sai-asynchronous
@@ -199,9 +178,8 @@ examples:
<&clks VF610_CLK_SAI2>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 0 21>,
- <&edma0 0 20>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 0 20>, <&edma0 0 21>;
big-endian;
lsb-first;
};
--
2.39.0
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding
@ 2023-01-05 14:41 ` Marek Vasut
0 siblings, 0 replies; 21+ messages in thread
From: Marek Vasut @ 2023-01-05 14:41 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Marek Vasut, Adam Ford, Alexander Stein, Fabio Estevam,
Joakim Zhang, Krzysztof Kozlowski, Laurent Pinchart,
Liam Girdwood, Lucas Stach, Mark Brown, Markus Niebel,
NXP Linux Team, Paul Elder, Peng Fan, Pengutronix Kernel Team,
Richard Cochran, Richard Zhu, Rob Herring, Sascha Hauer,
Shawn Guo, Shengjiu Wang, Stefan Agner, Tim Harvey, alsa-devel,
devicetree
Get rid of the vf610 sai special case, instead update the vfxxx.dtsi
DT to use the same DMA channel ordering as all the other devices. The
sai DMA channel ordering has not been aligned with other IP DMA channel
ordering in the vfxxx.dtsi anyway.
Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Adam Ford <aford173@gmail.com>
Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: Joakim Zhang <qiangqing.zhang@nxp.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Lucas Stach <l.stach@pengutronix.de>
Cc: Marek Vasut <marex@denx.de>
Cc: Mark Brown <broonie@kernel.org>
Cc: Markus Niebel <Markus.Niebel@ew.tq-group.com>
Cc: NXP Linux Team <linux-imx@nxp.com>
Cc: Paul Elder <paul.elder@ideasonboard.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
Cc: Richard Cochran <richardcochran@gmail.com>
Cc: Richard Zhu <hongxing.zhu@nxp.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Shengjiu Wang <shengjiu.wang@nxp.com>
Cc: Stefan Agner <stefan@agner.ch>
Cc: Tim Harvey <tharvey@gateworks.com>
Cc: alsa-devel@alsa-project.org
Cc: devicetree@vger.kernel.org
To: linux-arm-kernel@lists.infradead.org
---
.../devicetree/bindings/sound/fsl,sai.yaml | 38 ++++---------------
1 file changed, 8 insertions(+), 30 deletions(-)
diff --git a/Documentation/devicetree/bindings/sound/fsl,sai.yaml b/Documentation/devicetree/bindings/sound/fsl,sai.yaml
index 7e56337d8edc1..088c26b001cc0 100644
--- a/Documentation/devicetree/bindings/sound/fsl,sai.yaml
+++ b/Documentation/devicetree/bindings/sound/fsl,sai.yaml
@@ -76,10 +76,14 @@ properties:
minItems: 4
dmas:
- maxItems: 2
+ items:
+ - description: DMA controller phandle and request line for RX
+ - description: DMA controller phandle and request line for TX
dma-names:
- maxItems: 2
+ items:
+ - const: rx
+ - const: tx
interrupts:
items:
@@ -142,31 +146,6 @@ properties:
allOf:
- $ref: dai-common.yaml#
- - if:
- properties:
- compatible:
- contains:
- const: fsl,vf610-sai
- then:
- properties:
- dmas:
- items:
- - description: DMA controller phandle and request line for TX
- - description: DMA controller phandle and request line for RX
- dma-names:
- items:
- - const: tx
- - const: rx
- else:
- properties:
- dmas:
- items:
- - description: DMA controller phandle and request line for RX
- - description: DMA controller phandle and request line for TX
- dma-names:
- items:
- - const: rx
- - const: tx
- if:
required:
- fsl,sai-asynchronous
@@ -199,9 +178,8 @@ examples:
<&clks VF610_CLK_SAI2>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 0 21>,
- <&edma0 0 20>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 0 20>, <&edma0 0 21>;
big-endian;
lsb-first;
};
--
2.39.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order
2023-01-05 14:41 ` Marek Vasut
(?)
@ 2023-01-05 14:41 ` Marek Vasut
-1 siblings, 0 replies; 21+ messages in thread
From: Marek Vasut @ 2023-01-05 14:41 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Marek Vasut, Adam Ford, Alexander Stein, Fabio Estevam,
Joakim Zhang, Krzysztof Kozlowski, Laurent Pinchart,
Liam Girdwood, Lucas Stach, Mark Brown, Markus Niebel,
NXP Linux Team, Paul Elder, Peng Fan, Pengutronix Kernel Team,
Richard Cochran, Richard Zhu, Rob Herring, Sascha Hauer,
Shawn Guo, Shengjiu Wang, Stefan Agner, Tim Harvey, alsa-devel,
devicetree
Swap the SAI dmas and dma-names to match the order documented in
Documentation/devicetree/bindings/sound/fsl,sai.yaml
No functional change.
Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Adam Ford <aford173@gmail.com>
Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: Joakim Zhang <qiangqing.zhang@nxp.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Lucas Stach <l.stach@pengutronix.de>
Cc: Marek Vasut <marex@denx.de>
Cc: Mark Brown <broonie@kernel.org>
Cc: Markus Niebel <Markus.Niebel@ew.tq-group.com>
Cc: NXP Linux Team <linux-imx@nxp.com>
Cc: Paul Elder <paul.elder@ideasonboard.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
Cc: Richard Cochran <richardcochran@gmail.com>
Cc: Richard Zhu <hongxing.zhu@nxp.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Shengjiu Wang <shengjiu.wang@nxp.com>
Cc: Stefan Agner <stefan@agner.ch>
Cc: Tim Harvey <tharvey@gateworks.com>
Cc: alsa-devel@alsa-project.org
Cc: devicetree@vger.kernel.org
To: linux-arm-kernel@lists.infradead.org
---
arch/arm/boot/dts/vfxxx.dtsi | 20 ++++++++------------
1 file changed, 8 insertions(+), 12 deletions(-)
diff --git a/arch/arm/boot/dts/vfxxx.dtsi b/arch/arm/boot/dts/vfxxx.dtsi
index d53f9c9db8bfd..ff4479994b600 100644
--- a/arch/arm/boot/dts/vfxxx.dtsi
+++ b/arch/arm/boot/dts/vfxxx.dtsi
@@ -191,9 +191,8 @@ sai0: sai@4002f000 {
<&clks VF610_CLK_SAI0_DIV>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 0 17>,
- <&edma0 0 16>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 0 16>, <&edma0 0 17>;
status = "disabled";
};
@@ -205,9 +204,8 @@ sai1: sai@40030000 {
<&clks VF610_CLK_SAI1_DIV>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 0 19>,
- <&edma0 0 18>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 0 18>, <&edma0 0 19>;
status = "disabled";
};
@@ -219,9 +217,8 @@ sai2: sai@40031000 {
<&clks VF610_CLK_SAI2_DIV>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 0 21>,
- <&edma0 0 20>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 0 20>, <&edma0 0 21>;
status = "disabled";
};
@@ -233,9 +230,8 @@ sai3: sai@40032000 {
<&clks VF610_CLK_SAI3_DIV>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 1 9>,
- <&edma0 1 8>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 1 8>, <&edma0 1 9>;
status = "disabled";
};
--
2.39.0
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order
@ 2023-01-05 14:41 ` Marek Vasut
0 siblings, 0 replies; 21+ messages in thread
From: Marek Vasut @ 2023-01-05 14:41 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Peng Fan, Alexander Stein, alsa-devel, Stefan Agner,
Liam Girdwood, Laurent Pinchart, Krzysztof Kozlowski,
Fabio Estevam, Marek Vasut, Markus Niebel, Paul Elder,
NXP Linux Team, Tim Harvey, Sascha Hauer, devicetree,
Richard Zhu, Richard Cochran, Rob Herring, Adam Ford,
Shengjiu Wang, Joakim Zhang, Mark Brown, Pengutronix Kernel Team,
Shawn Guo, Lucas Stach
Swap the SAI dmas and dma-names to match the order documented in
Documentation/devicetree/bindings/sound/fsl,sai.yaml
No functional change.
Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Adam Ford <aford173@gmail.com>
Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: Joakim Zhang <qiangqing.zhang@nxp.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Lucas Stach <l.stach@pengutronix.de>
Cc: Marek Vasut <marex@denx.de>
Cc: Mark Brown <broonie@kernel.org>
Cc: Markus Niebel <Markus.Niebel@ew.tq-group.com>
Cc: NXP Linux Team <linux-imx@nxp.com>
Cc: Paul Elder <paul.elder@ideasonboard.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
Cc: Richard Cochran <richardcochran@gmail.com>
Cc: Richard Zhu <hongxing.zhu@nxp.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Shengjiu Wang <shengjiu.wang@nxp.com>
Cc: Stefan Agner <stefan@agner.ch>
Cc: Tim Harvey <tharvey@gateworks.com>
Cc: alsa-devel@alsa-project.org
Cc: devicetree@vger.kernel.org
To: linux-arm-kernel@lists.infradead.org
---
arch/arm/boot/dts/vfxxx.dtsi | 20 ++++++++------------
1 file changed, 8 insertions(+), 12 deletions(-)
diff --git a/arch/arm/boot/dts/vfxxx.dtsi b/arch/arm/boot/dts/vfxxx.dtsi
index d53f9c9db8bfd..ff4479994b600 100644
--- a/arch/arm/boot/dts/vfxxx.dtsi
+++ b/arch/arm/boot/dts/vfxxx.dtsi
@@ -191,9 +191,8 @@ sai0: sai@4002f000 {
<&clks VF610_CLK_SAI0_DIV>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 0 17>,
- <&edma0 0 16>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 0 16>, <&edma0 0 17>;
status = "disabled";
};
@@ -205,9 +204,8 @@ sai1: sai@40030000 {
<&clks VF610_CLK_SAI1_DIV>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 0 19>,
- <&edma0 0 18>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 0 18>, <&edma0 0 19>;
status = "disabled";
};
@@ -219,9 +217,8 @@ sai2: sai@40031000 {
<&clks VF610_CLK_SAI2_DIV>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 0 21>,
- <&edma0 0 20>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 0 20>, <&edma0 0 21>;
status = "disabled";
};
@@ -233,9 +230,8 @@ sai3: sai@40032000 {
<&clks VF610_CLK_SAI3_DIV>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 1 9>,
- <&edma0 1 8>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 1 8>, <&edma0 1 9>;
status = "disabled";
};
--
2.39.0
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order
@ 2023-01-05 14:41 ` Marek Vasut
0 siblings, 0 replies; 21+ messages in thread
From: Marek Vasut @ 2023-01-05 14:41 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Marek Vasut, Adam Ford, Alexander Stein, Fabio Estevam,
Joakim Zhang, Krzysztof Kozlowski, Laurent Pinchart,
Liam Girdwood, Lucas Stach, Mark Brown, Markus Niebel,
NXP Linux Team, Paul Elder, Peng Fan, Pengutronix Kernel Team,
Richard Cochran, Richard Zhu, Rob Herring, Sascha Hauer,
Shawn Guo, Shengjiu Wang, Stefan Agner, Tim Harvey, alsa-devel,
devicetree
Swap the SAI dmas and dma-names to match the order documented in
Documentation/devicetree/bindings/sound/fsl,sai.yaml
No functional change.
Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Adam Ford <aford173@gmail.com>
Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: Joakim Zhang <qiangqing.zhang@nxp.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Lucas Stach <l.stach@pengutronix.de>
Cc: Marek Vasut <marex@denx.de>
Cc: Mark Brown <broonie@kernel.org>
Cc: Markus Niebel <Markus.Niebel@ew.tq-group.com>
Cc: NXP Linux Team <linux-imx@nxp.com>
Cc: Paul Elder <paul.elder@ideasonboard.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
Cc: Richard Cochran <richardcochran@gmail.com>
Cc: Richard Zhu <hongxing.zhu@nxp.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Shengjiu Wang <shengjiu.wang@nxp.com>
Cc: Stefan Agner <stefan@agner.ch>
Cc: Tim Harvey <tharvey@gateworks.com>
Cc: alsa-devel@alsa-project.org
Cc: devicetree@vger.kernel.org
To: linux-arm-kernel@lists.infradead.org
---
arch/arm/boot/dts/vfxxx.dtsi | 20 ++++++++------------
1 file changed, 8 insertions(+), 12 deletions(-)
diff --git a/arch/arm/boot/dts/vfxxx.dtsi b/arch/arm/boot/dts/vfxxx.dtsi
index d53f9c9db8bfd..ff4479994b600 100644
--- a/arch/arm/boot/dts/vfxxx.dtsi
+++ b/arch/arm/boot/dts/vfxxx.dtsi
@@ -191,9 +191,8 @@ sai0: sai@4002f000 {
<&clks VF610_CLK_SAI0_DIV>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 0 17>,
- <&edma0 0 16>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 0 16>, <&edma0 0 17>;
status = "disabled";
};
@@ -205,9 +204,8 @@ sai1: sai@40030000 {
<&clks VF610_CLK_SAI1_DIV>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 0 19>,
- <&edma0 0 18>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 0 18>, <&edma0 0 19>;
status = "disabled";
};
@@ -219,9 +217,8 @@ sai2: sai@40031000 {
<&clks VF610_CLK_SAI2_DIV>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 0 21>,
- <&edma0 0 20>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 0 20>, <&edma0 0 21>;
status = "disabled";
};
@@ -233,9 +230,8 @@ sai3: sai@40032000 {
<&clks VF610_CLK_SAI3_DIV>,
<&clks 0>, <&clks 0>;
clock-names = "bus", "mclk1", "mclk2", "mclk3";
- dma-names = "tx", "rx";
- dmas = <&edma0 1 9>,
- <&edma0 1 8>;
+ dma-names = "rx", "tx";
+ dmas = <&edma0 1 8>, <&edma0 1 9>;
status = "disabled";
};
--
2.39.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH 1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding
2023-01-05 14:41 ` Marek Vasut
(?)
@ 2023-01-06 9:16 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-06 9:16 UTC (permalink / raw)
To: Marek Vasut, linux-arm-kernel
Cc: Adam Ford, Alexander Stein, Fabio Estevam, Joakim Zhang,
Krzysztof Kozlowski, Laurent Pinchart, Liam Girdwood,
Lucas Stach, Mark Brown, Markus Niebel, NXP Linux Team,
Paul Elder, Peng Fan, Pengutronix Kernel Team, Richard Cochran,
Richard Zhu, Rob Herring, Sascha Hauer, Shawn Guo, Shengjiu Wang,
Stefan Agner, Tim Harvey, alsa-devel, devicetree
On 05/01/2023 15:41, Marek Vasut wrote:
> Get rid of the vf610 sai special case, instead update the vfxxx.dtsi
> DT to use the same DMA channel ordering as all the other devices. The
> sai DMA channel ordering has not been aligned with other IP DMA channel
> ordering in the vfxxx.dtsi anyway.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
Assuming that Linux implementation does not care about the order of
these (and takes by name):
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Although other users might not be happy about change of order...
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding
@ 2023-01-06 9:16 ` Krzysztof Kozlowski
0 siblings, 0 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-06 9:16 UTC (permalink / raw)
To: Marek Vasut, linux-arm-kernel
Cc: Adam Ford, Alexander Stein, Fabio Estevam, Joakim Zhang,
Krzysztof Kozlowski, Laurent Pinchart, Liam Girdwood,
Lucas Stach, Mark Brown, Markus Niebel, NXP Linux Team,
Paul Elder, Peng Fan, Pengutronix Kernel Team, Richard Cochran,
Richard Zhu, Rob Herring, Sascha Hauer, Shawn Guo, Shengjiu Wang,
Stefan Agner, Tim Harvey, alsa-devel, devicetree
On 05/01/2023 15:41, Marek Vasut wrote:
> Get rid of the vf610 sai special case, instead update the vfxxx.dtsi
> DT to use the same DMA channel ordering as all the other devices. The
> sai DMA channel ordering has not been aligned with other IP DMA channel
> ordering in the vfxxx.dtsi anyway.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
Assuming that Linux implementation does not care about the order of
these (and takes by name):
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Although other users might not be happy about change of order...
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding
@ 2023-01-06 9:16 ` Krzysztof Kozlowski
0 siblings, 0 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-06 9:16 UTC (permalink / raw)
To: Marek Vasut, linux-arm-kernel
Cc: Peng Fan, Sascha Hauer, Alexander Stein, alsa-devel,
Stefan Agner, Liam Girdwood, Laurent Pinchart,
Krzysztof Kozlowski, Fabio Estevam, Markus Niebel, Paul Elder,
NXP Linux Team, Tim Harvey, devicetree, Richard Zhu,
Richard Cochran, Rob Herring, Adam Ford, Shengjiu Wang,
Joakim Zhang, Mark Brown, Pengutronix Kernel Team, Shawn Guo,
Lucas Stach
On 05/01/2023 15:41, Marek Vasut wrote:
> Get rid of the vf610 sai special case, instead update the vfxxx.dtsi
> DT to use the same DMA channel ordering as all the other devices. The
> sai DMA channel ordering has not been aligned with other IP DMA channel
> ordering in the vfxxx.dtsi anyway.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
Assuming that Linux implementation does not care about the order of
these (and takes by name):
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Although other users might not be happy about change of order...
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order
2023-01-05 14:41 ` Marek Vasut
(?)
@ 2023-01-06 9:18 ` Lucas Stach
-1 siblings, 0 replies; 21+ messages in thread
From: Lucas Stach @ 2023-01-06 9:18 UTC (permalink / raw)
To: Marek Vasut, linux-arm-kernel, Krzysztof Kozlowski, Rob Herring
Cc: Adam Ford, Alexander Stein, Fabio Estevam, Joakim Zhang,
Laurent Pinchart, Liam Girdwood, Mark Brown, Markus Niebel,
NXP Linux Team, Paul Elder, Peng Fan, Pengutronix Kernel Team,
Richard Cochran, Richard Zhu, Sascha Hauer, Shawn Guo,
Shengjiu Wang, Stefan Agner, Tim Harvey, alsa-devel, devicetree
Am Donnerstag, dem 05.01.2023 um 15:41 +0100 schrieb Marek Vasut:
> Swap the SAI dmas and dma-names to match the order documented in
> Documentation/devicetree/bindings/sound/fsl,sai.yaml
> No functional change.
>
While I'm not opposing this patch, I find this class of changes a bit
odd. The *-names properties are there so the properties they are naming
can be in any order in the DT without impacting the lookup. Enforcing a
fixed order for named properties just feels odd.
Not sure if the schema validation could take this into account or if
there is any policy in place already by the DT maintainer that we still
want to enforce the same order in the DTs as in the bindings.
Regards,
Lucas
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Adam Ford <aford173@gmail.com>
> Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: Joakim Zhang <qiangqing.zhang@nxp.com>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: Liam Girdwood <lgirdwood@gmail.com>
> Cc: Lucas Stach <l.stach@pengutronix.de>
> Cc: Marek Vasut <marex@denx.de>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Markus Niebel <Markus.Niebel@ew.tq-group.com>
> Cc: NXP Linux Team <linux-imx@nxp.com>
> Cc: Paul Elder <paul.elder@ideasonboard.com>
> Cc: Peng Fan <peng.fan@nxp.com>
> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> Cc: Richard Cochran <richardcochran@gmail.com>
> Cc: Richard Zhu <hongxing.zhu@nxp.com>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Shengjiu Wang <shengjiu.wang@nxp.com>
> Cc: Stefan Agner <stefan@agner.ch>
> Cc: Tim Harvey <tharvey@gateworks.com>
> Cc: alsa-devel@alsa-project.org
> Cc: devicetree@vger.kernel.org
> To: linux-arm-kernel@lists.infradead.org
> ---
> arch/arm/boot/dts/vfxxx.dtsi | 20 ++++++++------------
> 1 file changed, 8 insertions(+), 12 deletions(-)
>
> diff --git a/arch/arm/boot/dts/vfxxx.dtsi b/arch/arm/boot/dts/vfxxx.dtsi
> index d53f9c9db8bfd..ff4479994b600 100644
> --- a/arch/arm/boot/dts/vfxxx.dtsi
> +++ b/arch/arm/boot/dts/vfxxx.dtsi
> @@ -191,9 +191,8 @@ sai0: sai@4002f000 {
> <&clks VF610_CLK_SAI0_DIV>,
> <&clks 0>, <&clks 0>;
> clock-names = "bus", "mclk1", "mclk2", "mclk3";
> - dma-names = "tx", "rx";
> - dmas = <&edma0 0 17>,
> - <&edma0 0 16>;
> + dma-names = "rx", "tx";
> + dmas = <&edma0 0 16>, <&edma0 0 17>;
> status = "disabled";
> };
>
> @@ -205,9 +204,8 @@ sai1: sai@40030000 {
> <&clks VF610_CLK_SAI1_DIV>,
> <&clks 0>, <&clks 0>;
> clock-names = "bus", "mclk1", "mclk2", "mclk3";
> - dma-names = "tx", "rx";
> - dmas = <&edma0 0 19>,
> - <&edma0 0 18>;
> + dma-names = "rx", "tx";
> + dmas = <&edma0 0 18>, <&edma0 0 19>;
> status = "disabled";
> };
>
> @@ -219,9 +217,8 @@ sai2: sai@40031000 {
> <&clks VF610_CLK_SAI2_DIV>,
> <&clks 0>, <&clks 0>;
> clock-names = "bus", "mclk1", "mclk2", "mclk3";
> - dma-names = "tx", "rx";
> - dmas = <&edma0 0 21>,
> - <&edma0 0 20>;
> + dma-names = "rx", "tx";
> + dmas = <&edma0 0 20>, <&edma0 0 21>;
> status = "disabled";
> };
>
> @@ -233,9 +230,8 @@ sai3: sai@40032000 {
> <&clks VF610_CLK_SAI3_DIV>,
> <&clks 0>, <&clks 0>;
> clock-names = "bus", "mclk1", "mclk2", "mclk3";
> - dma-names = "tx", "rx";
> - dmas = <&edma0 1 9>,
> - <&edma0 1 8>;
> + dma-names = "rx", "tx";
> + dmas = <&edma0 1 8>, <&edma0 1 9>;
> status = "disabled";
> };
>
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order
@ 2023-01-06 9:18 ` Lucas Stach
0 siblings, 0 replies; 21+ messages in thread
From: Lucas Stach @ 2023-01-06 9:18 UTC (permalink / raw)
To: Marek Vasut, linux-arm-kernel, Krzysztof Kozlowski, Rob Herring
Cc: Adam Ford, Alexander Stein, Fabio Estevam, Joakim Zhang,
Laurent Pinchart, Liam Girdwood, Mark Brown, Markus Niebel,
NXP Linux Team, Paul Elder, Peng Fan, Pengutronix Kernel Team,
Richard Cochran, Richard Zhu, Sascha Hauer, Shawn Guo,
Shengjiu Wang, Stefan Agner, Tim Harvey, alsa-devel, devicetree
Am Donnerstag, dem 05.01.2023 um 15:41 +0100 schrieb Marek Vasut:
> Swap the SAI dmas and dma-names to match the order documented in
> Documentation/devicetree/bindings/sound/fsl,sai.yaml
> No functional change.
>
While I'm not opposing this patch, I find this class of changes a bit
odd. The *-names properties are there so the properties they are naming
can be in any order in the DT without impacting the lookup. Enforcing a
fixed order for named properties just feels odd.
Not sure if the schema validation could take this into account or if
there is any policy in place already by the DT maintainer that we still
want to enforce the same order in the DTs as in the bindings.
Regards,
Lucas
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Adam Ford <aford173@gmail.com>
> Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: Joakim Zhang <qiangqing.zhang@nxp.com>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: Liam Girdwood <lgirdwood@gmail.com>
> Cc: Lucas Stach <l.stach@pengutronix.de>
> Cc: Marek Vasut <marex@denx.de>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Markus Niebel <Markus.Niebel@ew.tq-group.com>
> Cc: NXP Linux Team <linux-imx@nxp.com>
> Cc: Paul Elder <paul.elder@ideasonboard.com>
> Cc: Peng Fan <peng.fan@nxp.com>
> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> Cc: Richard Cochran <richardcochran@gmail.com>
> Cc: Richard Zhu <hongxing.zhu@nxp.com>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Shengjiu Wang <shengjiu.wang@nxp.com>
> Cc: Stefan Agner <stefan@agner.ch>
> Cc: Tim Harvey <tharvey@gateworks.com>
> Cc: alsa-devel@alsa-project.org
> Cc: devicetree@vger.kernel.org
> To: linux-arm-kernel@lists.infradead.org
> ---
> arch/arm/boot/dts/vfxxx.dtsi | 20 ++++++++------------
> 1 file changed, 8 insertions(+), 12 deletions(-)
>
> diff --git a/arch/arm/boot/dts/vfxxx.dtsi b/arch/arm/boot/dts/vfxxx.dtsi
> index d53f9c9db8bfd..ff4479994b600 100644
> --- a/arch/arm/boot/dts/vfxxx.dtsi
> +++ b/arch/arm/boot/dts/vfxxx.dtsi
> @@ -191,9 +191,8 @@ sai0: sai@4002f000 {
> <&clks VF610_CLK_SAI0_DIV>,
> <&clks 0>, <&clks 0>;
> clock-names = "bus", "mclk1", "mclk2", "mclk3";
> - dma-names = "tx", "rx";
> - dmas = <&edma0 0 17>,
> - <&edma0 0 16>;
> + dma-names = "rx", "tx";
> + dmas = <&edma0 0 16>, <&edma0 0 17>;
> status = "disabled";
> };
>
> @@ -205,9 +204,8 @@ sai1: sai@40030000 {
> <&clks VF610_CLK_SAI1_DIV>,
> <&clks 0>, <&clks 0>;
> clock-names = "bus", "mclk1", "mclk2", "mclk3";
> - dma-names = "tx", "rx";
> - dmas = <&edma0 0 19>,
> - <&edma0 0 18>;
> + dma-names = "rx", "tx";
> + dmas = <&edma0 0 18>, <&edma0 0 19>;
> status = "disabled";
> };
>
> @@ -219,9 +217,8 @@ sai2: sai@40031000 {
> <&clks VF610_CLK_SAI2_DIV>,
> <&clks 0>, <&clks 0>;
> clock-names = "bus", "mclk1", "mclk2", "mclk3";
> - dma-names = "tx", "rx";
> - dmas = <&edma0 0 21>,
> - <&edma0 0 20>;
> + dma-names = "rx", "tx";
> + dmas = <&edma0 0 20>, <&edma0 0 21>;
> status = "disabled";
> };
>
> @@ -233,9 +230,8 @@ sai3: sai@40032000 {
> <&clks VF610_CLK_SAI3_DIV>,
> <&clks 0>, <&clks 0>;
> clock-names = "bus", "mclk1", "mclk2", "mclk3";
> - dma-names = "tx", "rx";
> - dmas = <&edma0 1 9>,
> - <&edma0 1 8>;
> + dma-names = "rx", "tx";
> + dmas = <&edma0 1 8>, <&edma0 1 9>;
> status = "disabled";
> };
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order
@ 2023-01-06 9:18 ` Lucas Stach
0 siblings, 0 replies; 21+ messages in thread
From: Lucas Stach @ 2023-01-06 9:18 UTC (permalink / raw)
To: Marek Vasut, linux-arm-kernel, Krzysztof Kozlowski, Rob Herring
Cc: Shengjiu Wang, Peng Fan, Markus Niebel, Richard Zhu, alsa-devel,
devicetree, Alexander Stein, Adam Ford, Richard Cochran,
Paul Elder, Liam Girdwood, Stefan Agner, Joakim Zhang,
Mark Brown, Laurent Pinchart, Pengutronix Kernel Team, Shawn Guo,
Tim Harvey, Fabio Estevam, Sascha Hauer, NXP Linux Team
Am Donnerstag, dem 05.01.2023 um 15:41 +0100 schrieb Marek Vasut:
> Swap the SAI dmas and dma-names to match the order documented in
> Documentation/devicetree/bindings/sound/fsl,sai.yaml
> No functional change.
>
While I'm not opposing this patch, I find this class of changes a bit
odd. The *-names properties are there so the properties they are naming
can be in any order in the DT without impacting the lookup. Enforcing a
fixed order for named properties just feels odd.
Not sure if the schema validation could take this into account or if
there is any policy in place already by the DT maintainer that we still
want to enforce the same order in the DTs as in the bindings.
Regards,
Lucas
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Adam Ford <aford173@gmail.com>
> Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: Joakim Zhang <qiangqing.zhang@nxp.com>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: Liam Girdwood <lgirdwood@gmail.com>
> Cc: Lucas Stach <l.stach@pengutronix.de>
> Cc: Marek Vasut <marex@denx.de>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Markus Niebel <Markus.Niebel@ew.tq-group.com>
> Cc: NXP Linux Team <linux-imx@nxp.com>
> Cc: Paul Elder <paul.elder@ideasonboard.com>
> Cc: Peng Fan <peng.fan@nxp.com>
> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> Cc: Richard Cochran <richardcochran@gmail.com>
> Cc: Richard Zhu <hongxing.zhu@nxp.com>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Shengjiu Wang <shengjiu.wang@nxp.com>
> Cc: Stefan Agner <stefan@agner.ch>
> Cc: Tim Harvey <tharvey@gateworks.com>
> Cc: alsa-devel@alsa-project.org
> Cc: devicetree@vger.kernel.org
> To: linux-arm-kernel@lists.infradead.org
> ---
> arch/arm/boot/dts/vfxxx.dtsi | 20 ++++++++------------
> 1 file changed, 8 insertions(+), 12 deletions(-)
>
> diff --git a/arch/arm/boot/dts/vfxxx.dtsi b/arch/arm/boot/dts/vfxxx.dtsi
> index d53f9c9db8bfd..ff4479994b600 100644
> --- a/arch/arm/boot/dts/vfxxx.dtsi
> +++ b/arch/arm/boot/dts/vfxxx.dtsi
> @@ -191,9 +191,8 @@ sai0: sai@4002f000 {
> <&clks VF610_CLK_SAI0_DIV>,
> <&clks 0>, <&clks 0>;
> clock-names = "bus", "mclk1", "mclk2", "mclk3";
> - dma-names = "tx", "rx";
> - dmas = <&edma0 0 17>,
> - <&edma0 0 16>;
> + dma-names = "rx", "tx";
> + dmas = <&edma0 0 16>, <&edma0 0 17>;
> status = "disabled";
> };
>
> @@ -205,9 +204,8 @@ sai1: sai@40030000 {
> <&clks VF610_CLK_SAI1_DIV>,
> <&clks 0>, <&clks 0>;
> clock-names = "bus", "mclk1", "mclk2", "mclk3";
> - dma-names = "tx", "rx";
> - dmas = <&edma0 0 19>,
> - <&edma0 0 18>;
> + dma-names = "rx", "tx";
> + dmas = <&edma0 0 18>, <&edma0 0 19>;
> status = "disabled";
> };
>
> @@ -219,9 +217,8 @@ sai2: sai@40031000 {
> <&clks VF610_CLK_SAI2_DIV>,
> <&clks 0>, <&clks 0>;
> clock-names = "bus", "mclk1", "mclk2", "mclk3";
> - dma-names = "tx", "rx";
> - dmas = <&edma0 0 21>,
> - <&edma0 0 20>;
> + dma-names = "rx", "tx";
> + dmas = <&edma0 0 20>, <&edma0 0 21>;
> status = "disabled";
> };
>
> @@ -233,9 +230,8 @@ sai3: sai@40032000 {
> <&clks VF610_CLK_SAI3_DIV>,
> <&clks 0>, <&clks 0>;
> clock-names = "bus", "mclk1", "mclk2", "mclk3";
> - dma-names = "tx", "rx";
> - dmas = <&edma0 1 9>,
> - <&edma0 1 8>;
> + dma-names = "rx", "tx";
> + dmas = <&edma0 1 8>, <&edma0 1 9>;
> status = "disabled";
> };
>
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order
2023-01-06 9:18 ` Lucas Stach
(?)
@ 2023-01-06 9:24 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-06 9:24 UTC (permalink / raw)
To: Lucas Stach, Marek Vasut, linux-arm-kernel, Krzysztof Kozlowski,
Rob Herring
Cc: Shengjiu Wang, Peng Fan, Markus Niebel, Richard Zhu, alsa-devel,
devicetree, Alexander Stein, Adam Ford, Richard Cochran,
Paul Elder, Liam Girdwood, Stefan Agner, Joakim Zhang,
Mark Brown, Laurent Pinchart, Pengutronix Kernel Team, Shawn Guo,
Tim Harvey, Fabio Estevam, Sascha Hauer, NXP Linux Team
On 06/01/2023 10:18, Lucas Stach wrote:
> Am Donnerstag, dem 05.01.2023 um 15:41 +0100 schrieb Marek Vasut:
>> Swap the SAI dmas and dma-names to match the order documented in
>> Documentation/devicetree/bindings/sound/fsl,sai.yaml
>> No functional change.
>>
> While I'm not opposing this patch, I find this class of changes a bit
> odd. The *-names properties are there so the properties they are naming
> can be in any order in the DT without impacting the lookup. Enforcing a
> fixed order for named properties just feels odd.
DT requires fixed order regardless of '-names' property. '-names' are
not there to fulfill "can be in any order", but merely a helper.
>
> Not sure if the schema validation could take this into account or if
> there is any policy in place already by the DT maintainer that we still
> want to enforce the same order in the DTs as in the bindings.
The order in DTS must obviously match bindings, however here both are
changed, so what is exactly your concern?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order
@ 2023-01-06 9:24 ` Krzysztof Kozlowski
0 siblings, 0 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-06 9:24 UTC (permalink / raw)
To: Lucas Stach, Marek Vasut, linux-arm-kernel, Krzysztof Kozlowski,
Rob Herring
Cc: Adam Ford, Alexander Stein, Fabio Estevam, Joakim Zhang,
Laurent Pinchart, Liam Girdwood, Mark Brown, Markus Niebel,
NXP Linux Team, Paul Elder, Peng Fan, Pengutronix Kernel Team,
Richard Cochran, Richard Zhu, Sascha Hauer, Shawn Guo,
Shengjiu Wang, Stefan Agner, Tim Harvey, alsa-devel, devicetree
On 06/01/2023 10:18, Lucas Stach wrote:
> Am Donnerstag, dem 05.01.2023 um 15:41 +0100 schrieb Marek Vasut:
>> Swap the SAI dmas and dma-names to match the order documented in
>> Documentation/devicetree/bindings/sound/fsl,sai.yaml
>> No functional change.
>>
> While I'm not opposing this patch, I find this class of changes a bit
> odd. The *-names properties are there so the properties they are naming
> can be in any order in the DT without impacting the lookup. Enforcing a
> fixed order for named properties just feels odd.
DT requires fixed order regardless of '-names' property. '-names' are
not there to fulfill "can be in any order", but merely a helper.
>
> Not sure if the schema validation could take this into account or if
> there is any policy in place already by the DT maintainer that we still
> want to enforce the same order in the DTs as in the bindings.
The order in DTS must obviously match bindings, however here both are
changed, so what is exactly your concern?
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order
@ 2023-01-06 9:24 ` Krzysztof Kozlowski
0 siblings, 0 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-06 9:24 UTC (permalink / raw)
To: Lucas Stach, Marek Vasut, linux-arm-kernel, Krzysztof Kozlowski,
Rob Herring
Cc: Adam Ford, Alexander Stein, Fabio Estevam, Joakim Zhang,
Laurent Pinchart, Liam Girdwood, Mark Brown, Markus Niebel,
NXP Linux Team, Paul Elder, Peng Fan, Pengutronix Kernel Team,
Richard Cochran, Richard Zhu, Sascha Hauer, Shawn Guo,
Shengjiu Wang, Stefan Agner, Tim Harvey, alsa-devel, devicetree
On 06/01/2023 10:18, Lucas Stach wrote:
> Am Donnerstag, dem 05.01.2023 um 15:41 +0100 schrieb Marek Vasut:
>> Swap the SAI dmas and dma-names to match the order documented in
>> Documentation/devicetree/bindings/sound/fsl,sai.yaml
>> No functional change.
>>
> While I'm not opposing this patch, I find this class of changes a bit
> odd. The *-names properties are there so the properties they are naming
> can be in any order in the DT without impacting the lookup. Enforcing a
> fixed order for named properties just feels odd.
DT requires fixed order regardless of '-names' property. '-names' are
not there to fulfill "can be in any order", but merely a helper.
>
> Not sure if the schema validation could take this into account or if
> there is any policy in place already by the DT maintainer that we still
> want to enforce the same order in the DTs as in the bindings.
The order in DTS must obviously match bindings, however here both are
changed, so what is exactly your concern?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order
2023-01-05 14:41 ` Marek Vasut
(?)
@ 2023-01-25 23:57 ` Shawn Guo
-1 siblings, 0 replies; 21+ messages in thread
From: Shawn Guo @ 2023-01-25 23:57 UTC (permalink / raw)
To: Marek Vasut
Cc: linux-arm-kernel, Adam Ford, Alexander Stein, Fabio Estevam,
Joakim Zhang, Krzysztof Kozlowski, Laurent Pinchart,
Liam Girdwood, Lucas Stach, Mark Brown, Markus Niebel,
NXP Linux Team, Paul Elder, Peng Fan, Pengutronix Kernel Team,
Richard Cochran, Richard Zhu, Rob Herring, Sascha Hauer,
Shengjiu Wang, Stefan Agner, Tim Harvey, alsa-devel, devicetree
On Thu, Jan 05, 2023 at 03:41:45PM +0100, Marek Vasut wrote:
> Swap the SAI dmas and dma-names to match the order documented in
> Documentation/devicetree/bindings/sound/fsl,sai.yaml
> No functional change.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
Applied, thanks!
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order
@ 2023-01-25 23:57 ` Shawn Guo
0 siblings, 0 replies; 21+ messages in thread
From: Shawn Guo @ 2023-01-25 23:57 UTC (permalink / raw)
To: Marek Vasut
Cc: Peng Fan, Sascha Hauer, Alexander Stein, alsa-devel,
Stefan Agner, Liam Girdwood, Laurent Pinchart,
Krzysztof Kozlowski, Fabio Estevam, Markus Niebel, Paul Elder,
NXP Linux Team, Tim Harvey, devicetree, Richard Zhu,
Richard Cochran, Rob Herring, Adam Ford, linux-arm-kernel,
Shengjiu Wang, Joakim Zhang, Mark Brown, Pengutronix Kernel Team,
Lucas Stach
On Thu, Jan 05, 2023 at 03:41:45PM +0100, Marek Vasut wrote:
> Swap the SAI dmas and dma-names to match the order documented in
> Documentation/devicetree/bindings/sound/fsl,sai.yaml
> No functional change.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
Applied, thanks!
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order
@ 2023-01-25 23:57 ` Shawn Guo
0 siblings, 0 replies; 21+ messages in thread
From: Shawn Guo @ 2023-01-25 23:57 UTC (permalink / raw)
To: Marek Vasut
Cc: linux-arm-kernel, Adam Ford, Alexander Stein, Fabio Estevam,
Joakim Zhang, Krzysztof Kozlowski, Laurent Pinchart,
Liam Girdwood, Lucas Stach, Mark Brown, Markus Niebel,
NXP Linux Team, Paul Elder, Peng Fan, Pengutronix Kernel Team,
Richard Cochran, Richard Zhu, Rob Herring, Sascha Hauer,
Shengjiu Wang, Stefan Agner, Tim Harvey, alsa-devel, devicetree
On Thu, Jan 05, 2023 at 03:41:45PM +0100, Marek Vasut wrote:
> Swap the SAI dmas and dma-names to match the order documented in
> Documentation/devicetree/bindings/sound/fsl,sai.yaml
> No functional change.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
Applied, thanks!
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: (subset) [PATCH 1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding
2023-01-05 14:41 ` Marek Vasut
(?)
@ 2023-01-30 21:00 ` Mark Brown
-1 siblings, 0 replies; 21+ messages in thread
From: Mark Brown @ 2023-01-30 21:00 UTC (permalink / raw)
To: linux-arm-kernel, Marek Vasut
Cc: Adam Ford, Alexander Stein, Fabio Estevam, Joakim Zhang,
Krzysztof Kozlowski, Laurent Pinchart, Liam Girdwood,
Lucas Stach, Markus Niebel, NXP Linux Team, Paul Elder, Peng Fan,
Pengutronix Kernel Team, Richard Cochran, Richard Zhu,
Rob Herring, Sascha Hauer, Shawn Guo, Shengjiu Wang,
Stefan Agner, Tim Harvey, alsa-devel, devicetree
On Thu, 05 Jan 2023 15:41:44 +0100, Marek Vasut wrote:
> Get rid of the vf610 sai special case, instead update the vfxxx.dtsi
> DT to use the same DMA channel ordering as all the other devices. The
> sai DMA channel ordering has not been aligned with other IP DMA channel
> ordering in the vfxxx.dtsi anyway.
>
>
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
Thanks!
[1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding
commit: 21d64f6f63eeca9f136ac514ca801a5a6485cd78
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: (subset) [PATCH 1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding
@ 2023-01-30 21:00 ` Mark Brown
0 siblings, 0 replies; 21+ messages in thread
From: Mark Brown @ 2023-01-30 21:00 UTC (permalink / raw)
To: linux-arm-kernel, Marek Vasut
Cc: Peng Fan, Sascha Hauer, Alexander Stein, alsa-devel,
Stefan Agner, Liam Girdwood, Laurent Pinchart,
Krzysztof Kozlowski, Fabio Estevam, Markus Niebel, Paul Elder,
NXP Linux Team, Tim Harvey, devicetree, Richard Zhu,
Richard Cochran, Rob Herring, Adam Ford, Shengjiu Wang,
Joakim Zhang, Pengutronix Kernel Team, Shawn Guo, Lucas Stach
On Thu, 05 Jan 2023 15:41:44 +0100, Marek Vasut wrote:
> Get rid of the vf610 sai special case, instead update the vfxxx.dtsi
> DT to use the same DMA channel ordering as all the other devices. The
> sai DMA channel ordering has not been aligned with other IP DMA channel
> ordering in the vfxxx.dtsi anyway.
>
>
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
Thanks!
[1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding
commit: 21d64f6f63eeca9f136ac514ca801a5a6485cd78
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: (subset) [PATCH 1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding
@ 2023-01-30 21:00 ` Mark Brown
0 siblings, 0 replies; 21+ messages in thread
From: Mark Brown @ 2023-01-30 21:00 UTC (permalink / raw)
To: linux-arm-kernel, Marek Vasut
Cc: Adam Ford, Alexander Stein, Fabio Estevam, Joakim Zhang,
Krzysztof Kozlowski, Laurent Pinchart, Liam Girdwood,
Lucas Stach, Markus Niebel, NXP Linux Team, Paul Elder, Peng Fan,
Pengutronix Kernel Team, Richard Cochran, Richard Zhu,
Rob Herring, Sascha Hauer, Shawn Guo, Shengjiu Wang,
Stefan Agner, Tim Harvey, alsa-devel, devicetree
On Thu, 05 Jan 2023 15:41:44 +0100, Marek Vasut wrote:
> Get rid of the vf610 sai special case, instead update the vfxxx.dtsi
> DT to use the same DMA channel ordering as all the other devices. The
> sai DMA channel ordering has not been aligned with other IP DMA channel
> ordering in the vfxxx.dtsi anyway.
>
>
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
Thanks!
[1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding
commit: 21d64f6f63eeca9f136ac514ca801a5a6485cd78
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 21+ messages in thread
end of thread, other threads:[~2023-01-30 21:01 UTC | newest]
Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-05 14:41 [PATCH 1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding Marek Vasut
2023-01-05 14:41 ` Marek Vasut
2023-01-05 14:41 ` Marek Vasut
2023-01-05 14:41 ` [PATCH 2/2] ARM: dts: vfxxx: Swap SAI DMA order Marek Vasut
2023-01-05 14:41 ` Marek Vasut
2023-01-05 14:41 ` Marek Vasut
2023-01-06 9:18 ` Lucas Stach
2023-01-06 9:18 ` Lucas Stach
2023-01-06 9:18 ` Lucas Stach
2023-01-06 9:24 ` Krzysztof Kozlowski
2023-01-06 9:24 ` Krzysztof Kozlowski
2023-01-06 9:24 ` Krzysztof Kozlowski
2023-01-25 23:57 ` Shawn Guo
2023-01-25 23:57 ` Shawn Guo
2023-01-25 23:57 ` Shawn Guo
2023-01-06 9:16 ` [PATCH 1/2] ASoC: dt-bindings: fsl-sai: Simplify the VFxxx dmas binding Krzysztof Kozlowski
2023-01-06 9:16 ` Krzysztof Kozlowski
2023-01-06 9:16 ` Krzysztof Kozlowski
2023-01-30 21:00 ` (subset) " Mark Brown
2023-01-30 21:00 ` Mark Brown
2023-01-30 21:00 ` Mark Brown
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.