Re: [PATCH v3 3/3] clk: qcom: Add Global Clock controller (GCC) driver for SC7180

Stephen Boyd
2019-10-03T16:01:36Z

Quoting Taniya Das (2019-10-03 03:31:15)
> Hi Stephen,
>
> On 10/1/2019 8:08 PM, Stephen Boyd wrote:
> >
> > Why do you want to keep them critical and registered? I'm suggesting
> > that any clk that is marked critical and doesn't have a parent should
> > instead become a register write in probe to turn the clk on.
> >
> Sure, let me do a one-time enable from probe for the clocks which
> doesn't have a parent.
> But I would now have to educate the clients of these clocks to remove
> using them.
>

If anyone is using these clks we can return NULL from the provider for
the specifier so that we indicate there isn't support for them in the
kernel. At least I hope that code path still works given all the recent
changes to clk_get().