Re: [PATCH 3/3] clk: qcom: add clock controller driver for qca8386/qca8084

[Jie Luo]

On 8/2/2023 3:16 AM, Randy Dunlap wrote:
> On 8/1/23 01:53, Luo Jie wrote:
> > diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig
> > index 263e55d75e3f..a17e8fa5a7e1 100644
> > --- a/drivers/clk/qcom/Kconfig
> > +++ b/drivers/clk/qcom/Kconfig
> > @@ -195,6 +195,14 @@ config IPQ_GCC_9574
> >   	  i2c, USB, SD/eMMC, etc. Select this for the root clock
> >   	  of ipq9574.
> >   
> > +config IPQ_NSSCC_QCA8K
> > +        tristate "QCA8K(QCA8386 or QCA8084) NSS Clock Controller"
> > +        help
>
> The 2 lines above should be indented with one tab (only; no spaces).
>
> > +          Support for NSS(Network SubSystem) clock controller on
>
> The line above should be indented with one tab + 2 spaces (like the following
> 3 lines).
>
> > +	  qca8386/qca8084 chip.
> > +	  Say Y if you want to use network function of switch or PHY
> > +	  function. Select this for the root clock of qca8xxx.
thanks Randy for the review, i will update the patch for this.