On 13.09.2026 09:32, [email protected] wrote:
Thank you for your contribution! Sashiko AI review found 2 potential issue(s) 
to consider:
- [Low] Incomplete Documentation Update
- [Low] Alphabetical ordering violation in compatible strings



[Severity: Low]
Could the main description block be updated to reflect these additions?
The top-level description property for novatek,nt36672a.yaml currently says
"Right now, support is added only for a Tianma FHD+ LCD display panel with
a resolution of 1080x2246." Since this commit adds 5 new panels to the
compatible items enum below, the description is now outdated.


I remember now that I wanted to update description, but forgot.


[Severity: Low]
Is it possible to sort these compatible strings alphabetically? In ASCII,
the character 4 comes before a, so it seems like "tianma,tl063fvmc43-02"
should precede "tianma,tl063fvmca01-00" in this enum list.

+          - txd,txdi600yanpa-43v3
        - const: novatek,nt36672a
      description: This indicates the panel manufacturer of the panel that is
        in turn using the NT36672A panel driver. This compatible string


This sorting issue and description will be fixed in v2.

--
Regards,
Alexey Minnekhanov

Reply via email to