[PATCH 1/1] arm64: dts: imx93: Fix LPUART compatible list

Sherry Sun sherry.sun at nxp.com
Wed Jul 26 05:28:22 PDT 2023



> -----Original Message-----
> From: Alexander Stein <alexander.stein at ew.tq-group.com>
> Sent: 2023年7月26日 19:11
> To: Rob Herring <robh+dt at kernel.org>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt at linaro.org>; Conor Dooley <conor+dt at kernel.org>;
> Shawn Guo <shawnguo at kernel.org>; Sascha Hauer
> <s.hauer at pengutronix.de>; Fabio Estevam <festevam at gmail.com>; Peng
> Fan <peng.fan at nxp.com>
> Cc: Alexander Stein <alexander.stein at ew.tq-group.com>; Pengutronix Kernel
> Team <kernel at pengutronix.de>; dl-linux-imx <linux-imx at nxp.com>;
> devicetree at vger.kernel.org; linux-arm-kernel at lists.infradead.org
> Subject: [PATCH 1/1] arm64: dts: imx93: Fix LPUART compatible list
> 
> The compatible list for imx93 LPUART does not include imx8ulp-lpuart variant.
> Drop it according to bindings.

Hi Alexander, no, imx93 should include the imx8ulp-lpuart compatible, please update your dt bindings.
https://lore.kernel.org/all/20230705015602.29569-4-sherry.sun@nxp.com/
This patch now in the tty git tree (tty-next branch).

Best Regards
Sherry

