Re: [RFC PATCH 4/9] opp: core: Don't warn if required OPP device does not exist

From: Viresh Kumar
Date: Thu Oct 14 2021 - 03:22:49 EST


On 14-10-21, 16:03, Hector Martin wrote:
> On 14/10/2021 15.56, Viresh Kumar wrote:
> > > + /*
> > > + * Attach the CPU device to its genpd domain (if any), to allow OPP
> > > + * dependencies to be satisfied.
> > > + */
> > > + ret = genpd_dev_pm_attach(cpu_dev);
> > > + if (ret <= 0) {
> > > + dev_err(cpu_dev, "Failed to attach CPU device to genpd\n");
> > > + goto out;
> > > + }
> > > +
> >
> > Other platform do this from some other place I think.
> >
> > Ulf, where should this code be moved ? cpu-clk driver ?
> >
>
> I see one driver that does this is drivers/clk/qcom/apcs-sdx55.c (via
> dev_pm_domain_attach).

That may be a good place since you are already adding it and it is related to
CPU clk.

> Though it only does it for CPU#0; we need to do it
> for all CPUs.

Sure.

--
viresh