Re: [PATCH 0/7] arm64: dts: qcom: x1e80100-*: Drop useless DP3 compatible override

[Johan Hovold <johan@kernel.org>]

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@linaro.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.

Johan