[v9,03/20] dt-bindings: gpu: Add Imagination Technologies PowerVR/IMG GPU

Commit Message

Donald Robson Nov. 22, 2023, 4:34 p.m. UTC

From: Sarah Walker <[e-mail]>

Add the device tree binding documentation for the IMG AXE GPU used in
TI AM62 SoCs.

Co-developed-by: Frank Binns <[e-mail]>
Signed-off-by: Frank Binns <[e-mail]>
Signed-off-by: Sarah Walker <[e-mail]>
Signed-off-by: Donald Robson <[e-mail]>
Reviewed-by: Maxime Ripard <[e-mail]>
Reviewed-by: Linus Walleij <[e-mail]>
Reviewed-by: Conor Dooley <[e-mail]>
---
Changes since v8:
- Updated maintainer

Changes since v7:
- Updated maintainer

Changes since v6:
- Remove unused gpu label from example
- Updated maintainer

Changes since v5:
- Update compatible string & description to match marketing name
- Remove unnecessary clock-names definition in ti,am62-gpu constraints
- Document that GPU revision is discoverable

Changes since v4:
- Add clocks constraint for ti,am62-gpu
- Remove excess address and size cells in example
- Remove interrupt name and add maxItems
- Make property order consistent between dts and bindings doc
- Update example to match dts

Changes since v3:
- Remove oneOf in compatible property
- Remove power-supply (not used on AM62)

Changes since v2:
- Add commit message description
- Remove mt8173-gpu support (not currently supported)
- Drop quotes from $id and $schema
- Remove reg: minItems
- Drop _clk suffixes from clock-names
- Remove operating-points-v2 property and cooling-cells (not currently
  used)
- Add additionalProperties: false
- Remove stray blank line at the end of file

 .../devicetree/bindings/gpu/img,powervr.yaml  | 73 +++++++++++++++++++
 MAINTAINERS                                   |  7 ++
 2 files changed, 80 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/gpu/img,powervr.yaml

Comments

Geert Uytterhoeven Nov. 27, 2023, 8:39 a.m. UTC | #1

Hi Donald,

On Wed, Nov 22, 2023 at 5:36 PM Donald Robson <[e-mail]> wrote:
> From: Sarah Walker <[e-mail]>
>
> Add the device tree binding documentation for the IMG AXE GPU used in
> TI AM62 SoCs.
>
> Co-developed-by: Frank Binns <[e-mail]>
> Signed-off-by: Frank Binns <[e-mail]>
> Signed-off-by: Sarah Walker <[e-mail]>
> Signed-off-by: Donald Robson <[e-mail]>

Thanks for your patch, which is now commit 6a85c3b14728f0d5
("dt-bindings: gpu: Add Imagination Technologies PowerVR/IMG GPU") in
drm-misc/for-linux-next.

> --- /dev/null
> +++ b/Documentation/devicetree/bindings/gpu/img,powervr.yaml

> +properties:
> +  compatible:
> +    items:
> +      - enum:
> +          - ti,am62-gpu
> +      - const: img,img-axe # IMG AXE GPU model/revision is fully discoverable

Why the double "img", and not just "img,axe"?

Gr{oetje,eeting}s,

                        Geert

Donald Robson Nov. 28, 2023, 9:38 a.m. UTC | #2

Hi Geert,

On Mon, 2023-11-27 at 09:39 +0100, Geert Uytterhoeven wrote:
> Hi Donald,
> 
> On Wed, Nov 22, 2023 at 5:36 PM Donald Robson <[e-mail]> wrote:
> > From: Sarah Walker <[e-mail]>
> > 
> > Add the device tree binding documentation for the IMG AXE GPU used in
> > TI AM62 SoCs.
> > 
> > Co-developed-by: Frank Binns <[e-mail]>
> > Signed-off-by: Frank Binns <[e-mail]>
> > Signed-off-by: Sarah Walker <[e-mail]>
> > Signed-off-by: Donald Robson <[e-mail]>
> 
> Thanks for your patch, which is now commit 6a85c3b14728f0d5
> ("dt-bindings: gpu: Add Imagination Technologies PowerVR/IMG GPU") in
> drm-misc/for-linux-next.
> 
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/gpu/img,powervr.yaml
> > +properties:
> > +  compatible:
> > +    items:
> > +      - enum:
> > +          - ti,am62-gpu
> > +      - const: img,img-axe # IMG AXE GPU model/revision is fully discoverable
> 
> Why the double "img", and not just "img,axe"?
> 

While the first 'img' is just short for 'Imagination', the second is the product
series name 'IMG-Axe' in lower case.  A quirk of the newer product names, really.
If we add older series they would be 'img,powervr-series[...]'.

Thanks for applying!
Donald

> Gr{oetje,eeting}s,
> 
>                         Geert
>

Patch

diff --git a/Documentation/devicetree/bindings/gpu/img,powervr.yaml b/Documentation/devicetree/bindings/gpu/img,powervr.yaml
new file mode 100644
index 000000000000..a13298f1a182
--- /dev/null
+++ b/Documentation/devicetree/bindings/gpu/img,powervr.yaml
@@ -0,0 +1,73 @@ 
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+# Copyright (c) 2023 Imagination Technologies Ltd.
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/gpu/img,powervr.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Imagination Technologies PowerVR and IMG GPU
+
+maintainers:
+  - Frank Binns <[e-mail]>
+
+properties:
+  compatible:
+    items:
+      - enum:
+          - ti,am62-gpu
+      - const: img,img-axe # IMG AXE GPU model/revision is fully discoverable
+
+  reg:
+    maxItems: 1
+
+  clocks:
+    minItems: 1
+    maxItems: 3
+
+  clock-names:
+    items:
+      - const: core
+      - const: mem
+      - const: sys
+    minItems: 1
+
+  interrupts:
+    maxItems: 1
+
+  power-domains:
+    maxItems: 1
+
+required:
+  - compatible
+  - reg
+  - clocks
+  - clock-names
+  - interrupts
+
+additionalProperties: false
+
+allOf:
+  - if:
+      properties:
+        compatible:
+          contains:
+            const: ti,am62-gpu
+    then:
+      properties:
+        clocks:
+          maxItems: 1
+
+examples:
+  - |
+    #include <dt-bindings/interrupt-controller/irq.h>
+    #include <dt-bindings/interrupt-controller/arm-gic.h>
+    #include <dt-bindings/soc/ti,sci_pm_domain.h>
+
+    gpu@fd00000 {
+        compatible = "ti,am62-gpu", "img,img-axe";
+        reg = <0x0fd00000 0x20000>;
+        clocks = <&k3_clks 187 0>;
+        clock-names = "core";
+        interrupts = <GIC_SPI 86 IRQ_TYPE_LEVEL_HIGH>;
+        power-domains = <&k3_pds 187 TI_SCI_PD_EXCLUSIVE>;
+    };
diff --git a/MAINTAINERS b/MAINTAINERS
index 8a70be8f08ee..59c60abf341e 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -10392,6 +10392,13 @@  IMGTEC IR DECODER DRIVER
 S:	Orphan
 F:	drivers/media/rc/img-ir/
 
+IMGTEC POWERVR DRM DRIVER
+M:	Frank Binns <[e-mail]>
+M:	Donald Robson <[e-mail]>
+M:	Matt Coster <[e-mail]>
+S:	Supported
+F:	Documentation/devicetree/bindings/gpu/img,powervr.yaml
+
 IMON SOUNDGRAPH USB IR RECEIVER
 M:	Sean Young <[e-mail]>
 L:	[e-mail]