Re: [GIT PULL] Qualcomm ARM64 DT updates for 5.5

From: Bjorn Andersson <bjorn.andersson@linaro.org>
To: Olof Johansson <olof@lixom.net>
Cc: Amit Kucheria <amit.kucheria@verdurent.com>,
	Stephan Gerhold <stephan@gerhold.net>,
	Andy Gross <agross@kernel.org>,
	linux-arm-msm <linux-arm-msm@vger.kernel.org>,
	lakml <linux-arm-kernel@lists.infradead.org>
Subject: Re: [GIT PULL] Qualcomm ARM64 DT updates for 5.5
Date: Tue, 10 Dec 2019 22:26:00 -0800	[thread overview]
Message-ID: <20191211062600.GK314059@yoga> (raw)
In-Reply-To: <CAOesGMiiMG8kND=ZGv93t525B4L3ogA7py+3fdw2F6XaZspw_w@mail.gmail.com>

On Tue 10 Dec 13:00 PST 2019, Olof Johansson wrote:

> On Tue, Dec 10, 2019 at 10:17 AM Bjorn Andersson
> <bjorn.andersson@linaro.org> wrote:
> >
> > On Tue 10 Dec 09:36 PST 2019, Amit Kucheria wrote:
> >
> > > On Tue, Dec 10, 2019 at 4:27 PM Stephan Gerhold <stephan@gerhold.net> wrote:
> > > >
> > > > On Thu, Nov 07, 2019 at 09:18:54PM -0600, Andy Gross wrote:
> > > > > On Thu, Nov 07, 2019 at 07:36:03PM +0530, Amit Kucheria wrote:
> > > > > > (Removing arm-soc)
> > > > > >
> > > > > > Hi Andy,
> > > > > >
> > > > > > On Thu, Nov 7, 2019 at 1:04 AM Andy Gross <agross@kernel.org> wrote:
> > > > > > >
> > > > > > > Arnd, Olof, and Kevin,
> > > > > > >
> > > > > > > I have one slight faux paux in this pull request.  A drivers: soc change got
> > > > > > > into my arm64 DTS branch and while it is innocuous, it wasn't easy to fix
> > > > > > > without messing up a lot of people who depend on the SHAs not changing.  So I'm
> > > > > > > sorry for this inclusion.  I'll scrub this better next time.
> > > > > > >
> > > > > > > Andy
> > > > > >
> > > > > > > ----------------------------------------------------------------
> > > > > > > Amit Kucheria (5):
> > > > > > >       arm64: dts: qcs404: thermal: Add interrupt support
> > > > > > >       arm64: dts: msm8998: thermal: Add interrupt support
> > > > > > >       arm64: dts: msm8996: thermal: Add interrupt support
> > > > > > >       arm64: dts: sdm845: thermal: Add interrupt support
> > > > > > >       arm64: dts: msm8916: thermal: Fixup HW ids for cpu sensors
> > > > > >
> > > > > > One of my patches to add interrupt support to msm8916 tsens is missing
> > > > > > here. Specifically this one:
> > > > > > https://patchwork.kernel.org/patch/11201853/
> > > > > >
> > > > > > Will there be a second PR this cycle?
> > > > >
> > > > > I can work up another and throw it on top.
> > > > >
> > > >
> > > > FYI, the patch seems to be still missing in 5.5-rc1.
> > > > tsens now fails to probe on MSM8916 with:
> > > >
> > > >   qcom-tsens 4a9000.thermal-sensor: IRQ uplow not found
> > > >
> > > > Can you queue up the patch as fix for 5.5?
> > >
> > > Indeed. Andy/Bjorn, let me know if you need anything from me to get
> > > this into -rc2.
> > >
> >
> > I'm pulling in some fixes for a rc-pull, so I could add it there.
> >
> > But why are we breaking backwards compatibility with existing dtbs to
> > add a new (optional) feature? Shouldn't there be a rc-fix in the driver
> > for this regression?
> 
> 100% agreed.
> 
> Driver shouldn't require the DT update to continue to function. It
> needs to fill in the previous behavior as default.
> 
> As a matter of fact, based on that I don't want to see that DT patch
> as a fix, please queue it up for 5.6 instead.
> 

I've picked the patch for 5.6

Thanks,
Bjorn