On 05/11/2025 01:55, Dmitry Baryshkov wrote: > On Tue, Nov 04, 2025 at 11:16:10PM +0100, David Heidelberg via B4 Relay wrote: >> From: David Heidelberg <[email protected]> >> >> sofef00 is name of the DDIC, it doesn't contain name of the panel used. >> The DDIC is also paired with other panels, so make clear which panel is >> used. >> >> cosmetic: sort the node. >> >> Signed-off-by: David Heidelberg <[email protected]> >> --- >> arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dts | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dts >> b/arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dts >> index a259eb9d45ae0..8aead6dc25e00 100644 >> --- a/arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dts >> +++ b/arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dts >> @@ -31,9 +31,9 @@ battery: battery { >> }; >> >> &display_panel { >> - status = "okay"; >> + compatible = "samsung,sofef00-ams628nw01", "samsung,sofef00"; > > This breaks the ABI. Please describe, why it is sensible. Other patches > should describe whether the old DTs will continue to work or not. > >> >> - compatible = "samsung,sofef00";
If I read the hunk correctly, I do not see any ABI break here. What exactly stops working or is negatively affected that it is called an ABI break? Best regards, Krzysztof
