On Tue, Feb 27, 2024 at 8:48 AM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote: > > Hi Adam, > > On Tue, Feb 27, 2024 at 4:46 AM Adam Ford <aford173@xxxxxxxxx> 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> > > Thanks for your patch! > > > --- 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 > > I would add a comment like this: > > - renesas,r8a774a1-gpu # PowerVR Series 6XT GX6650 on RZ/G2M After reading [1], s/Series 6XT/Series6XT/g. [1] "[PATCH 00/11] Device tree support for Imagination Series5 GPU" https://lore.kernel.org/all/20240109171950.31010-1-afd@xxxxxx/ Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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