Re: [PATCH v3 4/8] clk: qcom: cbf-msm8996: scale CBF clock according to the CPUfreq

[Stephen Boyd <sboyd@kernel.org>]

Quoting Dmitry Baryshkov (2023-01-20 14:53:21)
> On 21/01/2023 00:11, Stephen Boyd wrote:
> > Quoting Dmitry Baryshkov (2023-01-19 22:14:13)
> >> diff --git a/drivers/clk/qcom/clk-cbf-8996.c b/drivers/clk/qcom/clk-cbf-8996.c
> >> index 9cde0e660228..b049b4f7b270 100644
> >> --- a/drivers/clk/qcom/clk-cbf-8996.c
> >> +++ b/drivers/clk/qcom/clk-cbf-8996.c
> >> @@ -225,6 +228,133 @@ static const struct regmap_config cbf_msm8996_regmap_config = {
> >>          .val_format_endian      = REGMAP_ENDIAN_LITTLE,
> >>   };
> >>   
> >> +#ifdef CONFIG_INTERCONNECT
> > 
> > Can you move this driver to drivers/interconnect/ ?
> 
> Only the interconnect part? At some point I considered dropping the 

Yes only the interconnect part. Use auxiliary bus.

> whole CBF mux support and moving the whole driver to 
> drivers/interconnect, but I could not find a good way to use alpha-pll 
> from the interconnect driver. Would you recommend one?

I don't think you need to use alpha-pll code from the interconnect
driver, do you?