[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240118085551.GQ5185@atomide.com>
Date: Thu, 18 Jan 2024 10:55:51 +0200
From: Tony Lindgren <tony@...mide.com>
To: Andrew Davis <afd@...com>
Cc: Frank Binns <frank.binns@...tec.com>,
Matt Coster <matt.coster@...tec.com>,
"H . Nikolaus Schaller" <hns@...delico.com>,
Adam Ford <aford173@...il.com>,
Ivaylo Dimitrov <ivo.g.dimitrov.75@...il.com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>,
Thomas Zimmermann <tzimmermann@...e.de>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley <conor+dt@...nel.org>, Chen-Yu Tsai <wens@...e.org>,
Jernej Skrabec <jernej.skrabec@...il.com>,
Samuel Holland <samuel@...lland.org>,
BenoƮt Cousson <bcousson@...libre.com>,
Nishanth Menon <nm@...com>, Vignesh Raghavendra <vigneshr@...com>,
Tero Kristo <kristo@...nel.org>,
Paul Cercueil <paul@...pouillou.net>,
dri-devel@...ts.freedesktop.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-sunxi@...ts.linux.dev, linux-omap@...r.kernel.org,
linux-mips@...r.kernel.org
Subject: Re: [PATCH 08/11] ARM: dts: DRA7xx: Add device tree entry for SGX GPU
* Andrew Davis <afd@...com> [240117 15:52]:
> On 1/10/24 2:29 AM, Tony Lindgren wrote:
> > * Andrew Davis <afd@...com> [240109 17:20]:
> > > --- a/arch/arm/boot/dts/ti/omap/dra7.dtsi
> > > +++ b/arch/arm/boot/dts/ti/omap/dra7.dtsi
> > > @@ -850,12 +850,19 @@ target-module@...00000 {
> > > <SYSC_IDLE_SMART>;
> > > ti,sysc-sidle = <SYSC_IDLE_FORCE>,
> > > <SYSC_IDLE_NO>,
> > > - <SYSC_IDLE_SMART>;
> > > + <SYSC_IDLE_SMART>,
> > > + <SYSC_IDLE_SMART_WKUP>;
> >
> > You probably checked this already.. But just in case, can you please
> > confirm this is intentional. The documentation lists the smart wakeup
> > capability bit as reserved for dra7, maybe the documentation is wrong.
> >
>
> It was an intentional change, although I'm not sure it is correct :)
>
> This is how we had it in our "evil vendor tree" for years (back when it
> was hwmod based), so when converting these nodes to use "ti,sysc" I noticed
> this bit was set, but as you point out the documentation disagrees.
>
> I'd rather go with what has worked before, but it doesn't seem to
> break anything either way, so we could also break this change out into
> its own patch if you would prefer.
I agree it's best to stick what is known to work. How about let's add
the related information to the patch description?
Regards,
Tony
Powered by blists - more mailing lists