Hi Marek,
On Wed, 15 Oct 2025 at 17:40, Marek Vasut
<[email protected]> wrote:
> Document Imagination Technologies PowerVR Rogue GX6250 BNVC 4.45.2.58
> present in Renesas R-Car R8A77960 M3-W and R8A77961 M3-W+ SoC.
>
> Acked-by: Conor Dooley <[email protected]>
> Signed-off-by: Marek Vasut <[email protected]>
Thanks for the update!
> --- a/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml
> +++ b/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml
> @@ -13,6 +13,12 @@ maintainers:
> properties:
> compatible:
> oneOf:
> + - items:
> + - enum:
> + - renesas,r8a7796-gpu
> + - renesas,r8a77961-gpu
OK.
> + - const: img,img-gx6250
> + - const: img,img-rogue
> - items:
> - enum:
> - ti,am62-gpu
> @@ -146,6 +152,29 @@ allOf:
> clocks:
> maxItems: 1
>
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - renesas,r8a7796-gpu
> + - renesas,r8a77961-gpu
> + then:
> + properties:
> + clocks:
> + minItems: 3
> + clock-names:
> + minItems: 3
> + power-domains:
> + items:
> + - description: Power domain A
> + - description: Power domain B
> + power-domain-names:
> + minItems: 2
I think:
clocks:
minItems: 3
power-domains:
minItems: 2
plus the required below should be sufficient.
As Matt said, the power-domain-names are already specified at the top level.
I guess they should be dropped from the other device-specific
constraints, and their descriptions moved to the top, too (in a separate patch).
> + required:
> + - power-domains
> + - power-domain-names
> +
> examples:
> - |
> #include <dt-bindings/interrupt-controller/irq.h>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
