[PATCH 1/2] dt-bindings: display: simple: Add AM-1280800W8TZQW-T00H

[PATCH 1/2] dt-bindings: display: simple: Add AM-1280800W8TZQW-T00H

[Dario Binacchi]

Add dt-bindings for 10.1" TFT LCD module from Ampire Co. Ltd.
as part of panel-simple.

Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
---

 .../devicetree/bindings/display/panel/panel-simple.yaml         | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
index 3e41ed0ef5d5..ea5fe23a6fe8 100644
--- a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
+++ b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
@@ -29,6 +29,8 @@ properties:
     # compatible must be listed in alphabetical order, ordered by compatible.
     # The description in the comment is mandatory for each compatible.
 
+        # Ampire AM-1280800W8TZQW-T00H 10.1" WQVGA TFT LCD panel
+      - ampire,am-1280800w8tzqw-t00h
         # Ampire AM-1280800N3TZQW-T00H 10.1" WQVGA TFT LCD panel
       - ampire,am-1280800n3tzqw-t00h
         # Ampire AM-480272H3TMQW-T01H 4.3" WQVGA TFT LCD panel
-- 
2.43.0

base-commit: 70eda68668d1476b459b64e69b8f36659fa9dfa8
branch: am-1280800N3tzqw-to00h

Re: [PATCH 1/2] dt-bindings: display: simple: Add AM-1280800W8TZQW-T00H

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider:
- [Low] The new compatible string is not inserted in alphabetical order.
- [Low] The panel description comment incorrectly states the resolution as WQVGA.
--

commit bf39295cb05fc88a5c9fc03f7444ba1eb3e26cfc
Author: Dario Binacchi <dario.binacchi@amarulasolutions.com>

dt-bindings: display: simple: Add AM-1280800W8TZQW-T00H

This commit adds dt-bindings for a 10.1" TFT LCD module from Ampire Co. Ltd.
to the panel-simple binding.

> diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
> index 3e41ed0ef5d51..ea5fe23a6fe84 100644
> --- a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
> @@ -29,6 +29,8 @@ properties:
>      # compatible must be listed in alphabetical order, ordered by compatible.
>      # The description in the comment is mandatory for each compatible.
>  
> +        # Ampire AM-1280800W8TZQW-T00H 10.1" WQVGA TFT LCD panel

This isn't a bug, but is the WQVGA description accurate here? The part number
AM-1280800 suggests a 1280x800 resolution (WXGA), while WQVGA is typically
480x272. It looks like this might have been copied from another entry.

> +      - ampire,am-1280800w8tzqw-t00h
>          # Ampire AM-1280800N3TZQW-T00H 10.1" WQVGA TFT LCD panel
>        - ampire,am-1280800n3tzqw-t00h

This isn't a bug, but should this new compatible be placed after
ampire,am-1280800n3tzqw-t00h? The comment above requires compatibles to be
listed in alphabetical order, and w comes after n.

>          # Ampire AM-480272H3TMQW-T01H 4.3" WQVGA TFT LCD panel

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260515064721.1750990-1-dario.binacchi@amarulasolutions.com?part=1