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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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>





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux