* [PATCH v1 0/3] Add panels used by Asus Zenbook A14
@ 2025-07-27 16:50 Aleksandrs Vinarskis
2025-07-27 16:50 ` [PATCH v1 1/3] dt-bindings: display: panel: samsung, atna40cu11: document ATNA40CU11 Aleksandrs Vinarskis
` (2 more replies)
0 siblings, 3 replies; 10+ messages in thread
From: Aleksandrs Vinarskis @ 2025-07-27 16:50 UTC (permalink / raw)
To: Douglas Anderson, dri-devel, devicetree, linux-kernel
Cc: Neil Armstrong, Jessica Zhang, David Airlie, Simona Vetter,
Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Aleksandrs Vinarskis
Add x3 panels used by the abovementined device. The laptop comes in
two variants:
* UX3407RA, higher end, FHD+ OLED or WOXGA+ OLED panel
* UX3407QA, lower end, FHD+ OLED or FHD+ LCD panels
At last all three variants were succeffully tested. Device-tree change
to take advantage of Samsung driver for OLED panels [1] will come later
via separate series once this one lands to linux-next.
[1] https://github.com/alexVinarskis/linux-x1e80100-zenbook-a14/blob/master/0007-arm64-dts-qcom-Rework-X1-based-Asus-Zenbook-A14-s-di.patch
Aleksandrs Vinarskis (3):
dt-bindings: display: panel: samsung,atna40cu11: document ATNA40CU11
dt-bindings: display: panel: samsung,atna40ct06: document ATNA40CT06
drm/panel-edp: Add BOE NV140WUM-N64
.../devicetree/bindings/display/panel/samsung,atna33xc20.yaml | 4 ++++
drivers/gpu/drm/panel/panel-edp.c | 1 +
2 files changed, 5 insertions(+)
--
2.48.1
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v1 1/3] dt-bindings: display: panel: samsung, atna40cu11: document ATNA40CU11
2025-07-27 16:50 [PATCH v1 0/3] Add panels used by Asus Zenbook A14 Aleksandrs Vinarskis
@ 2025-07-27 16:50 ` Aleksandrs Vinarskis
2025-07-28 5:50 ` [PATCH v1 1/3] dt-bindings: display: panel: samsung,atna40cu11: " Krzysztof Kozlowski
2025-07-27 16:50 ` [PATCH v1 2/3] dt-bindings: display: panel: samsung, atna40ct06: document ATNA40CT06 Aleksandrs Vinarskis
2025-07-27 16:50 ` [PATCH v1 3/3] drm/panel-edp: Add BOE NV140WUM-N64 Aleksandrs Vinarskis
2 siblings, 1 reply; 10+ messages in thread
From: Aleksandrs Vinarskis @ 2025-07-27 16:50 UTC (permalink / raw)
To: Douglas Anderson, dri-devel, devicetree, linux-kernel
Cc: Neil Armstrong, Jessica Zhang, David Airlie, Simona Vetter,
Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Aleksandrs Vinarskis
The Samsung ATNA40CU11 panel is a 14" AMOLED eDP panel. It is
similar to the ATNA33XC20 except that it is larger and has a
different resolution. It is found in some arm64 laptops, eg.
Asus Zenbook A14 UX3407RA.
Raw panel edid:
00 ff ff ff ff ff ff 00 4c 83 9d 41 00 00 00 00
00 20 01 04 b5 1e 13 78 03 cf d1 ae 51 3e b6 23
0b 50 54 00 00 00 01 01 01 01 01 01 01 01 01 01
01 01 01 01 01 01 cb fe 40 64 b0 08 38 77 20 08
88 00 2e bd 10 00 00 1b 00 00 00 fd 00 30 78 da
da 42 01 00 00 00 00 00 00 00 00 00 00 00 00 00
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 fe
00 41 54 4e 41 34 30 43 55 31 31 2d 30 20 01 2a
70 20 79 02 00 20 00 0c 4c 83 00 9d 41 00 00 00
00 00 20 00 21 00 1d b8 0b 6c 07 40 0b 08 07 00
ee ea 50 ec d3 b6 3d 42 0b 01 45 54 40 5e d0 60
18 10 23 78 26 00 09 07 06 03 00 00 00 50 00 00
22 00 14 e7 f3 09 85 3f 0b 63 00 1f 00 07 00 07
07 17 00 07 00 07 00 81 00 1f 73 1a 00 00 03 03
30 78 00 a0 74 02 60 02 78 00 00 00 00 8d e3 05
80 00 e6 06 05 01 74 60 02 00 00 00 00 00 91 90
Signed-off-by: Aleksandrs Vinarskis <alex.vinarskis@gmail.com>
---
.../devicetree/bindings/display/panel/samsung,atna33xc20.yaml | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,atna33xc20.yaml b/Documentation/devicetree/bindings/display/panel/samsung,atna33xc20.yaml
index e36659340ef3..5e2ce200025f 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,atna33xc20.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,atna33xc20.yaml
@@ -21,6 +21,8 @@ properties:
- enum:
# Samsung 13" 3K (2880×1920 pixels) eDP AMOLED panel
- samsung,atna30dw01
+ # Samsung 14" WQXGA+ (2880x1800 pixels) eDP AMOLED panel
+ - samsung,atna40cu11
# Samsung 14" WQXGA+ (2880×1800 pixels) eDP AMOLED panel
- samsung,atna40yk20
# Samsung 14.5" WQXGA+ (2880x1800 pixels) eDP AMOLED panel
--
2.48.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v1 2/3] dt-bindings: display: panel: samsung, atna40ct06: document ATNA40CT06
2025-07-27 16:50 [PATCH v1 0/3] Add panels used by Asus Zenbook A14 Aleksandrs Vinarskis
2025-07-27 16:50 ` [PATCH v1 1/3] dt-bindings: display: panel: samsung, atna40cu11: document ATNA40CU11 Aleksandrs Vinarskis
@ 2025-07-27 16:50 ` Aleksandrs Vinarskis
2025-07-28 5:50 ` [PATCH v1 2/3] dt-bindings: display: panel: samsung,atna40ct06: " Krzysztof Kozlowski
2025-07-27 16:50 ` [PATCH v1 3/3] drm/panel-edp: Add BOE NV140WUM-N64 Aleksandrs Vinarskis
2 siblings, 1 reply; 10+ messages in thread
From: Aleksandrs Vinarskis @ 2025-07-27 16:50 UTC (permalink / raw)
To: Douglas Anderson, dri-devel, devicetree, linux-kernel
Cc: Neil Armstrong, Jessica Zhang, David Airlie, Simona Vetter,
Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Aleksandrs Vinarskis
The Samsung ATNA40CT06 panel is a 14" AMOLED eDP panel. It is
similar to the ATNA33XC20 except that it is larger and has a
different resolution. It is found in some arm64 laptops, eg.
Asus Zenbook A14 UX3407QA.
Raw panel edid:
00 ff ff ff ff ff ff 00 4c 83 0d 42 00 00 00 00
00 22 01 04 b5 1e 13 78 02 0c f1 ae 52 3c b9 23
0c 50 54 00 00 00 01 01 01 01 01 01 01 01 01 01
01 01 01 01 01 01 42 3c 80 a0 70 b0 24 40 30 20
88 00 2e bd 10 00 00 1b 00 00 00 00 00 00 00 00
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 fc
00 41 54 4e 41 34 30 43 54 30 36 2d 30 20 01 7d
70 20 79 02 00 20 00 0c 4c 83 00 0d 42 00 00 00
00 00 22 00 21 00 1d ca 0b 5e 07 80 07 b0 04 00
e1 fa 51 cb 13 b9 3d d2 0c 01 45 54 40 5e d0 60
18 10 23 78 26 00 09 07 06 03 00 00 00 50 00 00
22 00 14 8d 5a 02 85 7f 07 9f 00 2f 00 1f 00 af
04 23 00 07 00 07 00 81 00 0b e3 05 80 00 e6 06
05 01 74 60 02 2e 00 06 00 45 40 5e d0 60 00 00
00 00 00 00 00 00 00 00 00 00 00 00 00 00 b0 90
Signed-off-by: Aleksandrs Vinarskis <alex.vinarskis@gmail.com>
---
.../devicetree/bindings/display/panel/samsung,atna33xc20.yaml | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,atna33xc20.yaml b/Documentation/devicetree/bindings/display/panel/samsung,atna33xc20.yaml
index 5e2ce200025f..ccb574caed28 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,atna33xc20.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,atna33xc20.yaml
@@ -21,6 +21,8 @@ properties:
- enum:
# Samsung 13" 3K (2880×1920 pixels) eDP AMOLED panel
- samsung,atna30dw01
+ # Samsung 14" FHD+ (1920x1200 pixels) eDP AMOLED panel
+ - samsung,atna40ct06
# Samsung 14" WQXGA+ (2880x1800 pixels) eDP AMOLED panel
- samsung,atna40cu11
# Samsung 14" WQXGA+ (2880×1800 pixels) eDP AMOLED panel
--
2.48.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v1 3/3] drm/panel-edp: Add BOE NV140WUM-N64
2025-07-27 16:50 [PATCH v1 0/3] Add panels used by Asus Zenbook A14 Aleksandrs Vinarskis
2025-07-27 16:50 ` [PATCH v1 1/3] dt-bindings: display: panel: samsung, atna40cu11: document ATNA40CU11 Aleksandrs Vinarskis
2025-07-27 16:50 ` [PATCH v1 2/3] dt-bindings: display: panel: samsung, atna40ct06: document ATNA40CT06 Aleksandrs Vinarskis
@ 2025-07-27 16:50 ` Aleksandrs Vinarskis
2025-07-29 15:49 ` Doug Anderson
2 siblings, 1 reply; 10+ messages in thread
From: Aleksandrs Vinarskis @ 2025-07-27 16:50 UTC (permalink / raw)
To: Douglas Anderson, dri-devel, devicetree, linux-kernel
Cc: Neil Armstrong, Jessica Zhang, David Airlie, Simona Vetter,
Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Aleksandrs Vinarskis
Timings taken from NV140WUM-N41. It is found in some arm64 laptops,
eg. Asus Zenbook A14 UX3407QA.
The raw edid of the panel is:
00 ff ff ff ff ff ff 00 09 e5 f6 0c 00 00 00 00
10 22 01 04 a5 1e 13 78 07 8e 95 a6 52 4c 9d 26
0f 50 54 00 00 00 01 01 01 01 01 01 01 01 01 01
01 01 01 01 01 01 5d 30 80 a0 70 b0 28 40 30 20
36 00 2e bc 10 00 00 1a 00 00 00 fd 00 28 3c 4a
4a 0f 01 0a 20 20 20 20 20 20 00 00 00 fe 00 3d
4c 33 30 20 20 20 20 20 20 20 20 ff 00 00 00 fc
00 4e 56 31 34 30 57 55 4d 2d 4e 36 34 0a 01 f8
70 20 79 02 00 21 00 1d c8 0b 5d 07 80 07 b0 04
88 66 ea 51 cc 74 9d 66 52 0f 02 35 54 40 5e 40
5e 00 44 12 78 22 00 14 7f 5c 02 85 7f 07 9f 00
2f 00 1f 00 af 04 27 00 02 00 05 00 2b 00 0c 27
00 28 3b 00 00 27 00 28 2f 00 00 2e 00 06 00 44
40 5e 40 5e 81 00 1e 72 1a 00 00 03 71 28 3c 00
00 60 ff 60 ff 3c 00 00 00 00 e3 05 04 00 e6 06
01 01 60 60 ff 00 00 00 00 00 00 00 00 00 de 90
Signed-off-by: Aleksandrs Vinarskis <alex.vinarskis@gmail.com>
---
drivers/gpu/drm/panel/panel-edp.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/panel/panel-edp.c b/drivers/gpu/drm/panel/panel-edp.c
index 9a56e208cbdd..b334926e96ed 100644
--- a/drivers/gpu/drm/panel/panel-edp.c
+++ b/drivers/gpu/drm/panel/panel-edp.c
@@ -1947,6 +1947,7 @@ static const struct edp_panel_entry edp_panels[] = {
EDP_PANEL_ENTRY('B', 'O', 'E', 0x0c20, &delay_200_500_e80, "NT140FHM-N47"),
EDP_PANEL_ENTRY('B', 'O', 'E', 0x0c93, &delay_200_500_e200, "Unknown"),
EDP_PANEL_ENTRY('B', 'O', 'E', 0x0cb6, &delay_200_500_e200, "NT116WHM-N44"),
+ EDP_PANEL_ENTRY('B', 'O', 'E', 0x0cf6, &delay_200_500_e50_p2e80, "NV140WUM-N64"),
EDP_PANEL_ENTRY('B', 'O', 'E', 0x0cfa, &delay_200_500_e50, "NV116WHM-A4D"),
EDP_PANEL_ENTRY('B', 'O', 'E', 0x0d73, &delay_200_500_e80, "NE140WUM-N6S"),
--
2.48.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v1 1/3] dt-bindings: display: panel: samsung,atna40cu11: document ATNA40CU11
2025-07-27 16:50 ` [PATCH v1 1/3] dt-bindings: display: panel: samsung, atna40cu11: document ATNA40CU11 Aleksandrs Vinarskis
@ 2025-07-28 5:50 ` Krzysztof Kozlowski
0 siblings, 0 replies; 10+ messages in thread
From: Krzysztof Kozlowski @ 2025-07-28 5:50 UTC (permalink / raw)
To: Aleksandrs Vinarskis
Cc: Douglas Anderson, dri-devel, devicetree, linux-kernel,
Neil Armstrong, Jessica Zhang, David Airlie, Simona Vetter,
Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
On Sun, Jul 27, 2025 at 06:50:24PM +0200, Aleksandrs Vinarskis wrote:
> The Samsung ATNA40CU11 panel is a 14" AMOLED eDP panel. It is
> similar to the ATNA33XC20 except that it is larger and has a
> different resolution. It is found in some arm64 laptops, eg.
> Asus Zenbook A14 UX3407RA.
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 2/3] dt-bindings: display: panel: samsung,atna40ct06: document ATNA40CT06
2025-07-27 16:50 ` [PATCH v1 2/3] dt-bindings: display: panel: samsung, atna40ct06: document ATNA40CT06 Aleksandrs Vinarskis
@ 2025-07-28 5:50 ` Krzysztof Kozlowski
0 siblings, 0 replies; 10+ messages in thread
From: Krzysztof Kozlowski @ 2025-07-28 5:50 UTC (permalink / raw)
To: Aleksandrs Vinarskis
Cc: Douglas Anderson, dri-devel, devicetree, linux-kernel,
Neil Armstrong, Jessica Zhang, David Airlie, Simona Vetter,
Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
On Sun, Jul 27, 2025 at 06:50:25PM +0200, Aleksandrs Vinarskis wrote:
> The Samsung ATNA40CT06 panel is a 14" AMOLED eDP panel. It is
> similar to the ATNA33XC20 except that it is larger and has a
> different resolution. It is found in some arm64 laptops, eg.
> Asus Zenbook A14 UX3407QA.
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 3/3] drm/panel-edp: Add BOE NV140WUM-N64
2025-07-27 16:50 ` [PATCH v1 3/3] drm/panel-edp: Add BOE NV140WUM-N64 Aleksandrs Vinarskis
@ 2025-07-29 15:49 ` Doug Anderson
2025-07-30 20:38 ` Aleksandrs Vinarskis
0 siblings, 1 reply; 10+ messages in thread
From: Doug Anderson @ 2025-07-29 15:49 UTC (permalink / raw)
To: Aleksandrs Vinarskis
Cc: dri-devel, devicetree, linux-kernel, Neil Armstrong,
Jessica Zhang, David Airlie, Simona Vetter, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
Hi,
On Sun, Jul 27, 2025 at 9:58 AM Aleksandrs Vinarskis
<alex.vinarskis@gmail.com> wrote:
>
> Timings taken from NV140WUM-N41. It is found in some arm64 laptops,
> eg. Asus Zenbook A14 UX3407QA.
>
> The raw edid of the panel is:
> 00 ff ff ff ff ff ff 00 09 e5 f6 0c 00 00 00 00
> 10 22 01 04 a5 1e 13 78 07 8e 95 a6 52 4c 9d 26
> 0f 50 54 00 00 00 01 01 01 01 01 01 01 01 01 01
> 01 01 01 01 01 01 5d 30 80 a0 70 b0 28 40 30 20
> 36 00 2e bc 10 00 00 1a 00 00 00 fd 00 28 3c 4a
> 4a 0f 01 0a 20 20 20 20 20 20 00 00 00 fe 00 3d
> 4c 33 30 20 20 20 20 20 20 20 20 ff 00 00 00 fc
> 00 4e 56 31 34 30 57 55 4d 2d 4e 36 34 0a 01 f8
>
> 70 20 79 02 00 21 00 1d c8 0b 5d 07 80 07 b0 04
> 88 66 ea 51 cc 74 9d 66 52 0f 02 35 54 40 5e 40
> 5e 00 44 12 78 22 00 14 7f 5c 02 85 7f 07 9f 00
> 2f 00 1f 00 af 04 27 00 02 00 05 00 2b 00 0c 27
> 00 28 3b 00 00 27 00 28 2f 00 00 2e 00 06 00 44
> 40 5e 40 5e 81 00 1e 72 1a 00 00 03 71 28 3c 00
> 00 60 ff 60 ff 3c 00 00 00 00 e3 05 04 00 e6 06
> 01 01 60 60 ff 00 00 00 00 00 00 00 00 00 de 90
>
> Signed-off-by: Aleksandrs Vinarskis <alex.vinarskis@gmail.com>
> ---
> drivers/gpu/drm/panel/panel-edp.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/panel/panel-edp.c b/drivers/gpu/drm/panel/panel-edp.c
> index 9a56e208cbdd..b334926e96ed 100644
> --- a/drivers/gpu/drm/panel/panel-edp.c
> +++ b/drivers/gpu/drm/panel/panel-edp.c
> @@ -1947,6 +1947,7 @@ static const struct edp_panel_entry edp_panels[] = {
> EDP_PANEL_ENTRY('B', 'O', 'E', 0x0c20, &delay_200_500_e80, "NT140FHM-N47"),
> EDP_PANEL_ENTRY('B', 'O', 'E', 0x0c93, &delay_200_500_e200, "Unknown"),
> EDP_PANEL_ENTRY('B', 'O', 'E', 0x0cb6, &delay_200_500_e200, "NT116WHM-N44"),
> + EDP_PANEL_ENTRY('B', 'O', 'E', 0x0cf6, &delay_200_500_e50_p2e80, "NV140WUM-N64"),
Since this is a "guess" timing without any datasheet, I'd be more
comfortable picking the most conservative of the "cousin" timings. Can
you re-send with "delay_200_500_e200" instead?
-Doug
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 3/3] drm/panel-edp: Add BOE NV140WUM-N64
2025-07-29 15:49 ` Doug Anderson
@ 2025-07-30 20:38 ` Aleksandrs Vinarskis
2025-07-30 21:37 ` Doug Anderson
0 siblings, 1 reply; 10+ messages in thread
From: Aleksandrs Vinarskis @ 2025-07-30 20:38 UTC (permalink / raw)
To: Doug Anderson
Cc: dri-devel, devicetree, linux-kernel, Neil Armstrong,
Jessica Zhang, David Airlie, Simona Vetter, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
On Tue, 29 Jul 2025 at 17:50, Doug Anderson <dianders@chromium.org> wrote:
>
> Hi,
>
> On Sun, Jul 27, 2025 at 9:58 AM Aleksandrs Vinarskis
> <alex.vinarskis@gmail.com> wrote:
> >
> > Timings taken from NV140WUM-N41. It is found in some arm64 laptops,
> > eg. Asus Zenbook A14 UX3407QA.
> >
> > The raw edid of the panel is:
> > 00 ff ff ff ff ff ff 00 09 e5 f6 0c 00 00 00 00
> > 10 22 01 04 a5 1e 13 78 07 8e 95 a6 52 4c 9d 26
> > 0f 50 54 00 00 00 01 01 01 01 01 01 01 01 01 01
> > 01 01 01 01 01 01 5d 30 80 a0 70 b0 28 40 30 20
> > 36 00 2e bc 10 00 00 1a 00 00 00 fd 00 28 3c 4a
> > 4a 0f 01 0a 20 20 20 20 20 20 00 00 00 fe 00 3d
> > 4c 33 30 20 20 20 20 20 20 20 20 ff 00 00 00 fc
> > 00 4e 56 31 34 30 57 55 4d 2d 4e 36 34 0a 01 f8
> >
> > 70 20 79 02 00 21 00 1d c8 0b 5d 07 80 07 b0 04
> > 88 66 ea 51 cc 74 9d 66 52 0f 02 35 54 40 5e 40
> > 5e 00 44 12 78 22 00 14 7f 5c 02 85 7f 07 9f 00
> > 2f 00 1f 00 af 04 27 00 02 00 05 00 2b 00 0c 27
> > 00 28 3b 00 00 27 00 28 2f 00 00 2e 00 06 00 44
> > 40 5e 40 5e 81 00 1e 72 1a 00 00 03 71 28 3c 00
> > 00 60 ff 60 ff 3c 00 00 00 00 e3 05 04 00 e6 06
> > 01 01 60 60 ff 00 00 00 00 00 00 00 00 00 de 90
> >
> > Signed-off-by: Aleksandrs Vinarskis <alex.vinarskis@gmail.com>
> > ---
> > drivers/gpu/drm/panel/panel-edp.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/gpu/drm/panel/panel-edp.c b/drivers/gpu/drm/panel/panel-edp.c
> > index 9a56e208cbdd..b334926e96ed 100644
> > --- a/drivers/gpu/drm/panel/panel-edp.c
> > +++ b/drivers/gpu/drm/panel/panel-edp.c
> > @@ -1947,6 +1947,7 @@ static const struct edp_panel_entry edp_panels[] = {
> > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0c20, &delay_200_500_e80, "NT140FHM-N47"),
> > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0c93, &delay_200_500_e200, "Unknown"),
> > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0cb6, &delay_200_500_e200, "NT116WHM-N44"),
> > + EDP_PANEL_ENTRY('B', 'O', 'E', 0x0cf6, &delay_200_500_e50_p2e80, "NV140WUM-N64"),
>
> Since this is a "guess" timing without any datasheet, I'd be more
> comfortable picking the most conservative of the "cousin" timings. Can
> you re-send with "delay_200_500_e200" instead?
Sure.
Do I understand correctly that more conservative delay_200_500_e200
will for sure not make it worse, since its more conservative? In that
case can re-spin right away. Otherwise I would prefer to re-test it
first, may take a few days as I do not own the hardware so need to
propagate the change and get some feedback.
Thanks,
Alex
>
> -Doug
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 3/3] drm/panel-edp: Add BOE NV140WUM-N64
2025-07-30 20:38 ` Aleksandrs Vinarskis
@ 2025-07-30 21:37 ` Doug Anderson
2025-07-31 21:43 ` Aleksandrs Vinarskis
0 siblings, 1 reply; 10+ messages in thread
From: Doug Anderson @ 2025-07-30 21:37 UTC (permalink / raw)
To: Aleksandrs Vinarskis
Cc: dri-devel, devicetree, linux-kernel, Neil Armstrong,
Jessica Zhang, David Airlie, Simona Vetter, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
Hi,
On Wed, Jul 30, 2025 at 1:38 PM Aleksandrs Vinarskis
<alex.vinarskis@gmail.com> wrote:
>
> On Tue, 29 Jul 2025 at 17:50, Doug Anderson <dianders@chromium.org> wrote:
> >
> > Hi,
> >
> > On Sun, Jul 27, 2025 at 9:58 AM Aleksandrs Vinarskis
> > <alex.vinarskis@gmail.com> wrote:
> > >
> > > Timings taken from NV140WUM-N41. It is found in some arm64 laptops,
> > > eg. Asus Zenbook A14 UX3407QA.
> > >
> > > The raw edid of the panel is:
> > > 00 ff ff ff ff ff ff 00 09 e5 f6 0c 00 00 00 00
> > > 10 22 01 04 a5 1e 13 78 07 8e 95 a6 52 4c 9d 26
> > > 0f 50 54 00 00 00 01 01 01 01 01 01 01 01 01 01
> > > 01 01 01 01 01 01 5d 30 80 a0 70 b0 28 40 30 20
> > > 36 00 2e bc 10 00 00 1a 00 00 00 fd 00 28 3c 4a
> > > 4a 0f 01 0a 20 20 20 20 20 20 00 00 00 fe 00 3d
> > > 4c 33 30 20 20 20 20 20 20 20 20 ff 00 00 00 fc
> > > 00 4e 56 31 34 30 57 55 4d 2d 4e 36 34 0a 01 f8
> > >
> > > 70 20 79 02 00 21 00 1d c8 0b 5d 07 80 07 b0 04
> > > 88 66 ea 51 cc 74 9d 66 52 0f 02 35 54 40 5e 40
> > > 5e 00 44 12 78 22 00 14 7f 5c 02 85 7f 07 9f 00
> > > 2f 00 1f 00 af 04 27 00 02 00 05 00 2b 00 0c 27
> > > 00 28 3b 00 00 27 00 28 2f 00 00 2e 00 06 00 44
> > > 40 5e 40 5e 81 00 1e 72 1a 00 00 03 71 28 3c 00
> > > 00 60 ff 60 ff 3c 00 00 00 00 e3 05 04 00 e6 06
> > > 01 01 60 60 ff 00 00 00 00 00 00 00 00 00 de 90
> > >
> > > Signed-off-by: Aleksandrs Vinarskis <alex.vinarskis@gmail.com>
> > > ---
> > > drivers/gpu/drm/panel/panel-edp.c | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git a/drivers/gpu/drm/panel/panel-edp.c b/drivers/gpu/drm/panel/panel-edp.c
> > > index 9a56e208cbdd..b334926e96ed 100644
> > > --- a/drivers/gpu/drm/panel/panel-edp.c
> > > +++ b/drivers/gpu/drm/panel/panel-edp.c
> > > @@ -1947,6 +1947,7 @@ static const struct edp_panel_entry edp_panels[] = {
> > > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0c20, &delay_200_500_e80, "NT140FHM-N47"),
> > > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0c93, &delay_200_500_e200, "Unknown"),
> > > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0cb6, &delay_200_500_e200, "NT116WHM-N44"),
> > > + EDP_PANEL_ENTRY('B', 'O', 'E', 0x0cf6, &delay_200_500_e50_p2e80, "NV140WUM-N64"),
> >
> > Since this is a "guess" timing without any datasheet, I'd be more
> > comfortable picking the most conservative of the "cousin" timings. Can
> > you re-send with "delay_200_500_e200" instead?
>
> Sure.
>
> Do I understand correctly that more conservative delay_200_500_e200
> will for sure not make it worse, since its more conservative? In that
> case can re-spin right away. Otherwise I would prefer to re-test it
> first, may take a few days as I do not own the hardware so need to
> propagate the change and get some feedback.
Is anything really for sure? No. :-)
...but it's _highly_ likely to not make it worse. Presumably you
tested without ${SUBJECT} patch and things worked OK aside from the
big warning splat in your logs about using the conservative timings.
Those conservative timings basically give you:
desc->delay.enable = 200;
...so you probably already tested with an enable timing of 200 and
using 200 here will give you the same conservative number for
"enable", will avoid some other conservative timings, and will avoid
the warning splat.
...and the "p2e80" doesn't do anything useful when enable is 200.
-Doug
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 3/3] drm/panel-edp: Add BOE NV140WUM-N64
2025-07-30 21:37 ` Doug Anderson
@ 2025-07-31 21:43 ` Aleksandrs Vinarskis
0 siblings, 0 replies; 10+ messages in thread
From: Aleksandrs Vinarskis @ 2025-07-31 21:43 UTC (permalink / raw)
To: Doug Anderson
Cc: dri-devel, devicetree, linux-kernel, Neil Armstrong,
Jessica Zhang, David Airlie, Simona Vetter, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
On Wed, 30 Jul 2025 at 23:44, Doug Anderson <dianders@chromium.org> wrote:
>
> Hi,
>
> On Wed, Jul 30, 2025 at 1:38 PM Aleksandrs Vinarskis
> <alex.vinarskis@gmail.com> wrote:
> >
> > On Tue, 29 Jul 2025 at 17:50, Doug Anderson <dianders@chromium.org> wrote:
> > >
> > > Hi,
> > >
> > > On Sun, Jul 27, 2025 at 9:58 AM Aleksandrs Vinarskis
> > > <alex.vinarskis@gmail.com> wrote:
> > > >
> > > > Timings taken from NV140WUM-N41. It is found in some arm64 laptops,
> > > > eg. Asus Zenbook A14 UX3407QA.
> > > >
> > > > The raw edid of the panel is:
> > > > 00 ff ff ff ff ff ff 00 09 e5 f6 0c 00 00 00 00
> > > > 10 22 01 04 a5 1e 13 78 07 8e 95 a6 52 4c 9d 26
> > > > 0f 50 54 00 00 00 01 01 01 01 01 01 01 01 01 01
> > > > 01 01 01 01 01 01 5d 30 80 a0 70 b0 28 40 30 20
> > > > 36 00 2e bc 10 00 00 1a 00 00 00 fd 00 28 3c 4a
> > > > 4a 0f 01 0a 20 20 20 20 20 20 00 00 00 fe 00 3d
> > > > 4c 33 30 20 20 20 20 20 20 20 20 ff 00 00 00 fc
> > > > 00 4e 56 31 34 30 57 55 4d 2d 4e 36 34 0a 01 f8
> > > >
> > > > 70 20 79 02 00 21 00 1d c8 0b 5d 07 80 07 b0 04
> > > > 88 66 ea 51 cc 74 9d 66 52 0f 02 35 54 40 5e 40
> > > > 5e 00 44 12 78 22 00 14 7f 5c 02 85 7f 07 9f 00
> > > > 2f 00 1f 00 af 04 27 00 02 00 05 00 2b 00 0c 27
> > > > 00 28 3b 00 00 27 00 28 2f 00 00 2e 00 06 00 44
> > > > 40 5e 40 5e 81 00 1e 72 1a 00 00 03 71 28 3c 00
> > > > 00 60 ff 60 ff 3c 00 00 00 00 e3 05 04 00 e6 06
> > > > 01 01 60 60 ff 00 00 00 00 00 00 00 00 00 de 90
> > > >
> > > > Signed-off-by: Aleksandrs Vinarskis <alex.vinarskis@gmail.com>
> > > > ---
> > > > drivers/gpu/drm/panel/panel-edp.c | 1 +
> > > > 1 file changed, 1 insertion(+)
> > > >
> > > > diff --git a/drivers/gpu/drm/panel/panel-edp.c b/drivers/gpu/drm/panel/panel-edp.c
> > > > index 9a56e208cbdd..b334926e96ed 100644
> > > > --- a/drivers/gpu/drm/panel/panel-edp.c
> > > > +++ b/drivers/gpu/drm/panel/panel-edp.c
> > > > @@ -1947,6 +1947,7 @@ static const struct edp_panel_entry edp_panels[] = {
> > > > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0c20, &delay_200_500_e80, "NT140FHM-N47"),
> > > > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0c93, &delay_200_500_e200, "Unknown"),
> > > > EDP_PANEL_ENTRY('B', 'O', 'E', 0x0cb6, &delay_200_500_e200, "NT116WHM-N44"),
> > > > + EDP_PANEL_ENTRY('B', 'O', 'E', 0x0cf6, &delay_200_500_e50_p2e80, "NV140WUM-N64"),
> > >
> > > Since this is a "guess" timing without any datasheet, I'd be more
> > > comfortable picking the most conservative of the "cousin" timings. Can
> > > you re-send with "delay_200_500_e200" instead?
> >
> > Sure.
> >
> > Do I understand correctly that more conservative delay_200_500_e200
> > will for sure not make it worse, since its more conservative? In that
> > case can re-spin right away. Otherwise I would prefer to re-test it
> > first, may take a few days as I do not own the hardware so need to
> > propagate the change and get some feedback.
>
> Is anything really for sure? No. :-)
>
> ...but it's _highly_ likely to not make it worse. Presumably you
> tested without ${SUBJECT} patch and things worked OK aside from the
> big warning splat in your logs about using the conservative timings.
> Those conservative timings basically give you:
>
> desc->delay.enable = 200;
>
> ...so you probably already tested with an enable timing of 200 and
> using 200 here will give you the same conservative number for
> "enable", will avoid some other conservative timings, and will avoid
> the warning splat.
Indeed, this is mostly to get rid of the warning, and it worked
perfectly fine without adjusted timings.
Thanks for the explanation! Will re-spin shortly.
Alex
>
> ...and the "p2e80" doesn't do anything useful when enable is 200.
>
>
> -Doug
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2025-07-31 21:43 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-27 16:50 [PATCH v1 0/3] Add panels used by Asus Zenbook A14 Aleksandrs Vinarskis
2025-07-27 16:50 ` [PATCH v1 1/3] dt-bindings: display: panel: samsung, atna40cu11: document ATNA40CU11 Aleksandrs Vinarskis
2025-07-28 5:50 ` [PATCH v1 1/3] dt-bindings: display: panel: samsung,atna40cu11: " Krzysztof Kozlowski
2025-07-27 16:50 ` [PATCH v1 2/3] dt-bindings: display: panel: samsung, atna40ct06: document ATNA40CT06 Aleksandrs Vinarskis
2025-07-28 5:50 ` [PATCH v1 2/3] dt-bindings: display: panel: samsung,atna40ct06: " Krzysztof Kozlowski
2025-07-27 16:50 ` [PATCH v1 3/3] drm/panel-edp: Add BOE NV140WUM-N64 Aleksandrs Vinarskis
2025-07-29 15:49 ` Doug Anderson
2025-07-30 20:38 ` Aleksandrs Vinarskis
2025-07-30 21:37 ` Doug Anderson
2025-07-31 21:43 ` Aleksandrs Vinarskis
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).