2024-04-09 09:10:03

by Shengjiu Wang

[permalink] [raw]
Subject: [PATCH v3 2/2] ARM: dts: imx6: exchange fallback and specific compatible string

exchange fallback and specific compatible string for spdif sound card.

Signed-off-by: Shengjiu Wang <[email protected]>
---
arch/arm/boot/dts/nxp/imx/imx6qdl-sabreauto.dtsi | 4 ++--
arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtsi | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-sabreauto.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-sabreauto.dtsi
index 68e97180d33e..51517e27418c 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-sabreauto.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-sabreauto.dtsi
@@ -144,8 +144,8 @@ sound-cs42888 {
};

sound-spdif {
- compatible = "fsl,imx-audio-spdif",
- "fsl,imx-sabreauto-spdif";
+ compatible = "fsl,imx-sabreauto-spdif",
+ "fsl,imx-audio-spdif";
model = "imx-spdif";
spdif-controller = <&spdif>;
spdif-in;
diff --git a/arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtsi b/arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtsi
index c6e85e4a0883..67872c16372a 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6sx-sdb.dtsi
@@ -184,8 +184,8 @@ panel_in: endpoint {
};

sound-spdif {
- compatible = "fsl,imx-audio-spdif",
- "fsl,imx6sx-sdb-spdif";
+ compatible = "fsl,imx6sx-sdb-spdif",
+ "fsl,imx-audio-spdif";
model = "imx-spdif";
spdif-controller = <&spdif>;
spdif-out;
--
2.34.1



2024-04-09 10:02:22

by Fabio Estevam

[permalink] [raw]
Subject: Re: [PATCH v3 2/2] ARM: dts: imx6: exchange fallback and specific compatible string

On Tue, Apr 9, 2024 at 6:08 AM Shengjiu Wang <[email protected]> wrote:
>
> exchange fallback and specific compatible string for spdif sound card.

Ok, but please explain the reason.

2024-04-09 10:08:23

by Fabio Estevam

[permalink] [raw]
Subject: Re: [PATCH v3 2/2] ARM: dts: imx6: exchange fallback and specific compatible string

On Tue, Apr 9, 2024 at 7:02 AM Fabio Estevam <[email protected]> wrote:
>
> On Tue, Apr 9, 2024 at 6:08 AM Shengjiu Wang <[email protected]> wrote:
> >
> > exchange fallback and specific compatible string for spdif sound card.
>
> Ok, but please explain the reason.

I mean, why can't these two .dts files be changed to

compatible = "fsl,imx-audio-spdif";

, like all the others?

2024-04-09 11:22:45

by Shengjiu Wang

[permalink] [raw]
Subject: Re: [PATCH v3 2/2] ARM: dts: imx6: exchange fallback and specific compatible string

On Tue, Apr 9, 2024 at 6:06 PM Fabio Estevam <[email protected]> wrote:
>
> On Tue, Apr 9, 2024 at 7:02 AM Fabio Estevam <[email protected]> wrote:
> >
> > On Tue, Apr 9, 2024 at 6:08 AM Shengjiu Wang <shengjiu.wang@nxpcom> wrote:
> > >
> > > exchange fallback and specific compatible string for spdif sound card.
> >
> > Ok, but please explain the reason.
>
> I mean, why can't these two .dts files be changed to
>
> compatible = "fsl,imx-audio-spdif";
>
> , like all the others?

The specific compatible string should be first place
as the dts rules I think. This patch is just to fix this problem.

For removing the specific compatible string, I think we can
do that. but this change may be out of the scope of this patch
series I think.

So I just change the order of compatible strings

best regards
wang shengjiu

2024-04-11 17:27:51

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH v3 2/2] ARM: dts: imx6: exchange fallback and specific compatible string

On Tue, Apr 09, 2024 at 04:50:44PM +0800, Shengjiu Wang wrote:
> exchange fallback and specific compatible string for spdif sound card.
>
> Signed-off-by: Shengjiu Wang <[email protected]>

Acked-by: Conor Dooley <[email protected]>


Attachments:
(No filename) (253.00 B)
signature.asc (235.00 B)
Download all attachments