> 
> Signed-off-by: Alexander Stein <alexander.stein at ew.tq-group.com>
> ---
> This fixes warnings like:
> arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dtb:
> serial at 44390000: compatible: 'oneOf' conditional failed, one must be fixed:
>         ['fsl,imx93-lpuart', 'fsl,imx8ulp-lpuart', 'fsl,imx7ulp-lpuart'] is too long
>         'fsl,imx93-lpuart' is not one of ['fsl,vf610-lpuart', 'fsl,ls1021a-lpuart',
> 'fsl,ls1028a-lpuart', 'fsl,imx7ulp-lpuart', 'fsl,imx8qxp-lpuart', 'fsl,imxrt1050-
> lpuart']
>         'fsl,imx93-lpuart' is not one of ['fsl,imx8qm-lpuart', 'fsl,imx8dxl-lpuart']
>         'fsl,imxrt1050-lpuart' was expected
>         'fsl,imx7ulp-lpuart' was expected
>         'fsl,imx8qxp-lpuart' was expected
>         'fsl,imxrt1170-lpuart' was expected
>         from schema $id:
> https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevicet
> ree.org%2Fschemas%2Fserial%2Ffsl-
> lpuart.yaml%23&data=05%7C01%7Csherry.sun%40nxp.com%7C80432b48b2
> d848c100e908db8dc902a2%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7
> C0%7C638259666697434989%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4w
> LjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C
> %7C%7C&sdata=mHorQA%2BLsCoGhPtd8SqV%2B45xT1irTLkI6qKCKcqc4BI%3
> D&reserved=0
> arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dtb:
> serial at 44390000: Unevaluated properties are not allowed ('compatible' was
> unexpected)
>         from schema $id:
> https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevicet
> ree.org%2Fschemas%2Fserial%2Ffsl-
> lpuart.yaml%23&data=05%7C01%7Csherry.sun%40nxp.com%7C80432b48b2
> d848c100e908db8dc902a2%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7
> C0%7C638259666697434989%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4w
> LjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C
> %7C%7C&sdata=mHorQA%2BLsCoGhPtd8SqV%2B45xT1irTLkI6qKCKcqc4BI%3
> D&reserved=0
> 
>  arch/arm64/boot/dts/freescale/imx93.dtsi | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx93.dtsi
> b/arch/arm64/boot/dts/freescale/imx93.dtsi
> index 4ec9df78f2050..5ba92c9be78ba 100644
> --- a/arch/arm64/boot/dts/freescale/imx93.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx93.dtsi
> @@ -291,7 +291,7 @@ lpspi2: spi at 44370000 {
>  			};
> 
>  			lpuart1: serial at 44380000 {
> -				compatible = "fsl,imx93-lpuart", "fsl,imx8ulp-
> lpuart", "fsl,imx7ulp-lpuart";
> +				compatible = "fsl,imx93-lpuart", "fsl,imx7ulp-
> lpuart";
>  				reg = <0x44380000 0x1000>;
>  				interrupts = <GIC_SPI 19
> IRQ_TYPE_LEVEL_HIGH>;
>  				clocks = <&clk IMX93_CLK_LPUART1_GATE>;
> @@ -300,7 +300,7 @@ lpuart1: serial at 44380000 {
>  			};
> 
>  			lpuart2: serial at 44390000 {
> -				compatible = "fsl,imx93-lpuart", "fsl,imx8ulp-
> lpuart", "fsl,imx7ulp-lpuart";
> +				compatible = "fsl,imx93-lpuart", "fsl,imx7ulp-
> lpuart";
>  				reg = <0x44390000 0x1000>;
>  				interrupts = <GIC_SPI 20
> IRQ_TYPE_LEVEL_HIGH>;
>  				clocks = <&clk IMX93_CLK_LPUART2_GATE>;
> @@ -545,7 +545,7 @@ lpspi4: spi at 42560000 {
>  			};
> 
>  			lpuart3: serial at 42570000 {
> -				compatible = "fsl,imx93-lpuart", "fsl,imx8ulp-
> lpuart", "fsl,imx7ulp-lpuart";
> +				compatible = "fsl,imx93-lpuart", "fsl,imx7ulp-
> lpuart";
>  				reg = <0x42570000 0x1000>;
>  				interrupts = <GIC_SPI 68
> IRQ_TYPE_LEVEL_HIGH>;
>  				clocks = <&clk IMX93_CLK_LPUART3_GATE>;
> @@ -554,7 +554,7 @@ lpuart3: serial at 42570000 {
>  			};
> 
>  			lpuart4: serial at 42580000 {
> -				compatible = "fsl,imx93-lpuart", "fsl,imx8ulp-
> lpuart", "fsl,imx7ulp-lpuart";
> +				compatible = "fsl,imx93-lpuart", "fsl,imx7ulp-
> lpuart";
>  				reg = <0x42580000 0x1000>;
>  				interrupts = <GIC_SPI 69
> IRQ_TYPE_LEVEL_HIGH>;
>  				clocks = <&clk IMX93_CLK_LPUART4_GATE>;
> @@ -563,7 +563,7 @@ lpuart4: serial at 42580000 {
>  			};
> 
>  			lpuart5: serial at 42590000 {
> -				compatible = "fsl,imx93-lpuart", "fsl,imx8ulp-
> lpuart", "fsl,imx7ulp-lpuart";
> +				compatible = "fsl,imx93-lpuart", "fsl,imx7ulp-
> lpuart";
>  				reg = <0x42590000 0x1000>;
>  				interrupts = <GIC_SPI 70
> IRQ_TYPE_LEVEL_HIGH>;
>  				clocks = <&clk IMX93_CLK_LPUART5_GATE>;
> @@ -572,7 +572,7 @@ lpuart5: serial at 42590000 {
>  			};
> 
>  			lpuart6: serial at 425a0000 {
> -				compatible = "fsl,imx93-lpuart", "fsl,imx8ulp-
> lpuart", "fsl,imx7ulp-lpuart";
> +				compatible = "fsl,imx93-lpuart", "fsl,imx7ulp-
> lpuart";
>  				reg = <0x425a0000 0x1000>;
>  				interrupts = <GIC_SPI 71
> IRQ_TYPE_LEVEL_HIGH>;
>  				clocks = <&clk IMX93_CLK_LPUART6_GATE>;
> @@ -610,7 +610,7 @@ flexspi1: spi at 425e0000 {
>  			};
> 
>  			lpuart7: serial at 42690000 {
> -				compatible = "fsl,imx93-lpuart", "fsl,imx8ulp-
> lpuart", "fsl,imx7ulp-lpuart";
> +				compatible = "fsl,imx93-lpuart", "fsl,imx7ulp-
> lpuart";
>  				reg = <0x42690000 0x1000>;
>  				interrupts = <GIC_SPI 210
> IRQ_TYPE_LEVEL_HIGH>;
>  				clocks = <&clk IMX93_CLK_LPUART7_GATE>;
> @@ -619,7 +619,7 @@ lpuart7: serial at 42690000 {
>  			};
> 
>  			lpuart8: serial at 426a0000 {
> -				compatible = "fsl,imx93-lpuart", "fsl,imx8ulp-
> lpuart", "fsl,imx7ulp-lpuart";
> +				compatible = "fsl,imx93-lpuart", "fsl,imx7ulp-
> lpuart";
>  				reg = <0x426a0000 0x1000>;
>  				interrupts = <GIC_SPI 211
> IRQ_TYPE_LEVEL_HIGH>;
>  				clocks = <&clk IMX93_CLK_LPUART8_GATE>;
> --
> 2.34.1



More information about the linux-arm-kernel mailing list