* [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
* 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
* [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
* 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
* [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 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).