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?