devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 0/3] ARM: dts: imx[6qdl|7]-[apalis|colibri]: Update audio card names
@ 2024-10-10 17:11 Hiago De Franco
  2024-10-10 17:11 ` [PATCH v2 1/3] ARM: dts: imx6qdl-apalis: Update audio card name Hiago De Franco
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: Hiago De Franco @ 2024-10-10 17:11 UTC (permalink / raw)
  To: Shawn Guo
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, devicetree, imx,
	linux-arm-kernel, linux-kernel, Hiago De Franco

From: Hiago De Franco <hiago.franco@toradex.com>

This patch series update the audio card names for Apalis and Colibri
iMX6/7 to match their specific SoM names, making it shorter (less than
15 characters, as shown in [1]) to fix warnings related to ALSA in
Apalis iMX6 and Colibri iMX6. For Colibri iMX7, also do the change to
make it compliant with the other card names.

[1] Documentation/sound/alsa-configuration.rst

v1:
  - https://lore.kernel.org/lkml/20240909114902.82380-1-hiagofranco@gmail.com/

v2:
  - Update commit message to make clearer these are breaking changes.

Hiago De Franco (3):
  ARM: dts: imx6qdl-apalis: Update audio card name
  ARM: dts: imx6qdl-colibri: Update audio card name
  ARM: dts: imx7-colibri: Update audio card name

 arch/arm/boot/dts/nxp/imx/imx6qdl-apalis.dtsi  | 2 +-
 arch/arm/boot/dts/nxp/imx/imx6qdl-colibri.dtsi | 2 +-
 arch/arm/boot/dts/nxp/imx/imx7-colibri.dtsi    | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

-- 
2.39.5


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

* [PATCH v2 1/3] ARM: dts: imx6qdl-apalis: Update audio card name
  2024-10-10 17:11 [PATCH v2 0/3] ARM: dts: imx[6qdl|7]-[apalis|colibri]: Update audio card names Hiago De Franco
@ 2024-10-10 17:11 ` Hiago De Franco
  2024-10-10 17:11 ` [PATCH v2 2/3] ARM: dts: imx6qdl-colibri: " Hiago De Franco
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Hiago De Franco @ 2024-10-10 17:11 UTC (permalink / raw)
  To: Shawn Guo
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, devicetree, imx,
	linux-arm-kernel, linux-kernel, Hiago De Franco

From: Hiago De Franco <hiago.franco@toradex.com>

Update the audio card name for Apalis iMX6 to match its specific SoM
name, making it less than 15 characters to fix the following warning

fsl-asoc-card sound: ASoC: driver name too long 'imx6q-apalis-sgtl5000'
-> 'imx6q-apalis-sg'

making it compliant with the ALSA configuration specification [1].

While this is a breaking change for userspace tooling, it seems
time to implement it since no ALSA UCM or related configuration files
currently exist and we are in the the process of creating them.

[1] Documentation/sound/alsa-configuration.rst

Signed-off-by: Hiago De Franco <hiago.franco@toradex.com>
---
 arch/arm/boot/dts/nxp/imx/imx6qdl-apalis.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-apalis.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-apalis.dtsi
index edf55760a5c1..1c72da417011 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-apalis.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-apalis.dtsi
@@ -191,7 +191,7 @@ sound {
 			"MIC_IN", "Mic Jack",
 			"Mic Jack", "Mic Bias",
 			"Headphone Jack", "HP_OUT";
-		model = "imx6q-apalis-sgtl5000";
+		model = "apalis-imx6";
 		mux-ext-port = <4>;
 		mux-int-port = <1>;
 		ssi-controller = <&ssi1>;
-- 
2.39.5


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

* [PATCH v2 2/3] ARM: dts: imx6qdl-colibri: Update audio card name
  2024-10-10 17:11 [PATCH v2 0/3] ARM: dts: imx[6qdl|7]-[apalis|colibri]: Update audio card names Hiago De Franco
  2024-10-10 17:11 ` [PATCH v2 1/3] ARM: dts: imx6qdl-apalis: Update audio card name Hiago De Franco
@ 2024-10-10 17:11 ` Hiago De Franco
  2024-10-10 17:11 ` [PATCH v2 3/3] ARM: dts: imx7-colibri: " Hiago De Franco
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Hiago De Franco @ 2024-10-10 17:11 UTC (permalink / raw)
  To: Shawn Guo
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, devicetree, imx,
	linux-arm-kernel, linux-kernel, Hiago De Franco

From: Hiago De Franco <hiago.franco@toradex.com>

Update the audio card name for Colibri iMX6 to match its specific SoM
name, making it less than 15 characters to fix the following warning

fsl-asoc-card sound: ASoC: driver name too long
'imx6dl-colibri-sgtl5000' -> 'imx6dl-colibri-'

making it compliant with the ALSA configuration specification [1].

While this is a breaking change for userspace tooling, it seems
time to implement it since no ALSA UCM or related configuration files
currently exist and we are in the the process of creating them.

[1] Documentation/sound/alsa-configuration.rst

Signed-off-by: Hiago De Franco <hiago.franco@toradex.com>
---
 arch/arm/boot/dts/nxp/imx/imx6qdl-colibri.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-colibri.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-colibri.dtsi
