Re: [PATCH v2 1/2] dt-bindings: arm,scmi: Do not use clocks for SCMI performance domains

From: Sudeep Holla
Date: Fri Oct 23 2020 - 09:59:12 EST


On Fri, Oct 23, 2020 at 08:24:06AM -0500, Rob Herring wrote:
> On Thu, Oct 22, 2020 at 1:22 PM Sudeep Holla <sudeep.holla@xxxxxxx> wrote:
> >
> > On Wed, Oct 21, 2020 at 07:31:03PM +0100, Sudeep Holla wrote:
> > > Commit dd461cd9183f ("opp: Allow dev_pm_opp_get_opp_table() to return
> > > -EPROBE_DEFER") handles -EPROBE_DEFER for the clock/interconnects within
> > > _allocate_opp_table() which is called from dev_pm_opp_add and it
> > > now propagates the error back to the caller.
> > >
> > > SCMI performance domain re-used clock bindings to keep it simple. However
> > > with the above mentioned change, if clock property is present in a device
> > > node, opps can't be added until clk_get succeeds. So in order to fix the
> > > issue, we can register dummy clocks which is completely ugly.
> > >
> > > Since there are no upstream users for the SCMI performance domain clock
> > > bindings, let us introduce separate performance domain bindings for the
> > > same.
> > >
> > > Signed-off-by: Sudeep Holla <sudeep.holla@xxxxxxx>
> > > ---
> > > .../devicetree/bindings/arm/arm,scmi.txt | 19 ++++++++++++++++---
> > > 1 file changed, 16 insertions(+), 3 deletions(-)
> > >
> > > v1[1]->v2:
> > > - Changed the generic #perf-domain-cells to more SCMI specific
> > > property #arm,scmi-perf-domain-cells
> > >
> >
> > Is more specific #arm,scmi-perf-domain-cells acceptable ?
> > Sorry for the rush, but this fixes SCMI cpufreq which is broken after
> > commit dd461cd9183f ("opp: Allow dev_pm_opp_get_opp_table() to return
> > -EPROBE_DEFER")
>
> If you are in a rush, you'd better go the dummy clock route. We should
> get this binding right and I think that means something common, not
> SCMI specific.
>

Ah OK, I assumed you wanted to make it SCMI specific. It makes sense to
have something generic like clocks for OPP domains.

There was discussion on the other thread to use empty OPP list for
firmware discoverable OPP list. Random thought here is to add domain ID
to OPP binding and use that ?

--
Regards,
Sudeep