Re: [PATCH] arm64: dts: ti: k3-am62-main: Add GPU device node

From: mripard@xxxxxxxxxx
Date: Tue Dec 05 2023 - 06:43:41 EST


On Tue, Dec 05, 2023 at 11:26:04AM +0000, Donald Robson wrote:
> Hello Alexander,
>
> On Tue, 2023-12-05 at 11:15 +0000, Sverdlin, Alexander wrote:
> > Hi Donald, Sarah et al,
> >
> > thanks for the patch!
> >
> > On Mon, 2023-12-04 at 12:15 +0000, Donald Robson wrote:
> > > From: Sarah Walker <sarah.walker@xxxxxxxxxx>
> > >
> > > Add the Series AXE GPU node to the AM62 device tree.
> > >
> > > Signed-off-by: Sarah Walker <sarah.walker@xxxxxxxxxx>
> > > Signed-off-by: Donald Robson <donald.robson@xxxxxxxxxx>
> >
> > powervr fd00000.gpu: [drm] loaded firmware powervr/rogue_33.15.11.3_v1.fw
> > powervr fd00000.gpu: [drm] FW version v1.0 (build 6503725 OS)
> > [drm] Initialized powervr 1.0.0 20230904 for fd00000.gpu on minor 0
> >
> > In general it allows the driver to be probed on TI AM625 EK, so
> > Tested-by: Alexander Sverdlin <alexander.sverdlin@xxxxxxxxxxx>
>
> Thanks!
>
> >
> > But I'm not sure if you maybe want to disable it by default like
> > other HW blocks for the (potential) boards which don't have video at all.
>
> Nor I. I guess it should still be enabled so it can be used for compute?

It's standalone and doesn't require any additional (or external)
resources that boards could just ignore.

So there's no real reason to disable it, ever.

Maxime

Attachment: signature.asc
Description: PGP signature