[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aBCZ3RpfuK0DsoQ1@linaro.org>
Date: Tue, 29 Apr 2025 12:20:29 +0300
From: Abel Vesa <abel.vesa@...aro.org>
To: Johan Hovold <johan@...nel.org>
Cc: Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konradybcio@...nel.org>,
Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Sibi Sankar <quic_sibis@...cinc.com>,
Rajendra Nayak <quic_rjendra@...cinc.com>,
Xilin Wu <wuxilin123@...il.com>,
Jens Glathe <jens.glathe@...schoolsolutions.biz>,
Srinivas Kandagatla <srini@...nel.org>,
Johan Hovold <johan+linaro@...nel.org>,
linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org,
Konrad Dybcio <konrad.dybcio@....qualcomm.com>,
Konrad Dybcio <quic_kdybcio@...cinc.com>
Subject: Re: [PATCH 0/7] arm64: dts: qcom: x1e80100-*: Drop useless DP3
compatible override
On 25-04-29 11:13:04, Johan Hovold wrote:
> On Tue, Apr 29, 2025 at 12:05:58PM +0300, Abel Vesa wrote:
> > On 25-04-29 10:57:44, Johan Hovold wrote:
> > > On Tue, Apr 29, 2025 at 10:42:28AM +0300, Abel Vesa wrote:
> > > > It all started with the support for CRD back when we had different
> > > > compatibles for eDP and DP. Meanwhile, that has been sorted out and it
> > > > is now figured out at runtime while using only the DP compatible.
> > > >
> > > > It's almost funny how this got copied over from CRD and spread to all
> > > > X Elite platforms.
> > > >
> > > > TBH, the best reason to drop it ASAP is to make sure this doesn't spread
> > > > beyond X Elite to newer platforms.
> > > >
> > > > Functionally nothing changes.
> > > >
> > > > Signed-off-by: Abel Vesa <abel.vesa@...aro.org>
> > > > ---
> > > > Abel Vesa (7):
> > > > arm64: dts: qcom: x1e-crd: Drop useless DP3 compatible override
> > > > arm64: dts: acom: x1e80100-qcp: Drop useless DP3 compatible override
> > > > arm64: dts: qcom: x1e80100-t14s: Drop useless DP3 compatible override
> > > > arm64: dts: qcom: x1e80100-s15: Drop useless DP3 compatible override
> > > > arm64: dts: qcom: x1e80100-hp-x14: Drop useless DP3 compatible override
> > > > arm64: dts: qcom: x1e80100: Drop useless DP3 compatible override
> > > > arm64: dts: qcom: x1e80100-romulus: Drop useless DP3 compatible override
> > >
> > > Since this is essentially a clean up perhaps you should have squashed
> > > these into one patch.
> >
> > I was actually thinking that before sending, but then I decided to add
> > the Fixes tag to each one. Since it's such a trivial worthless cleanup,
> > I wasn't sure if the Fixes tags were worth it either.
>
> Right, since it's not a bug you should probably have skipped the Fixes
> tags too.
>
> > I can squash them if the consensus is that it's not backporting.
>
> We should definitely not backport these as they are not fixing any bugs.
Sure. Will drop Fixes tags and squash in the next version.
>
> Johan
Abel
Powered by blists - more mailing lists