[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aBCWdpk2HXPaJPlH@linaro.org>
Date: Tue, 29 Apr 2025 12:05:58 +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 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.
I can squash them if the consensus is that it's not backporting.
>
> Either way:
>
> Reviewed-by: Johan Hovold <johan+linaro@...nel.org>
Thanks!
Abel
Powered by blists - more mailing lists