Re: [PATCH v7] clk: qcom: Add lpass clock controller driver for SDM845

[Stephen Boyd]

Quoting Taniya Das (2018-10-09 11:27:59)
> Add support for the lpass clock controller found on SDM845 based devices.
> This would allow lpass peripheral loader drivers to control the clocks to
> bring the subsystem out of reset.
> LPASS clocks present on the global clock controller would be registered
> with the clock framework based on the device tree flag. Also do not gate
> these clocks if they are left unused, as the lpass clocks require the
> global clock controller lpass clocks to be enabled before they are
> accessed. During clk_disable_unused as these dependency cannot be
> maintained accessing these LPASS clocks would result in unclocked access.
> 
> Signed-off-by: Taniya Das <tdas@xxxxxxxxxxxxxx>
> ---

I still have open questions about the GCC clks and if they can be left
on all the time or turned on in this clk driver instead. Either way, the
DT binding should specify that this clock controller node uses the ones
provided by GCC. So I'm marking this as 'changes requested' to drop from
my queue.