Re: [PATCH v8 2/2] ARM: dts: mxs: Add descriptions for imx287 based btt3-[012] devices
From: Lukasz Majewski
Date: Thu Oct 17 2024 - 08:18:36 EST
Hi Fabio,
> Hi Lukasz,
>
> On Thu, Oct 17, 2024 at 5:35 AM Lukasz Majewski <lukma@xxxxxxx> 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@xxxxxxx
Attachment:
pgpIYYkfiG_XO.pgp
Description: OpenPGP digital signature