Re: [PATCH v8 01/34] opp: Add dev_pm_opp_sync() helper

From: Viresh Kumar
Date: Wed Aug 25 2021 - 22:55:59 EST


On 26-08-21, 08:24, Viresh Kumar wrote:
> On 25-08-21, 18:41, Dmitry Osipenko wrote:
> > Thinking a bit more about this, I got a nicer variant which actually works in all cases for Tegra.
> >
> > Viresh / Ulf, what do you think about this:
>
> This is what I have been suggesting from day 1 :)
>
> https://lore.kernel.org/linux-staging/20210818055849.ybfajzu75ecpdrbn@vireshk-i7/
>
> "
> And if it is all about just syncing the genpd core, then can the
> genpd core do something like what clk framework does? i.e. allow a
> new optional genpd callback, get_performance_state() (just like
> set_performance_state()), which can be called initially by the core
> to get the performance to something other than zero.
> "
>
> Looks good to me :)

When you refresh this stuff, please send only 3-4 patches to update
the core stuff and show an example. Once we finalize with the
interface, you can update all the users. Else this is just noise for
everyone else.

--
viresh