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:
pgp0YOpd_YhLt.pgp
Description: OpenPGP digital signature
