Re: [PATCH 1/6] dt-bindings: gpu: powervr-rogue: Add PowerVR support for some Renesas GPUs

From: Frank Binns
Date: Thu Mar 07 2024 - 07:41:34 EST


Hi Adam,

On Mon, 2024-02-26 at 21:45 -0600, Adam Ford wrote:
> Update the binding to add support for various Renesas SoC's with PowerVR
> Rogue GX6250 and GX6650 GPUs. These devices only need one clock, so update
> the table to indicate such like what was done for the ti,am62-gpu.
>
> Signed-off-by: Adam Ford <aford173@xxxxxxxxx>
>
> diff --git a/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml b/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml
> index 256e252f8087..7c75104df09f 100644
> --- a/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml
> +++ b/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml
> @@ -14,6 +14,11 @@ properties:
> compatible:
> items:
> - enum:
> + - renesas,r8a774a1-gpu
> + - renesas,r8a774e1-gpu
> + - renesas,r8a77951-gpu
> + - renesas,r8a77960-gpu
> + - renesas,r8a77961-gpu
> - ti,am62-gpu
> - const: img,img-axe # IMG AXE GPU model/revision is fully discoverable

A new set of items should be added for 'img,powervr-series6xt' and the Renesas
models along the lines of [1].

Thanks
Frank

[1]
https://gitlab.freedesktop.org/imagination/linux/-/blob/powervr-next/Documentation/devicetree/bindings/gpu/img,powervr.yaml?ref_type=heads#L16-19

>
> @@ -51,7 +56,13 @@ allOf:
> properties:
> compatible:
> contains:
> - const: ti,am62-gpu
> + enum:
> + - ti,am62-gpu
> + - renesas,r8a774a1-gpu
> + - renesas,r8a774e1-gpu
> + - renesas,r8a77951-gpu
> + - renesas,r8a77960-gpu
> + - renesas,r8a77961-gpu
> then:
> properties:
> clocks: