Re: [PATCH stm32-next v3 0/2] ARM: dts: stm32: lxa-fairytux2: add gen{1,2} boards

[Marc Kleine-Budde]

On 23.01.2025 09:57:47, Rob Herring (Arm) wrote:
> 
> On Tue, 21 Jan 2025 12:14:04 +0100, Marc Kleine-Budde wrote:
> > Hello,
> > 
> > this series adds support for the Linux Automation GmbH FairyTux2
> > boards generation 1 and 2.
> > 
> > The FairyTux2 is a small Linux device based on an Octavo Systems
> > OSD32MP153c SiP, that occupies just two slots on a DIN rail.
> > 
> > regards,
> > Marc
> > 
> > Signed-off-by: Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx>
> > ---
> > Changes in v3:
> > - 2/2: fix use generic node names for external GPIO and USB Type-C controller (thanks Krzysztof)
> > - 2/2: LEDs: replace label by color and function property (thanks Krzysztof)
> > - 2/2: gen2: sort nodes of i2c1 by address (thanks Krzysztof)
> > - Link to v2: https://patch.msgid.link/20250120-lxa-fairytux-v2-0-95f4a0eaa44d@xxxxxxxxxxxxxx
> > 
> > Changes in v2:
> > - 1/2: Added Krzysztof's Acked-by
> > - 2/2: fix alignment for gpio-line-names (thanks Krzysztof)
> > - 2/2: only use color:function for LED labels (thanks Krzysztof)
> > - 2/2: use generic node names for external GPIO and USB Type-C controller
> > - Link to v1: https://patch.msgid.link/20241210-lxa-fairytux-v1-0-9aea01cdb83e@xxxxxxxxxxxxxx
> > 
> > ---
> > Leonard Göhrs (2):
> >       dt-bindings: arm: stm32: add compatible strings for Linux Automation GmbH LXA FairyTux 2
> >       ARM: dts: stm32: lxa-fairytux2: add Linux Automation GmbH FairyTux 2
> > 
> >  .../devicetree/bindings/arm/stm32/stm32.yaml       |   2 +
> >  arch/arm/boot/dts/st/Makefile                      |   2 +
> >  .../boot/dts/st/stm32mp153c-lxa-fairytux2-gen1.dts | 103 ++++++
> >  .../boot/dts/st/stm32mp153c-lxa-fairytux2-gen2.dts | 147 ++++++++
> >  .../arm/boot/dts/st/stm32mp153c-lxa-fairytux2.dtsi | 397 +++++++++++++++++++++
> 
> My bot found new DTB warnings on the .dts files added or changed in this
> series.

This patch doesn't touch the offending file "ste-hrefv60plus-tvk.dtb".
Might be a new warning, but not due to this patch, could this be a false
positive?

> Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
> are fixed by another series. Ultimately, it is up to the platform
> maintainer whether these warnings are acceptable or not. No need to reply
> unless the platform maintainer has comments.
> 
> If you already ran DT checks and didn't see these error(s), then
> make sure dt-schema is up to date:
> 
>   pip3 install dtschema --upgrade
> 
> 
> New warnings running 'make CHECK_DTBS=y for arch/arm/boot/dts/st/' for 20250121-lxa-fairytux-v3-0-8d42d7d232fb@xxxxxxxxxxxxxx:
> 
> arch/arm/boot/dts/st/ste-hrefv60plus-tvk.dtb: gpio@a03fe000: compatible:0: 'stericsson,db8500-gpio' is not one of ['st,nomadik-gpio', 'mobileye,eyeq5-gpio']
> 	from schema $id: http://devicetree.org/schemas/gpio/st,nomadik-gpio.yaml#

regards,
Marc

-- 
Pengutronix e.K.                 | Marc Kleine-Budde          |
Embedded Linux                   | https://www.pengutronix.de |
Vertretung Nürnberg              | Phone: +49-5121-206917-129 |
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-9   |

Attachment: signature.asc
Description: PGP signature