On Thu, Oct 19, 2023 at 04:12:25PM +0300, Dmitry Baryshkov wrote: > On Thu, 19 Oct 2023 at 11:42, Varadarajan Narayanan > <quic_varada@xxxxxxxxxxx> wrote: > > > > The config IPQ_APSS_6018 should depend on QCOM_SMEM, to > > avoid the following error. > > Which error? Sorry. I assumed that it would refer to the info mentioned in the 'Reported-by' & 'Closes' tags. Have included the error message in the commit log, addressed other comments and posted v5. Kindly take a look. Thanks Varada > > Fixes: 5e77b4ef1b19 ("clk: qcom: Add ipq6018 apss clock controller") > > Reported-by: kernel test robot <yujie.liu@xxxxxxxxx> > > Closes: https://lore.kernel.org/r/202310181650.g8THtfsm-lkp@xxxxxxxxx/ > > Signed-off-by: Varadarajan Narayanan <quic_varada@xxxxxxxxxxx> > > --- > > drivers/clk/qcom/Kconfig | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig > > index 3194c8b..ad1acd9 100644 > > --- a/drivers/clk/qcom/Kconfig > > +++ b/drivers/clk/qcom/Kconfig > > @@ -131,6 +131,7 @@ config IPQ_APSS_6018 > > tristate "IPQ APSS Clock Controller" > > select IPQ_APSS_PLL > > depends on QCOM_APCS_IPC || COMPILE_TEST > > + depends on QCOM_SMEM > > help > > Support for APSS clock controller on IPQ platforms. The > > APSS clock controller manages the Mux and enable block that feeds the > > -- > > 2.7.4 > > > > > -- > With best wishes > Dmitry