On 01.11.24 14:54, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@xxxxxxx>
>
> imx6sx.dtsi has the following lcdif entries:
>
> compatible = "fsl,imx6sx-lcdif", "fsl,imx28-lcdif";
>
> This causes the following dt-schema warning:
>
> ['fsl,imx6sx-lcdif', 'fsl,imx28-lcdif'] is too long
>
> To keep DT compatibility, document 'fsl,imx28-lcdif' as a possible
> 'fsl,imx6sx-lcdif' fallback.
>
> Signed-off-by: Fabio Estevam <festevam@xxxxxxx>
Reviewed-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx>
> ---
> Changes since v3:
> - Also update the example.
>
> Documentation/devicetree/bindings/display/fsl,lcdif.yaml | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/fsl,lcdif.yaml b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml
> index 8e3a98aeec32..aa3204b6aff9 100644
> --- a/Documentation/devicetree/bindings/display/fsl,lcdif.yaml
> +++ b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml
> @@ -19,7 +19,6 @@ properties:
> - enum:
> - fsl,imx23-lcdif
> - fsl,imx28-lcdif
> - - fsl,imx6sx-lcdif
> - fsl,imx8mp-lcdif
> - fsl,imx93-lcdif
> - items:
> @@ -32,6 +31,10 @@ properties:
> - fsl,imx8mn-lcdif
> - fsl,imx8mq-lcdif
> - const: fsl,imx6sx-lcdif
> + - items:
> + - enum:
> + - fsl,imx6sx-lcdif
> + - const: fsl,imx28-lcdif
>
> reg:
> maxItems: 1
> @@ -182,7 +185,7 @@ examples:
> #include <dt-bindings/interrupt-controller/arm-gic.h>
>
> display-controller@2220000 {
> - compatible = "fsl,imx6sx-lcdif";
> + compatible = "fsl,imx6sx-lcdif", "fsl,imx28-lcdif";
> reg = <0x02220000 0x4000>;
> interrupts = <GIC_SPI 5 IRQ_TYPE_LEVEL_HIGH>;
> clocks = <&clks IMX6SX_CLK_LCDIF1_PIX>,
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
