linux-kernel - Re: [PATCH 01/12] dt-bindings: panel: Add Samsung SOFEF00 DDIC with panel

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <dedf91bd-f182-4bf1-a351-6e1c29e90010@kernel.org>
Date: Sat, 8 Nov 2025 12:26:29 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Casey Connolly <casey.connolly@...aro.org>,
 Neil Armstrong <neil.armstrong@...aro.org>, David Heidelberg <david@...t.cz>
Cc: Jessica Zhang <jessica.zhang@....qualcomm.com>,
 Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
 Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
 David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
 Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Thierry Reding
 <thierry.reding@...il.com>, Sam Ravnborg <sam@...nborg.org>,
 Bjorn Andersson <andersson@...nel.org>,
 Konrad Dybcio <konradybcio@...nel.org>, dri-devel@...ts.freedesktop.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 linux-arm-msm@...r.kernel.org, phone-devel@...r.kernel.org
Subject: Re: [PATCH 01/12] dt-bindings: panel: Add Samsung SOFEF00 DDIC with
 panel

On 07/11/2025 14:10, Casey Connolly wrote:
> 
> 
> On 11/7/25 12:27, Krzysztof Kozlowski wrote:
>> On 07/11/2025 12:23, Neil Armstrong wrote:
>>> On 11/6/25 09:48, Krzysztof Kozlowski wrote:
>>>> On Tue, Nov 04, 2025 at 11:16:09PM +0100, David Heidelberg wrote:
>>>>> Basic description for S6E3FC2X01 DDIC with attached panels
>>>>>
>>>>>    - Samsung AMS601NT22 6.01 inch, 1080x2160 pixels, 18:9 ratio
>>>>>    - Samsung AMS628NW01 6.28 inch, 1080x2280 pixels, 19:9 ratio
>>>>>
>>>>> This panel has three supplies, while panel-simple-dsi is limited to one.
>>>>> There is no user of this compatible, nor the compatible make sense.
>>>>
>>>> There are. git grep samsung,sofef00, gives me two users.
>>>
>>> Hmm, on -next I only see a single one:
>>>
>>> $ grep samsung,sofef00 arch/*/boot/dts/ -R
>>> arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dts:  compatible = "samsung,sofef00";
>>>
>>
>> These are the users:
>>
>> arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dts
>> drivers/gpu/drm/panel/panel-samsung-sofef00.c
>>
>>>>
>>>>> Remove it from simple DSI panel definitions.
>>>
>>> Can't you mark is deprecated at first ?
>>
>> Not sure what would be the benefit here. There is no negative ABI impact
>> here.
> 
> If we want to acknowledge the potential scenario where someone is 
> booting a newer kernel with an older DT then we should keep the old 
> compatible in the driver right? And marking it deprecated in bindings 
> would make sense.


We are not talking about removing compatible from the driver. If by any
chance we do, it is not explained here in this commit msg and would be
obviously a no-go.

Best regards,
Krzysztof