index b01670cdd52c..9f33419c260b 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-colibri.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-colibri.dtsi
@@ -136,7 +136,7 @@ sound {
 			"LINE_IN", "Line In Jack",
 			"MIC_IN", "Mic Jack",
 			"Mic Jack", "Mic Bias";
-		model = "imx6dl-colibri-sgtl5000";
+		model = "colibri-imx6";
 		mux-int-port = <1>;
 		mux-ext-port = <5>;
 		ssi-controller = <&ssi1>;
-- 
2.39.5


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

* [PATCH v2 3/3] ARM: dts: imx7-colibri: Update audio card name
  2024-10-10 17:11 [PATCH v2 0/3] ARM: dts: imx[6qdl|7]-[apalis|colibri]: Update audio card names Hiago De Franco
  2024-10-10 17:11 ` [PATCH v2 1/3] ARM: dts: imx6qdl-apalis: Update audio card name Hiago De Franco
  2024-10-10 17:11 ` [PATCH v2 2/3] ARM: dts: imx6qdl-colibri: " Hiago De Franco
@ 2024-10-10 17:11 ` Hiago De Franco
  2024-10-10 17:42 ` [PATCH v2 0/3] ARM: dts: imx[6qdl|7]-[apalis|colibri]: Update audio card names Hiago De Franco
  2024-10-22  1:19 ` Shawn Guo
  4 siblings, 0 replies; 6+ messages in thread
From: Hiago De Franco @ 2024-10-10 17:11 UTC (permalink / raw)
  To: Shawn Guo
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, devicetree, imx,
	linux-arm-kernel, linux-kernel, Hiago De Franco

From: Hiago De Franco <hiago.franco@toradex.com>

Update the audio card name for Colibri iMX7 to match its specific SoM
name, making it compliant with the other names for Colibri iMX6 and
Apalis iMX6.

While this is a breaking change for userspace tooling, it seems
time to implement it since no ALSA UCM or related configuration files
currently exist and we are in the the process of creating them.

Signed-off-by: Hiago De Franco <hiago.franco@toradex.com>
---
 arch/arm/boot/dts/nxp/imx/imx7-colibri.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx7-colibri.dtsi b/arch/arm/boot/dts/nxp/imx/imx7-colibri.dtsi
index 62e41edcaf1d..8666dcd7fe97 100644
--- a/arch/arm/boot/dts/nxp/imx/imx7-colibri.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx7-colibri.dtsi
@@ -120,7 +120,7 @@ sound {
 		simple-audio-card,bitclock-master = <&dailink_master>;
 		simple-audio-card,format = "i2s";
 		simple-audio-card,frame-master = <&dailink_master>;
-		simple-audio-card,name = "imx7-sgtl5000";
+		simple-audio-card,name = "colibri-imx7";
 
 		simple-audio-card,cpu {
 			sound-dai = <&sai1>;
-- 
2.39.5


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

* Re: [PATCH v2 0/3] ARM: dts: imx[6qdl|7]-[apalis|colibri]: Update audio card names
  2024-10-10 17:11 [PATCH v2 0/3] ARM: dts: imx[6qdl|7]-[apalis|colibri]: Update audio card names Hiago De Franco
                   ` (2 preceding siblings ...)
  2024-10-10 17:11 ` [PATCH v2 3/3] ARM: dts: imx7-colibri: " Hiago De Franco
@ 2024-10-10 17:42 ` Hiago De Franco
  2024-10-22  1:19 ` Shawn Guo
  4 siblings, 0 replies; 6+ messages in thread
From: Hiago De Franco @ 2024-10-10 17:42 UTC (permalink / raw)
  To: Shawn Guo
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, devicetree, imx,
	linux-arm-kernel, linux-kernel, Hiago De Franco

> This patch series update the audio card names for Apalis and Colibri
> iMX6/7 to match their specific SoM names, making it shorter (less than
> 15 characters, as shown in [1]) to fix warnings related to ALSA in
> Apalis iMX6 and Colibri iMX6. For Colibri iMX7, also do the change to
> make it compliant with the other card names.
>
> [1] Documentation/sound/alsa-configuration.rst

Related ALSA UCM PR https://github.com/alsa-project/alsa-ucm-conf/pull/448

Regards,
Hiago.

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

* Re: [PATCH v2 0/3] ARM: dts: imx[6qdl|7]-[apalis|colibri]: Update audio card names
  2024-10-10 17:11 [PATCH v2 0/3] ARM: dts: imx[6qdl|7]-[apalis|colibri]: Update audio card names Hiago De Franco
                   ` (3 preceding siblings ...)
  2024-10-10 17:42 ` [PATCH v2 0/3] ARM: dts: imx[6qdl|7]-[apalis|colibri]: Update audio card names Hiago De Franco
@ 2024-10-22  1:19 ` Shawn Guo
  4 siblings, 0 replies; 6+ messages in thread
From: Shawn Guo @ 2024-10-22  1:19 UTC (permalink / raw)
  To: Hiago De Franco
  Cc: Shawn Guo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, devicetree,
	imx, linux-arm-kernel, linux-kernel, Hiago De Franco

On Thu, Oct 10, 2024 at 02:11:25PM -0300, Hiago De Franco wrote:
> Hiago De Franco (3):
>   ARM: dts: imx6qdl-apalis: Update audio card name
>   ARM: dts: imx6qdl-colibri: Update audio card name
>   ARM: dts: imx7-colibri: Update audio card name

Applied all, thanks!


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

end of thread, other threads:[~2024-10-22  1:20 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-10 17:11 [PATCH v2 0/3] ARM: dts: imx[6qdl|7]-[apalis|colibri]: Update audio card names Hiago De Franco
2024-10-10 17:11 ` [PATCH v2 1/3] ARM: dts: imx6qdl-apalis: Update audio card name Hiago De Franco
2024-10-10 17:11 ` [PATCH v2 2/3] ARM: dts: imx6qdl-colibri: " Hiago De Franco
2024-10-10 17:11 ` [PATCH v2 3/3] ARM: dts: imx7-colibri: " Hiago De Franco
2024-10-10 17:42 ` [PATCH v2 0/3] ARM: dts: imx[6qdl|7]-[apalis|colibri]: Update audio card names Hiago De Franco
2024-10-22  1:19 ` Shawn Guo

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).