diff mbox series

[1/3] arm64: dts: imx8mq: Swap SAI DMA order

Message ID 20221103181920.95736-1-marex@denx.de (mailing list archive)
State New, archived
Headers show
Series [1/3] arm64: dts: imx8mq: Swap SAI DMA order | expand

Commit Message

Marek Vasut Nov. 3, 2022, 6:19 p.m. UTC
Swap the SAI dmas and dma-names to match the order documented in
Documentation/devicetree/bindings/sound/fsl-sai.txt
using the following regex:

"
s@\(dma[^=]\+= \)\([^,]\+\), \([^;]\+\);@\1\3, \2;
"

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: Lucas Stach <l.stach@pengutronix.de>
Cc: Marek Vasut <marex@denx.de>
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 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: Tim Harvey <tharvey@gateworks.com>
Cc: devicetree@vger.kernel.org
To: linux-arm-kernel@lists.infradead.org
---
 arch/arm64/boot/dts/freescale/imx8mq.dtsi | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

Comments

Alexander Stein Nov. 4, 2022, 7:13 a.m. UTC | #1
Hi Marek,

thanks for this patch.

Am Donnerstag, 3. November 2022, 19:19:18 CET schrieb Marek Vasut:
> Swap the SAI dmas and dma-names to match the order documented in
> Documentation/devicetree/bindings/sound/fsl-sai.txt

