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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <ZrD6Ue2sP+Fa3WZ2@lizhi-Precision-Tower-5810>
Date: Mon, 5 Aug 2024 12:14:09 -0400
From: Frank Li <Frank.li@....com>
To: Shawn Guo <shawnguo@...nel.org>, Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Sascha Hauer <s.hauer@...gutronix.de>,
	Pengutronix Kernel Team <kernel@...gutronix.de>,
	Fabio Estevam <festevam@...il.com>
Cc: linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org, imx@...ts.linux.dev
Subject: Re: [PATCH 4/9] arm64: dts: imx: add fallback compatible string
 fsl,imx8mm

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