| Message ID | 20220111133937.1099917-1-martin.kepplinger@puri.sm (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2,1/2] dt-binding: media: hynix,hi846: use $defs/port-base port description | expand |
Hi Martin, Thank you for the patch. On Tue, Jan 11, 2022 at 02:39:36PM +0100, Martin Kepplinger wrote: > This is supposed to fix "make dt_binding_check": Maybe s/is supposed to fix/fixes/ ? :-) No need to post a new version just for that, I think it can be changed when applying the patch. Sakari ? Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > 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> > --- > > > revision history > ---------------- > v2: thank you, Laurent > * add unevaluatedProperties: false > v1: > https://lore.kernel.org/linux-media/20220110123804.377944-1-martin.kepplinger@puri.sm/ > > > Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml b/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml > index 85a8877c2f38..5d0fc18a2ea2 100644 > --- a/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml > +++ b/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml > @@ -49,7 +49,8 @@ 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 > + unevaluatedProperties: false > > properties: > endpoint:
On Tue, 11 Jan 2022 14:39:36 +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> > --- > > > revision history > ---------------- > v2: thank you, Laurent > * add unevaluatedProperties: false > v1: > https://lore.kernel.org/linux-media/20220110123804.377944-1-martin.kepplinger@puri.sm/ > > > Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > Reviewed-by: Rob Herring <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml b/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml index 85a8877c2f38..5d0fc18a2ea2 100644 --- a/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml +++ b/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml @@ -49,7 +49,8 @@ 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 + unevaluatedProperties: false 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> --- revision history ---------------- v2: thank you, Laurent * add unevaluatedProperties: false v1: https://lore.kernel.org/linux-media/20220110123804.377944-1-martin.kepplinger@puri.sm/ Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)