[PATCH v5 1/4] dt-bindings: display: bridge: ldb: Adjust imx6sx entries

Marek Vasut marex at denx.de
Mon May 22 06:12:46 PDT 2023


On 5/22/23 14:51, Fabio Estevam wrote:
> From: Fabio Estevam <festevam at denx.de>
> 
> On the i.MX6SX there is a single entry for 'reg' and 'reg-names', so add
> some logic to reflect that.
> 
> Signed-off-by: Fabio Estevam <festevam at denx.de>
> ---
> Changes since v4:
> - Replaced items: with const: for the single item (Conor)
> 
>   .../bindings/display/bridge/fsl,ldb.yaml      | 26 ++++++++++++++++---
>   1 file changed, 23 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
> index 07388bf2b90d..d97064140547 100644
> --- a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
> @@ -28,12 +28,12 @@ properties:
>       const: ldb
>   
>     reg:
> +    minItems: 1
>       maxItems: 2
>   
>     reg-names:
> -    items:
> -      - const: ldb
> -      - const: lvds
> +    minItems: 1
> +    maxItems: 2
>   
>     ports:
>       $ref: /schemas/graph.yaml#/properties/ports
> @@ -74,6 +74,26 @@ allOf:
>             properties:
>               port at 2: false
>   
> +  - if:
> +      properties:
> +        compatible:
> +          contains:
> +            const: fsl,imx6sx-ldb
> +    then:
> +      properties:
> +        reg:
> +          maxItems: 1
> +        reg-names:
> +           const: ldb

Can we drop the reg-names on MX6SX if we have reg maxItems=1 here ?

I think we can, but we need to retain that on the newer iMXes which have 
reg maxItems=2



More information about the linux-arm-kernel mailing list