On 19.06.19 14:44, Marc Gonzalez wrote: > QCOM_A53PLL and QCOM_CLK_APCS_MSM8916 stand out as the only options > built by default. List them in defconfig after dropping the default. > > Reviewed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> > Signed-off-by: Marc Gonzalez <marc.w.gonzalez@xxxxxxx> > --- > arch/arm64/configs/defconfig | 2 ++ > drivers/clk/qcom/Kconfig | 2 -- > 2 files changed, 2 insertions(+), 2 deletions(-) Hi Marc, Sorry, I wasn't very clear. IMHO the defconfig change should be a separate patch and the v2 of this patch is ok as it is. It would just make things easier to merge. Thanks, Georgi > > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig > index 5a8e853833cf..3277944626c2 100644 > --- a/arch/arm64/configs/defconfig > +++ b/arch/arm64/configs/defconfig > @@ -658,6 +658,8 @@ CONFIG_CLK_IMX8MQ=y > CONFIG_CLK_IMX8QXP=y > CONFIG_TI_SCI_CLK=y > CONFIG_COMMON_CLK_QCOM=y > +CONFIG_QCOM_A53PLL=y > +CONFIG_QCOM_CLK_APCS_MSM8916=y > CONFIG_QCOM_CLK_SMD_RPM=y > CONFIG_QCOM_CLK_RPMH=y > CONFIG_IPQ_GCC_8074=y > diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig > index e1ff83cc361e..6461a1aa7325 100644 > --- a/drivers/clk/qcom/Kconfig > +++ b/drivers/clk/qcom/Kconfig > @@ -21,7 +21,6 @@ if COMMON_CLK_QCOM > > config QCOM_A53PLL > tristate "MSM8916 A53 PLL" > - default ARCH_QCOM > help > Support for the A53 PLL on MSM8916 devices. It provides > the CPU with frequencies above 1GHz. > @@ -31,7 +30,6 @@ config QCOM_A53PLL > config QCOM_CLK_APCS_MSM8916 > tristate "MSM8916 APCS Clock Controller" > depends on QCOM_APCS_IPC || COMPILE_TEST > - default ARCH_QCOM > help > Support for the APCS Clock Controller on msm8916 devices. The > APCS is managing the mux and divider which feeds the CPUs. >