This file has been converted to Documentation/devicetree/bindings/sound/
fsl,sai.yaml in commit d563336877b2 ("ASoC: dt-bindings: fsl,sai: Convert 
format to json-schema"). In next since next-20220816.
In this schema the DMA order 'tx', 'rx' is only for fsl,vf610-sai, all other 
compatibles use 'rx', 'tx'. So this swap is not needed anymore.
Same is for imx8mm and imx8mn.
This change should have been made before schema conversion, so they would have 
been identical on all compatibles...

Best regards,
Alexander

> using the following regex:
> 
> "
> s@\(dma[^=]\+= \)\([^,]\+\), \([^;]\+\);@\1\3, \2;
> "
> 
> 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: Lucas Stach <l.stach@pengutronix.de>
> Cc: Marek Vasut <marex@denx.de>
> 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 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: Tim Harvey <tharvey@gateworks.com>
> Cc: devicetree@vger.kernel.org
> To: linux-arm-kernel@lists.infradead.org
> ---
>  arch/arm64/boot/dts/freescale/imx8mq.dtsi | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> b/arch/arm64/boot/dts/freescale/imx8mq.dtsi index
> c47e2d7235d3e..f00b040ca03e9 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> @@ -345,8 +345,8 @@ sai1: sai@30010000 {
>  				         <&clk IMX8MQ_CLK_SAI1_ROOT>,
>  				         <&clk IMX8MQ_CLK_DUMMY>, 
<&clk IMX8MQ_CLK_DUMMY>;
>  				clock-names = "bus", "mclk1", "mclk2", 
"mclk3";
> -				dmas = <&sdma2 8 24 0>, <&sdma1 9 24 
0>;
> -				dma-names = "rx", "tx";
> +				dmas = <&sdma1 9 24 0>, <&sdma2 8 24 
0>;
> +				dma-names = "tx", "rx";
>  				status = "disabled";
>  			};
> 
> @@ -359,8 +359,8 @@ sai6: sai@30030000 {
>  				         <&clk IMX8MQ_CLK_SAI6_ROOT>,
>  				         <&clk IMX8MQ_CLK_DUMMY>, 
<&clk IMX8MQ_CLK_DUMMY>;
>  				clock-names = "bus", "mclk1", "mclk2", 
"mclk3";
> -				dmas = <&sdma2 4 24 0>, <&sdma2 5 24 
0>;
> -				dma-names = "rx", "tx";
> +				dmas = <&sdma2 5 24 0>, <&sdma2 4 24 
0>;
> +				dma-names = "tx", "rx";
>  				status = "disabled";
>  			};
> 
> @@ -373,8 +373,8 @@ sai5: sai@30040000 {
>  				         <&clk IMX8MQ_CLK_SAI5_ROOT>,
>  				         <&clk IMX8MQ_CLK_DUMMY>, 
<&clk IMX8MQ_CLK_DUMMY>;
>  				clock-names = "bus", "mclk1", "mclk2", 
"mclk3";
> -				dmas = <&sdma2 2 24 0>, <&sdma2 3 24 
0>;
> -				dma-names = "rx", "tx";
> +				dmas = <&sdma2 3 24 0>, <&sdma2 2 24 
0>;
> +				dma-names = "tx", "rx";
>  				status = "disabled";
>  			};
> 
> @@ -387,8 +387,8 @@ sai4: sai@30050000 {
>  				         <&clk IMX8MQ_CLK_SAI4_ROOT>,
>  				         <&clk IMX8MQ_CLK_DUMMY>, 
<&clk IMX8MQ_CLK_DUMMY>;
>  				clock-names = "bus", "mclk1", "mclk2", 
"mclk3";
> -				dmas = <&sdma2 0 24 0>, <&sdma2 1 24 
0>;
> -				dma-names = "rx", "tx";
> +				dmas = <&sdma2 1 24 0>, <&sdma2 0 24 
0>;
> +				dma-names = "tx", "rx";
>  				status = "disabled";
>  			};
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index c47e2d7235d3e..f00b040ca03e9 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -345,8 +345,8 @@  sai1: sai@30010000 {
 				         <&clk IMX8MQ_CLK_SAI1_ROOT>,
 				         <&clk IMX8MQ_CLK_DUMMY>, <&clk IMX8MQ_CLK_DUMMY>;
 				clock-names = "bus", "mclk1", "mclk2", "mclk3";
-				dmas = <&sdma2 8 24 0>, <&sdma1 9 24 0>;
-				dma-names = "rx", "tx";
+				dmas = <&sdma1 9 24 0>, <&sdma2 8 24 0>;
+				dma-names = "tx", "rx";
 				status = "disabled";
 			};
 
@@ -359,8 +359,8 @@  sai6: sai@30030000 {
 				         <&clk IMX8MQ_CLK_SAI6_ROOT>,
 				         <&clk IMX8MQ_CLK_DUMMY>, <&clk IMX8MQ_CLK_DUMMY>;
 				clock-names = "bus", "mclk1", "mclk2", "mclk3";
-				dmas = <&sdma2 4 24 0>, <&sdma2 5 24 0>;
-				dma-names = "rx", "tx";
+				dmas = <&sdma2 5 24 0>, <&sdma2 4 24 0>;
+				dma-names = "tx", "rx";
 				status = "disabled";
 			};
 
@@ -373,8 +373,8 @@  sai5: sai@30040000 {
 				         <&clk IMX8MQ_CLK_SAI5_ROOT>,
 				         <&clk IMX8MQ_CLK_DUMMY>, <&clk IMX8MQ_CLK_DUMMY>;
 				clock-names = "bus", "mclk1", "mclk2", "mclk3";
-				dmas = <&sdma2 2 24 0>, <&sdma2 3 24 0>;
-				dma-names = "rx", "tx";
+				dmas = <&sdma2 3 24 0>, <&sdma2 2 24 0>;
+				dma-names = "tx", "rx";
 				status = "disabled";
 			};
 
@@ -387,8 +387,8 @@  sai4: sai@30050000 {
 				         <&clk IMX8MQ_CLK_SAI4_ROOT>,
 				         <&clk IMX8MQ_CLK_DUMMY>, <&clk IMX8MQ_CLK_DUMMY>;
 				clock-names = "bus", "mclk1", "mclk2", "mclk3";
-				dmas = <&sdma2 0 24 0>, <&sdma2 1 24 0>;
-				dma-names = "rx", "tx";
+				dmas = <&sdma2 1 24 0>, <&sdma2 0 24 0>;
+				dma-names = "tx", "rx";
 				status = "disabled";
 			};