Hi Rob, Shawn > Subject: Re: [PATCH v6 0/3] arm64: dts: add i.MX95 and EVK board > > > On Wed, 05 Jun 2024 09:22:47 +0800, Peng Fan (OSS) wrote: > > Add a minimal i.MX95 dtsi and EVK board dts. > > i.MX95 has a M33 running SCMI firmware that supports > > pinctrl/power/perf/clock and etc. > > > > imx95-pinfunc.h will trigger checkpatch error, that is expected and > > same as other i.MX platforms. > > > > In v6, I added back a dependency on pinctrl, because [1] has got > > A-b/R-b from Maintainers, so it would be soon got merged. > > > > There will be dtbs_check error before [1] got landed. With [1] merged, > > there will be no dtbs_check error. > > .... > > > > > > > > > My bot found new DTB warnings on the .dts files added or changed in this > series. > > 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 freescale/imx95-19x19-evk.dtb' > for 20240605-imx95-dts-v3-v6-0-2ce275ed0e80@xxxxxxx: > > arch/arm64/boot/dts/freescale/imx95-19x19-evk.dtb: scmi: protocol@19: > Unevaluated properties are not allowed ('regusdhc2vmmcgrp', 'uart1grp', > 'usdhc1-100mhzgrp', 'usdhc1-200mhzgrp', 'usdhc1grp', 'usdhc2-100mhzgrp', > 'usdhc2-200mhzgrp', 'usdhc2gpiogrp', 'usdhc2grp' were unexpected) > from schema $id: Since Linus has applied the pinctrl patchset[1], the check will pass. [1] https://lore.kernel.org/all/CACRpkdbpL=HUXj0hFAo+JNki_R A9aix2sW1cg13g9=89d93PZw@xxxxxxxxxxxxxx/ Shawn, In patch 3, there is one minor comment, do you expect me to send v7 or you could help update it? Thanks, Peng. > > >