Re: [PATCH 4/9] arm64: dts: imx: add fallback compatible string fsl,imx8mm

Frank Li
2024-08-05T16:14:20Z

On Mon, Aug 05, 2024 at 11:49:46AM -0400, Frank Li wrote:
> Add fallback compatible string fsl,imx8mm to fix below warning
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtb: /: compatible: 'oneOf' conditional failed, one must be fixed:
> ['gw,imx8mm-gw72xx-0x'] is too short
>
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

I just saw Fabio's patch merge into shawn's dt64.

f384d2828f0d5 arm64: dts: imx8mm-venice-gw72xx-0x: Remove compatible from dtso

This is not needed at all. Please skip this one.

Frank Li

> ---
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso | 2 +-
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso | 2 +-
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso | 2 +-
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso | 2 +-
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso | 2 +-
> arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso | 2 +-
> 6 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso
> index f6ad1a4b8b665..6930b2786afc2 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs232-rts.dtso
> @@ -16,7 +16,7 @@
> /plugin/;
>
> &{/} {
> - compatible = "gw,imx8mm-gw72xx-0x";
> + compatible = "gw,imx8mm-gw72xx-0x", "fsl,imx8mm";
> };
>
> &gpio4 {
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso
> index c3cd9f2b0db34..a2d338ec714c3 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs422.dtso
> @@ -19,7 +19,7 @@
> /plugin/;
>
> &{/} {
> - compatible = "gw,imx8mm-gw72xx-0x";
> + compatible = "gw,imx8mm-gw72xx-0x", "fsl,imx8mm";
> };
>
> &gpio4 {
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso
> index cc0a287226ab8..19fff8473970c 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rs485.dtso
> @@ -19,7 +19,7 @@
> /plugin/;
>
> &{/} {
> - compatible = "gw,imx8mm-gw72xx-0x";
> + compatible = "gw,imx8mm-gw72xx-0x", "fsl,imx8mm";
> };
>
> &gpio4 {
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso
> index 1f8ea20dfafcb..6092d37e0069e 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs232-rts.dtso
> @@ -16,7 +16,7 @@
> /plugin/;
>
> &{/} {
> - compatible = "gw,imx8mm-gw73xx-0x";
> + compatible = "gw,imx8mm-gw73xx-0x", "fsl,imx8mm";
> };
>
> &gpio4 {
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso
> index 3e6404340d529..5e7b2d639bff3 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs422.dtso
> @@ -19,7 +19,7 @@
> /plugin/;
>
> &{/} {
> - compatible = "gw,imx8mm-gw73xx-0x";
> + compatible = "gw,imx8mm-gw73xx-0x", "fsl,imx8mm";
> };
>
> &gpio4 {
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso
> index 2c71ab9854cb3..a1f0e4e85fd7d 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw73xx-0x-rs485.dtso
> @@ -19,7 +19,7 @@
> /plugin/;
>
> &{/} {
> - compatible = "gw,imx8mm-gw73xx-0x";
> + compatible = "gw,imx8mm-gw73xx-0x", "fsl,imx8mm";
> };
>
> &gpio4 {
>
> --
> 2.34.1
>