From: David Heidelberg <david@ixit.cz>
Basic description for S6E3FC2X01 DDIC with attached panel AMS641RW.
Samsung AMS641RW is 6.41 inch, 1080x2340 pixels, 19.5:9 ratio panel
Signed-off-by: David Heidelberg <david@ixit.cz>
---
.../bindings/display/panel/samsung,s6e3fc2x01.yaml | 78 ++++++++++++++++++++++
MAINTAINERS | 5 ++
2 files changed, 83 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6e3fc2x01.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6e3fc2x01.yaml
new file mode 100644
index 0000000000000..e74a96321f4d5
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/samsung,s6e3fc2x01.yaml
@@ -0,0 +1,78 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/panel/samsung,s6e3fc2x01.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Samsung S6E3FC2X01 AMOLED DDIC
+
+description: The S6E3FC2X01 is display driver IC with connected panel.
+
+maintainers:
+ - David Heidelberg <david@ixit.cz>
+
+allOf:
+ - $ref: panel-common.yaml#
+
+properties:
+ compatible:
+ # Samsung 6.41 inch, 1080x2340 pixels, 19.5:9 ratio (~402 ppi density)
+ const: samsung,s6e3fc2x01-ams641rw
+
+ reg:
+ maxItems: 1
+
+ reset-gpios: true
+
+ port: true
+
+ vddio-supply:
+ description: VDD regulator
+
+ vci-supply:
+ description: VCI regulator
+
+ poc-supply:
+ description: POC regulator
+
+required:
+ - compatible
+ - reset-gpios
+ - vddio-supply
+ - vci-supply
+ - poc-supply
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/gpio/gpio.h>
+
+ dsi {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ panel@0 {
+ compatible = "samsung,s6e3fc2x01-ams641rw";
+ reg = <0>;
+
+ vddio-supply = <&vreg_l14a_1p88>;
+ vci-supply = <&s2dos05_buck1>;
+ poc-supply = <&s2dos05_ldo1>;
+
+ te-gpios = <&tlmm 10 GPIO_ACTIVE_HIGH>;
+ reset-gpios = <&tlmm 6 GPIO_ACTIVE_HIGH>;
+
+ pinctrl-names = "default", "sleep";
+ pinctrl-0 = <&sde_dsi_active &sde_te_active_sleep>;
+ pinctrl-1 = <&sde_dsi_suspend &sde_te_active_sleep>;
+
+ port {
+ panel_in: endpoint {
+ remote-endpoint = <&mdss_dsi0_out>;
+ };
+ };
+ };
+ };
+
+...
diff --git a/MAINTAINERS b/MAINTAINERS
index 47fbc5e06808f..e5c5a19f6d710 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8052,6 +8052,11 @@ S: Maintained
F: Documentation/devicetree/bindings/display/panel/samsung,s6d7aa0.yaml
F: drivers/gpu/drm/panel/panel-samsung-s6d7aa0.c
+DRM DRIVER FOR SAMSUNG S6E3FC2X01 DDIC
+M: David Heidelberg <david@ixit.cz>
+S: Maintained
+F: Documentation/devicetree/bindings/display/panel/samsung,s6e3fc2x01.yaml
+
DRM DRIVER FOR SAMSUNG S6E3HA8 PANELS
M: Dzmitry Sankouski <dsankouski@gmail.com>
S: Maintained
--
2.51.0On Wed, Oct 08, 2025 at 04:05:28PM +0200, David Heidelberg via B4 Relay wrote: > From: David Heidelberg <david@ixit.cz> > > Basic description for S6E3FC2X01 DDIC with attached panel AMS641RW. > > Samsung AMS641RW is 6.41 inch, 1080x2340 pixels, 19.5:9 ratio panel > > Signed-off-by: David Heidelberg <david@ixit.cz> > --- > .../bindings/display/panel/samsung,s6e3fc2x01.yaml | 78 ++++++++++++++++++++++ > MAINTAINERS | 5 ++ > 2 files changed, 83 insertions(+) > Please also describe, why it's not enough to use defined compatible, samsung,s6e3fc2x01. Why do we need a separate schema and can't use the panel-simple-dsi.yaml -- With best wishes Dmitry
On 10/8/25 8:57 PM, Dmitry Baryshkov wrote: > On Wed, Oct 08, 2025 at 04:05:28PM +0200, David Heidelberg via B4 Relay wrote: >> From: David Heidelberg <david@ixit.cz> >> >> Basic description for S6E3FC2X01 DDIC with attached panel AMS641RW. >> >> Samsung AMS641RW is 6.41 inch, 1080x2340 pixels, 19.5:9 ratio panel >> >> Signed-off-by: David Heidelberg <david@ixit.cz> >> --- >> .../bindings/display/panel/samsung,s6e3fc2x01.yaml | 78 ++++++++++++++++++++++ >> MAINTAINERS | 5 ++ >> 2 files changed, 83 insertions(+) >> > > Please also describe, why it's not enough to use defined compatible, > samsung,s6e3fc2x01. Why do we need a separate schema and can't use the > panel-simple-dsi.yaml panel-simple works for 'dumb' (perhaps a harsh word for 'made with just the in-spec DCS commands in mind') panels, but Samsungs are widely known to require a ton of vendor magic Perhaps the original change was made with an "oh it just works surely there's no drawbacks possible" attitude, as the panel was left initialized by the bootloader Konrad
On Thu, Oct 09, 2025 at 10:51:31AM +0200, Konrad Dybcio wrote: > On 10/8/25 8:57 PM, Dmitry Baryshkov wrote: > > On Wed, Oct 08, 2025 at 04:05:28PM +0200, David Heidelberg via B4 Relay wrote: > >> From: David Heidelberg <david@ixit.cz> > >> > >> Basic description for S6E3FC2X01 DDIC with attached panel AMS641RW. > >> > >> Samsung AMS641RW is 6.41 inch, 1080x2340 pixels, 19.5:9 ratio panel > >> > >> Signed-off-by: David Heidelberg <david@ixit.cz> > >> --- > >> .../bindings/display/panel/samsung,s6e3fc2x01.yaml | 78 ++++++++++++++++++++++ > >> MAINTAINERS | 5 ++ > >> 2 files changed, 83 insertions(+) > >> > > > > Please also describe, why it's not enough to use defined compatible, > > samsung,s6e3fc2x01. Why do we need a separate schema and can't use the > > panel-simple-dsi.yaml > > panel-simple works for 'dumb' (perhaps a harsh word for 'made with > just the in-spec DCS commands in mind') panels, but Samsungs are > widely known to require a ton of vendor magic The question is about the _schema_. I think it's fine to have a driver for a panel covered by panel-simple-dsi.yaml. > > Perhaps the original change was made with an "oh it just works > surely there's no drawbacks possible" attitude, as the panel > was left initialized by the bootloader -- With best wishes Dmitry
On 09/10/2025 15:21, Dmitry Baryshkov wrote: > On Thu, Oct 09, 2025 at 10:51:31AM +0200, Konrad Dybcio wrote: >> On 10/8/25 8:57 PM, Dmitry Baryshkov wrote: >>> On Wed, Oct 08, 2025 at 04:05:28PM +0200, David Heidelberg via B4 Relay wrote: >>>> From: David Heidelberg <david@ixit.cz> >>>> >>>> Basic description for S6E3FC2X01 DDIC with attached panel AMS641RW. >>>> >>>> Samsung AMS641RW is 6.41 inch, 1080x2340 pixels, 19.5:9 ratio panel >>>> >>>> Signed-off-by: David Heidelberg <david@ixit.cz> >>>> --- >>>> .../bindings/display/panel/samsung,s6e3fc2x01.yaml | 78 ++++++++++++++++++++++ >>>> MAINTAINERS | 5 ++ >>>> 2 files changed, 83 insertions(+) >>>> >>> >>> Please also describe, why it's not enough to use defined compatible, >>> samsung,s6e3fc2x01. Why do we need a separate schema and can't use the >>> panel-simple-dsi.yaml >> >> panel-simple works for 'dumb' (perhaps a harsh word for 'made with >> just the in-spec DCS commands in mind') panels, but Samsungs are >> widely known to require a ton of vendor magic > > The question is about the _schema_. I think it's fine to have a driver > for a panel covered by panel-simple-dsi.yaml. see display/panel/samsung,amoled-mipi-dsi.yaml the OLED display don't fit well, but I wouldn't mind consolidating at some point, but since we know very little (no datasheets), it's hard to do for now. Maybe in the future when there will be more panels schemas, we can find a way to consolidate into one big? > >> >> Perhaps the original change was made with an "oh it just works >> surely there's no drawbacks possible" attitude, as the panel >> was left initialized by the bootloader > -- David Heidelberg
On Thu, Oct 09, 2025 at 03:32:22PM +0200, David Heidelberg wrote: > > > On 09/10/2025 15:21, Dmitry Baryshkov wrote: > > On Thu, Oct 09, 2025 at 10:51:31AM +0200, Konrad Dybcio wrote: > > > On 10/8/25 8:57 PM, Dmitry Baryshkov wrote: > > > > On Wed, Oct 08, 2025 at 04:05:28PM +0200, David Heidelberg via B4 Relay wrote: > > > > > From: David Heidelberg <david@ixit.cz> > > > > > > > > > > Basic description for S6E3FC2X01 DDIC with attached panel AMS641RW. > > > > > > > > > > Samsung AMS641RW is 6.41 inch, 1080x2340 pixels, 19.5:9 ratio panel > > > > > > > > > > Signed-off-by: David Heidelberg <david@ixit.cz> > > > > > --- > > > > > .../bindings/display/panel/samsung,s6e3fc2x01.yaml | 78 ++++++++++++++++++++++ > > > > > MAINTAINERS | 5 ++ > > > > > 2 files changed, 83 insertions(+) > > > > > > > > > > > > > Please also describe, why it's not enough to use defined compatible, > > > > samsung,s6e3fc2x01. Why do we need a separate schema and can't use the > > > > panel-simple-dsi.yaml > > > > > > panel-simple works for 'dumb' (perhaps a harsh word for 'made with > > > just the in-spec DCS commands in mind') panels, but Samsungs are > > > widely known to require a ton of vendor magic > > > > The question is about the _schema_. I think it's fine to have a driver > > for a panel covered by panel-simple-dsi.yaml. > > see display/panel/samsung,amoled-mipi-dsi.yaml > the OLED display don't fit well, but I wouldn't mind consolidating at some > point, but since we know very little (no datasheets), it's hard to do for > now. Maybe in the future when there will be more panels schemas, we can find > a way to consolidate into one big? I'm looking for a simple answer ATM: it doesn't fit panel-simple-dsi.yaml because it needs foo bar baz, which is not a part of that schema. > > > > > > > > > Perhaps the original change was made with an "oh it just works > > > surely there's no drawbacks possible" attitude, as the panel > > > was left initialized by the bootloader -- With best wishes Dmitry
On 09/10/2025 16:26, Dmitry Baryshkov wrote:
> On Thu, Oct 09, 2025 at 03:32:22PM +0200, David Heidelberg wrote:
>>
>>
>> On 09/10/2025 15:21, Dmitry Baryshkov wrote:
>>> On Thu, Oct 09, 2025 at 10:51:31AM +0200, Konrad Dybcio wrote:
>>>> On 10/8/25 8:57 PM, Dmitry Baryshkov wrote:
>>>>> On Wed, Oct 08, 2025 at 04:05:28PM +0200, David Heidelberg via B4 Relay wrote:
>>>>>> From: David Heidelberg <david@ixit.cz>
>>>>>>
>>>>>> Basic description for S6E3FC2X01 DDIC with attached panel AMS641RW.
>>>>>>
>>>>>> Samsung AMS641RW is 6.41 inch, 1080x2340 pixels, 19.5:9 ratio panel
>>>>>>
>>>>>> Signed-off-by: David Heidelberg <david@ixit.cz>
>>>>>> ---
>>>>>> .../bindings/display/panel/samsung,s6e3fc2x01.yaml | 78 ++++++++++++++++++++++
>>>>>> MAINTAINERS | 5 ++
>>>>>> 2 files changed, 83 insertions(+)
>>>>>>
>>>>>
>>>>> Please also describe, why it's not enough to use defined compatible,
>>>>> samsung,s6e3fc2x01. Why do we need a separate schema and can't use the
>>>>> panel-simple-dsi.yaml
>>>>
>>>> panel-simple works for 'dumb' (perhaps a harsh word for 'made with
>>>> just the in-spec DCS commands in mind') panels, but Samsungs are
>>>> widely known to require a ton of vendor magic
>>>
>>> The question is about the _schema_. I think it's fine to have a driver
>>> for a panel covered by panel-simple-dsi.yaml.
>>
>> see display/panel/samsung,amoled-mipi-dsi.yaml
>> the OLED display don't fit well, but I wouldn't mind consolidating at some
>> point, but since we know very little (no datasheets), it's hard to do for
>> now. Maybe in the future when there will be more panels schemas, we can find
>> a way to consolidate into one big?
>
> I'm looking for a simple answer ATM: it doesn't fit
> panel-simple-dsi.yaml because it needs foo bar baz, which is not a part
> of that schema.
v3 will have:
dt-bindings: panel: Add Samsung S6E3FC2X01 DDIC with panel
Basic description for S6E3FC2X01 DDIC with attached panel AMS641RW.
Samsung AMS641RW is 6.41 inch, 1080x2340 pixels, 19.5:9 ratio panel
panel-simple-dsi cannot be used because it's limited to one
power-supply, while we use three.
>
>>
>>>
>>>>
>>>> Perhaps the original change was made with an "oh it just works
>>>> surely there's no drawbacks possible" attitude, as the panel
>>>> was left initialized by the bootloader
>
--
David Heidelberg
On Thu, Oct 09, 2025 at 05:44:46PM +0200, David Heidelberg wrote: > On 09/10/2025 16:26, Dmitry Baryshkov wrote: > > On Thu, Oct 09, 2025 at 03:32:22PM +0200, David Heidelberg wrote: > > > > > > > > > On 09/10/2025 15:21, Dmitry Baryshkov wrote: > > > > On Thu, Oct 09, 2025 at 10:51:31AM +0200, Konrad Dybcio wrote: > > > > > On 10/8/25 8:57 PM, Dmitry Baryshkov wrote: > > > > > > On Wed, Oct 08, 2025 at 04:05:28PM +0200, David Heidelberg via B4 Relay wrote: > > > > > > > From: David Heidelberg <david@ixit.cz> > > > > > > > > > > > > > > Basic description for S6E3FC2X01 DDIC with attached panel AMS641RW. > > > > > > > > > > > > > > Samsung AMS641RW is 6.41 inch, 1080x2340 pixels, 19.5:9 ratio panel > > > > > > > > > > > > > > Signed-off-by: David Heidelberg <david@ixit.cz> > > > > > > > --- > > > > > > > .../bindings/display/panel/samsung,s6e3fc2x01.yaml | 78 ++++++++++++++++++++++ > > > > > > > MAINTAINERS | 5 ++ > > > > > > > 2 files changed, 83 insertions(+) > > > > > > > > > > > > > > > > > > > Please also describe, why it's not enough to use defined compatible, > > > > > > samsung,s6e3fc2x01. Why do we need a separate schema and can't use the > > > > > > panel-simple-dsi.yaml > > > > > > > > > > panel-simple works for 'dumb' (perhaps a harsh word for 'made with > > > > > just the in-spec DCS commands in mind') panels, but Samsungs are > > > > > widely known to require a ton of vendor magic > > > > > > > > The question is about the _schema_. I think it's fine to have a driver > > > > for a panel covered by panel-simple-dsi.yaml. > > > > > > see display/panel/samsung,amoled-mipi-dsi.yaml > > > the OLED display don't fit well, but I wouldn't mind consolidating at some > > > point, but since we know very little (no datasheets), it's hard to do for > > > now. Maybe in the future when there will be more panels schemas, we can find > > > a way to consolidate into one big? > > > > I'm looking for a simple answer ATM: it doesn't fit > > panel-simple-dsi.yaml because it needs foo bar baz, which is not a part > > of that schema. > > v3 will have: > > dt-bindings: panel: Add Samsung S6E3FC2X01 DDIC with panel > > Basic description for S6E3FC2X01 DDIC with attached panel AMS641RW. > > Samsung AMS641RW is 6.41 inch, 1080x2340 pixels, 19.5:9 ratio panel > > panel-simple-dsi cannot be used because it's limited to one > power-supply, while we use three. Perfect, thank you. If the bindings for samsumg,s6e3fc2x01 are incorrect, they should be dropped too. > > > > > Perhaps the original change was made with an "oh it just works > > > > > surely there's no drawbacks possible" attitude, as the panel > > > > > was left initialized by the bootloader -- With best wishes Dmitry
© 2016 - 2026 Red Hat, Inc.