Re: [PATCH v7 2/3] dt-bindings: display: sn65dsi83: Add output data-lanes property

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] The `data-lanes` property requires exactly 4 items, breaking support for 18-bit (3-lane) LVDS configurations.
--

--- Patch [2]: [PATCH v7 2/3] dt-bindings: display: sn65dsi83: Add output data-lanes property ---
commit 9dc2b5a1e0c25c9fb7936d93599a48a919079b3b
Author: Wojciech Dubowik <Wojciech.Dubowik@mt.com>

dt-bindings: display: sn65dsi83: Add output data-lanes property

Add an optional output lvds data lanes property with two allowed values
for default and reversed layout.

> diff --git a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
> index e69b6343a8ebb..0eb3913acc9d4 100644
> --- a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
> @@ -125,6 +125,21 @@ $defs:
>                lanes in microvolts.
>              maxItems: 2
>  
> +          data-lanes:
> +            description:
> +              Array of physical LVDS data lane indexes
> +            oneOf:
> +              - items:
> +                  - const: 1
> +                  - const: 2
> +                  - const: 3
> +                  - const: 4
> +              - items:
> +                  - const: 4
> +                  - const: 3
> +                  - const: 2
> +                  - const: 1

[Severity: Medium]
Will this strict 4-item constraint break validation for 18-bit (3-lane) LVDS
configurations?

In dt-schema, specifying an items list automatically implies minItems equal to
the length of the list unless explicitly overridden. Without a minItems setting
defined here, the schema will reject arrays shorter than 4 elements, such as
data-lanes = <1 2 3> or <4 3 2>.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260728122032.692482-1-wojciech.dubowik@mt.com?part=2