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.
It could be removed from the driver in the future if there is another
variant of the panel added or we decide to cleanup.
It's still not entirely clear to what extent we care about DT/kernel
backwards compatibility though.
Best regards,
Krzysztof
