Re: [PATCH] clk: qcom: msm8916: Add 2 clk options in defconfig

[Bjorn Andersson <bjorn.andersson@linaro.org>]

On Tue 27 Aug 07:48 PDT 2019, Niklas Cassel wrote:

> On Wed, Aug 07, 2019 at 09:29:24PM -0700, Stephen Boyd wrote:
> > Quoting Marc Gonzalez (2019-07-03 07:00:31)
> > > On 24/06/2019 15:57, Amit Kucheria wrote:
> > > 
> > > > On Mon, Jun 24, 2019 at 6:56 PM Marc Gonzalez <marc.w.gonzalez@free.fr> wrote:
> > > >>
> > > >> QCOM_A53PLL and QCOM_CLK_APCS_MSM8916 used to be enabled by default
> > > >> in drivers/clk/qcom/Kconfig. A recent patch changed that by dropping
> > > >> the 'default ARCH_QCOM' directive.
> > > >>
> > > >> Add the two options explicitly in the arm64 defconfig, to avoid
> > > >> functional regressions.
> > > >>
> > > >> Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
> > > > 
> > > > Acked-by: Amit Kucheria <amit.kucheria@linaro.org>
> > > 
> > > Stephen,
> > > 
> > > Can you take the following two patches through the clk tree?
> > > 
> > > [PATCH v2] clk: qcom: msm8916: Don't build by default
> > > [PATCH] clk: qcom: msm8916: Add 2 clk options in defconfig
> > > 
> > 
> > Did Andy pick up this defconfig change? The subject is misleading.
> > defconfig changes should be something like
> 
> Neither Andy nor Bjorn has picked this.
> 
> Could you please pick it up? (possibly with subject fixed up)
> 

Picked up and updated the subject.

Thanks Marc!

Regards,
Bjorn

> > 
> > 	arm64: defconfig: Add qcom clk options
> > 
> > I don't think I need to pick up the defconfig change. It can go through
> > Andy and arm-soc. So I'll just grab the first one and guess it won't
> > break much on the way in.
> >