Re: [PATCH 1/2] dt-bindings: panel-simple-dsi: add New Khadas TS050 panel bindings

[Christian Hewitt]

> On 9 Apr 2024, at 12:26 PM, Jacobe Zang <jacobe.zang@xxxxxxxxxx> wrote:
> 
> This add the bindings for the New Khadas TS050 1080x1920 5" LCD DSI panel
> designed to work with the Khadas VIM3 and VIM3L Single Board Computers.
> 
> Signed-off-by: Jacobe Zang <jacobe.zang@xxxxxxxxxx>
> ---
> .../devicetree/bindings/display/panel/panel-simple-dsi.yaml     | 2 ++
> 1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple-dsi.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple-dsi.yaml
> index f9160d7bac3ca..e194309f31b72 100644
> --- a/Documentation/devicetree/bindings/display/panel/panel-simple-dsi.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/panel-simple-dsi.yaml
> @@ -36,6 +36,8 @@ properties:
>       - jdi,fhd-r63452
>         # Khadas TS050 5" 1080x1920 LCD panel
>       - khadas,ts050
> +        # Khadas NEW TS050 5" 1080x1920 LCD panel
> +      - khadas,newts050

Products are only new until they are old. At some future point there will
inevitably be a third iteration requiring a ‘new new’ name. IMHO it would
be better to use something like khadas,ts050v2.

CH.

>         # Kingdisplay KD097D04 9.7" 1536x2048 TFT LCD panel
>       - kingdisplay,kd097d04
>         # LG ACX467AKM-7 4.95" 1080×1920 LCD Panel
> -- 
> 2.34.1
> 
> 
> _______________________________________________
> linux-amlogic mailing list
> linux-amlogic@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/linux-amlogic