Re: [PATCH v8 2/2] ARM: dts: mxs: Add descriptions for imx287 based btt3-[012] devices

[Lukasz Majewski <lukma@denx.de>]

[-- Attachment #1: Type: text/plain, Size: 1845 bytes --]

Hi Fabio,

> Hi Lukasz,
> 
> On Thu, Oct 17, 2024 at 5:35 AM Lukasz Majewski <lukma@denx.de> wrote:
> 
> > Stefan, do you have comments for this version?  
> 
> Rob's bot reported new warnings after applying your series:
> 
>         from schema $id:
> http://devicetree.org/schemas/display/panel/panel-dpi.yaml#
> arch/arm/boot/dts/nxp/mxs/imx28-btt3-2.dtb: panel: compatible:
> ['panel-dpi'] is too short
>         from schema $id:
> http://devicetree.org/schemas/display/panel/panel-dpi.yaml#
> arch/arm/boot/dts/nxp/mxs/imx28-btt3-1.dtb: panel: compatible:
> ['panel-dpi'] is too short
>         from schema $id:
> http://devicetree.org/schemas/display/panel/panel-dpi.yaml#
> 
> Please address it.

The problem is that I don't know the name of the display itself
(different vendors) [*].

Schema requires following syntax:
arch/arm/boot/dts/ti/omap/am437x-gp-evm.dts:89:         compatible =
"osddisplays,osd070t1718-19ts", "panel-dpi";

arch/arm/boot/dts/ti/omap/omap3-thunder.dts:86:         compatible =
"samsung,lte430wq-f0c", "panel-dpi";

which require two compatible entries - one specific type of the display
(e.g. "samsung,lte430wq-f0c") and the second one generic ("panel-dpi").

As I don't know what is the manufacturer of it - I can add:

 compatible = "foo", "panel-dpi";

and then Schema and Rob's scripts would be happy...


Another option is to adjust the
Documentation/devicetree/bindings/display/panel/panel-dpi.yaml to allow
only single entry.


[*] maybe I can obtain the info regarding at least one vendor ...

Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]