Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: - [Medium] Redefining `endpoint` in `port@2` and `port@3` with `unevaluatedProperties: false` unconditionally rejects valid vendor-specific properties (`ti,lvds-*`), breaking schema validation. --
--- Patch [2]: [PATCH v6 2/3] dt-bindings: display: sn65dsi83: Add output data-lanes property --- commit c8014187b83b236fe065488764ddd649223ae575 Author: Wojciech Dubowik <[email protected]> dt-bindings: display: sn65dsi83: Add output data-lanes property Adds an optional output lvds data lanes property with two allowed values for standard and reversed layouts, informing the driver if the reverse lvds config option needs to be set. > diff --git > a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml > b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml > index e69b6343a8ebb..4000bf0b13709 100644 > --- a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml > @@ -86,10 +86,54 @@ properties: > description: Video port for LVDS Channel-A output (panel or bridge). > $ref: '#/$defs/lvds-port' > > + properties: > + endpoint: > + $ref: /schemas/media/video-interfaces.yaml# > + unevaluatedProperties: false [Severity: Medium] Does this local redefinition of endpoint with unevaluatedProperties: false unconditionally reject valid vendor-specific properties (like ti,lvds-*) and break schema validation? In JSON Schema, unevaluatedProperties only considers properties evaluated within its own local evaluation path, so it won't see properties evaluated by the parallel $ref: '#/$defs/lvds-port'. Could data-lanes be added directly to the #/$defs/lvds-port/properties/endpoint definition instead? [ ... ] > port@3: > description: Video port for LVDS Channel-B output (panel or bridge). > $ref: '#/$defs/lvds-port' > > + properties: > + endpoint: > + $ref: /schemas/media/video-interfaces.yaml# > + unevaluatedProperties: false [Severity: Medium] Will this local redefinition also mask properties evaluated by $defs/lvds-port and cause dt_binding_check to falsely report errors for compliant device trees? -- Sashiko AI review ยท https://sashiko.dev/#/patchset/[email protected]?part=2
