Re: [PATCH v4] dt-bindings: adv748x: Convert bindings to json-schema

[Rob Herring <robh@kernel.org>]

On Thu, 28 Oct 2021 11:37:49 +0200, Niklas Söderlund wrote:
> Convert ADV748X analog video decoder documentation to json-schema.
> 
> While converting the bindings extend it to enforce that all port@n nodes
> shall be encapsulated inside a ports node. This change does not have an
> effect on drivers parsing the ports@n nodes.
> 
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
> ---
> * Changes since v3
> - For 'interrupt-names' use the pattern
>     minItems: 1
>     maxItems: 3
>     items:
>       enum: [ intrq1, intrq2, intrq3 ]
> 
>  Instead of an items list with three '- enum: [ intrq1, intrq2, intrq3 ]'
>  rows.
> 
> * Changes since v2
> - Add adv748x.yaml to MAINTAINERS.
> - Update commit message.
> - Add myself to under the maintainers section after talking with Kieran.
> - Split reg in examples in two lines to match reg-names.
> 
> * Changes since v1
> - Update commit message to mention the added ports node.
> 
> Hello,
> 
> This conversion revealed a problem with the Renesas DTSI files for the
> adv7482 nodes. A fix for that have been submitted in a separate patch,
> 
>     [PATCH] arm64: dts: renesas: Add ports node to all adv7482 nodes
> 
> Kind Regards,
> Niklas Söderlund
> ---
>  .../devicetree/bindings/media/i2c/adv748x.txt | 116 ----------
>  .../bindings/media/i2c/adv748x.yaml           | 212 ++++++++++++++++++
>  MAINTAINERS                                   |   1 +
>  3 files changed, 213 insertions(+), 116 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/media/i2c/adv748x.txt
>  create mode 100644 Documentation/devicetree/bindings/media/i2c/adv748x.yaml
> 

Reviewed-by: Rob Herring <robh@kernel.org>