[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230907-22495ffd0407dacc5cf1cd12@fedora>
Date: Thu, 7 Sep 2023 11:20:50 +0100
From: Conor Dooley <conor@...nel.org>
To: Sarah Walker <sarah.walker@...tec.com>
Cc: dri-devel@...ts.freedesktop.org, frank.binns@...tec.com,
donald.robson@...tec.com, boris.brezillon@...labora.com,
airlied@...il.com, daniel@...ll.ch, robh+dt@...nel.org,
krzysztof.kozlowski+dt@...aro.org, conor+dt@...nel.org,
maarten.lankhorst@...ux.intel.com, mripard@...nel.org,
tzimmermann@...e.de, corbet@....net, christian.koenig@....com,
linus.walleij@...aro.org, matt.coster@...tec.com,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
linux-doc@...r.kernel.org, faith.ekstrand@...labora.com,
afd@...com, hns@...delico.com, matthew.brost@...el.com,
luben.tuikov@....com, dakr@...hat.com
Subject: Re: [PATCH v6 03/20] dt-bindings: gpu: Add Imagination Technologies
PowerVR/IMG GPU
Hey,
On Wed, Sep 06, 2023 at 10:55:25AM +0100, Sarah Walker wrote:
> +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: gpu@...0000 {
This "gpu" label isn't used and can be dropped if you respin.
Otherwise, this seems fine to me,
Reviewed-by: Conor Dooley <conor.dooley@...rochip.com>
Thanks,
Conor.
> + 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>;
> + };
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists