Re: [PATCH v6] dt-bindings: drm/msm/gpu: convert to YAML

[David Heidelberg]

Hello Rob,

can this patch can go in?

Thank you
David


On Mon, Oct 18 2021 at 08:48:23 -0500, Rob Herring <robh@xxxxxxxxxx> 
wrote:
> On Sun, 17 Oct 2021 16:43:50 +0200, David Heidelberg wrote:
> >  Conversion of text binding for Adreno GPU to the YAML format.
> >
> >  Signed-off-by: David Heidelberg <david@xxxxxxx>
> >  ---
> >  v2:
> >    - added compatbile description from Rob Clark
> >    - dropped reg description
> >    - reg numbers increased to 3 (since we also have uncommon cx_dbgc)
> >    - specified interconnect-names items range
> >    - defined zap-shader as an object and added it's properties
> >    - enforce 0 clocks for Andreno >= 6xx since it's defined in GMU 
> > node
> >
> >  v3:
> >   - fix patterns (add backslash before dot)
> >   - add additional clocks
> >   - adreno@ -> gpu@
> >   - add few const and change enum to anyOf
> >   - added clock & clock-names placeholder in non-conditional part
> >   (description)
> >
> >  v4:
> >   - clock & clock-names - true,false instead maxItems
> >   - impl. #stream-id-cells, nvmem-cell-names, nvmem-cells
> >   - dropped requirement on firmware-name in zap_shader
> >
> >  v5:
> >   - maxOtems -> maxItems typo fix
> >
> >  v6:
> >   - enum -> const to fix ordering
> >   - opp-table is object
> >   - spacing
> >   - drop undocumented and unused #stream-id-cells
> >
> >   .../devicetree/bindings/display/msm/gpu.txt   | 157 ----------
> >   .../devicetree/bindings/display/msm/gpu.yaml  | 288 
> > ++++++++++++++++++
> >   2 files changed, 288 insertions(+), 157 deletions(-)
> >   delete mode 100644 
> > Documentation/devicetree/bindings/display/msm/gpu.txt
> >   create mode 100644 
> > Documentation/devicetree/bindings/display/msm/gpu.yaml
>
> Reviewed-by: Rob Herring <robh@xxxxxxxxxx>