RE: [PATCH 07/15] arm64: defconfig: Enable QorIQ cpufreq driver

[Leo Li]

> -----Original Message-----
> From: Shawn Guo <shawnguo@xxxxxxxxxx>
> Sent: Thursday, March 12, 2020 9:23 AM
> To: Leo Li <leoyang.li@xxxxxxx>
> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 07/15] arm64: defconfig: Enable QorIQ cpufreq driver
> 
> On Wed, Mar 11, 2020 at 06:54:00PM +0000, Leo Li wrote:
> >
> >
> > > -----Original Message-----
> > > From: Shawn Guo <shawnguo@xxxxxxxxxx>
> > > Sent: Wednesday, March 11, 2020 1:12 AM
> > > To: Leo Li <leoyang.li@xxxxxxx>
> > > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx;
> > > linux-kernel@xxxxxxxxxxxxxxx
> > > Subject: Re: [PATCH 07/15] arm64: defconfig: Enable QorIQ cpufreq
> > > driver
> > >
> > > On Mon, Feb 24, 2020 at 05:08:02PM -0600, Li Yang wrote:
> > > > Enables the generic QorIQ cpufreq driver to support frequency
> > > > scaling for various QorIQ SoCs.  Enabled as built-in as it is a core feature.
> > > >
> > > > Signed-off-by: Li Yang <leoyang.li@xxxxxxx>
> > > > ---
> > > >  arch/arm64/configs/defconfig | 2 +-
> > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/arch/arm64/configs/defconfig
> > > > b/arch/arm64/configs/defconfig index e97ef8b944b8..996dc749ea5c
> > > > 100644
> > > > --- a/arch/arm64/configs/defconfig
> > > > +++ b/arch/arm64/configs/defconfig
> > > > @@ -90,6 +90,7 @@ CONFIG_ARM_QCOM_CPUFREQ_NVMEM=y
> > > > CONFIG_ARM_QCOM_CPUFREQ_HW=y
> CONFIG_ARM_RASPBERRYPI_CPUFREQ=m
> > > > CONFIG_ARM_TEGRA186_CPUFREQ=y
> > > > +CONFIG_QORIQ_CPUFREQ=y
> > > >  CONFIG_ARM_SCPI_PROTOCOL=y
> > > >  CONFIG_RASPBERRYPI_FIRMWARE=y
> > > >  CONFIG_INTEL_STRATIX10_SERVICE=y
> > > > @@ -722,7 +723,6 @@ CONFIG_COMMON_CLK_RK808=y
> > > > CONFIG_COMMON_CLK_SCPI=y
> CONFIG_COMMON_CLK_CS2000_CP=y
> > > > CONFIG_COMMON_CLK_S2MPS11=y -CONFIG_CLK_QORIQ=y
> > >
> > > Why is this getting removed?
> >
> > Newly added QORIQ_CPUFREQ selects CLK_QORIQ, so it is removed by
> savedefconfig.
> 
> Note it in the commit log please.

Sure.  Already in v2.

Regards,
Leo