Re: [PATCH v4 2/4] clk: qcom: Add CMN PLL clock controller driver for IPQ SoC

[Jie Luo <quic_luoj@xxxxxxxxxxx>]

On 10/18/2024 1:40 AM, Stephen Boyd wrote:
> Quoting Jie Luo (2024-10-17 08:35:43)
> > On 10/17/2024 5:37 AM, Stephen Boyd wrote:
> > > Quoting Luo Jie (2024-10-15 07:16:52)
> > > > diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig
> > > > index 30eb8236c9d8..3def659fc5cb 100644
> > > > --- a/drivers/clk/qcom/Kconfig
> > > > +++ b/drivers/clk/qcom/Kconfig
> > > > @@ -190,6 +190,16 @@ config IPQ_APSS_6018
> > > >             Say Y if you want to support CPU frequency scaling on
> > > >             ipq based devices.
> > > >    
> > > > +config IPQ_CMN_PLL
> > > > +       tristate "IPQ CMN PLL Clock Controller"
> > > > +       depends on IPQ_GCC_9574
> > > What is the build dependency?
> > Will remove this dependency. There is no build dependency on
> > IPQ_GCC_9574, but only a functional dependency on the SoC's GCC block
> > since the CMNPLL consumes AHB/SYS clocks from the GCC.
> Ok. It can probably be a select or imply statement then.

Thanks for suggestions. I will enhance the help text to imply the GCC
dependency.