Message ID | 20220110123804.377944-1-martin.kepplinger@puri.sm |
---|---|
State | Superseded, archived |
Headers | show |
Series | [v1,1/2] dt-binding: media: hynix,hi846: use $defs/port-base port description | expand |
Context | Check | Description |
---|---|---|
robh/checkpatch | warning | total: 0 errors, 2 warnings, 8 lines checked |
robh/dtbs-check | success | |
robh/dt-meta-schema | success |
Hi Martin, Thank you for the patch. On Mon, Jan 10, 2022 at 01:38:03PM +0100, Martin Kepplinger wrote: > This is supposed to fix "make dt_binding_check": > > Documentation/devicetree/bindings/media/i2c/hynix,hi846.example.dt.yaml: > camera@20: port:endpoint: Unevaluated properties are not allowed > ('link-frequencies', 'data-lanes' were unexpected) > From schema: Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml > > Fixes: f3ce7200ca18 ("media: dt-bindings: media: document SK Hynix Hi-846 MIPI CSI-2 8M pixel sensor") > Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> > Signed-off-by: Martin Kepplinger <martin.kepplinger@puri.sm> > --- > > Link: > https://lore.kernel.org/linux-media/CAL_JsqKzaZC0A4OwnMyAuEWm2pCcHyQxHyrBVtkiPNUeMDd+oA@mail.gmail.com/ > > Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml b/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml > index 85a8877c2f38..e61255cf3fb9 100644 > --- a/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml > +++ b/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml > @@ -49,7 +49,7 @@ properties: > description: Definition of the regulator used for the VDDD power supply. > > port: > - $ref: /schemas/graph.yaml#/properties/port > + $ref: /schemas/graph.yaml#/$defs/port-base You also need to add unevaluatedProperties: false to reject any property that isn't defined in either port-base or in this schema. Otherwise any extra property in the port node will be accepted. > > properties: > endpoint:
diff --git a/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml b/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml index 85a8877c2f38..e61255cf3fb9 100644 --- a/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml +++ b/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml @@ -49,7 +49,7 @@ properties: description: Definition of the regulator used for the VDDD power supply. port: - $ref: /schemas/graph.yaml#/properties/port + $ref: /schemas/graph.yaml#/$defs/port-base properties: endpoint:
This is supposed to fix "make dt_binding_check": Documentation/devicetree/bindings/media/i2c/hynix,hi846.example.dt.yaml: camera@20: port:endpoint: Unevaluated properties are not allowed ('link-frequencies', 'data-lanes' were unexpected) From schema: Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml Fixes: f3ce7200ca18 ("media: dt-bindings: media: document SK Hynix Hi-846 MIPI CSI-2 8M pixel sensor") Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by: Martin Kepplinger <martin.kepplinger@puri.sm> --- Link: https://lore.kernel.org/linux-media/CAL_JsqKzaZC0A4OwnMyAuEWm2pCcHyQxHyrBVtkiPNUeMDd+oA@mail.gmail.com/ Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)