On Sat, Jul 16, 2022 at 8:02 AM Rob Herring <robh@xxxxxxxxxx> wrote: > > On Thu, Jun 23, 2022 at 8:44 AM Dmitry Baryshkov > <dmitry.baryshkov@xxxxxxxxxx> wrote: > > > > It is possible to build a single dtb, but not with DT schema validation > > enabled. Enable the schema validation to run for %.dtb and %.dtbo > > targets. Anyone building a dtb for a specific platform *should* pay > > attention to schema warnings. > > > > This could be supported with a separate %.dt.yaml target instead. > > However, the .dt.yaml format is considered an intermediate format and > > could possibly go away at some point if schema checking is integrated > > into dtc. Also, the plan is to enable the schema checks by default once > > platforms are free of warnings, and this is a move in that direction. > > > > This patch differs from the previous one ([1]) in the fact that it > > requires specifying VALIDATE_DT=1 to run the checks while doing the > > build. Thus default build procedures would not obtain additional build > > dependency, while maintainers can still build a single DTB file an get > > only corresponding warnings. > > I'd rather this be a kconfig option, so that eventually 'make > allmodconfig; make dtbs' also runs the schema checks. If something can > be enabled for allmodconfig, then builders will automatically start > testing it. Though the extra dependency is a problem here. The dependency on libyaml is gone. As for the dependency on dt-schema, is it a good idea to pull it into the kernel tree somewhere, like we periodically sync scripts/dtc/ with its upstream? Any other problematic tool dependency? > > > > > [1] https://lore.kernel.org/all/20210913145146.766080-1-robh@xxxxxxxxxx/ > > > > Cc: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> > > Cc: Tom Rini <trini@xxxxxxxxxxxx> > > Cc: Masahiro Yamada <masahiroy@xxxxxxxxxx> > > Cc: linux-kbuild@xxxxxxxxxxxxxxx > > Co-developed-by: Rob Herring <robh@xxxxxxxxxx> > > Signed-off-by: Rob Herring <robh@xxxxxxxxxx> > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > > --- > > Makefile | 18 ++++++++++++++---- > > 1 file changed, 14 insertions(+), 4 deletions(-) > > > > diff --git a/Makefile b/Makefile > > index c43d825a3c4c..0942922384c4 100644 > > --- a/Makefile > > +++ b/Makefile > > @@ -1365,11 +1365,17 @@ endif > > > > ifneq ($(dtstree),) > > > > -%.dtb: include/config/kernel.release scripts_dtc > > - $(Q)$(MAKE) $(build)=$(dtstree) $(dtstree)/$@ > > +ifneq ($(VALIDATE_DT),) > > +DT_YAML = $(dtstree)/$*.dt.yaml > > .dt.yaml files are deprecated now. This probably doesn't do anything. Right, this causes a build error. masahiro@grover:~/ref/linux$ make ARCH=arm64 VALIDATE_DT=1 arm/foundation-v8.dtb arch/arm64/Makefile:36: Detected assembler with broken .inst; disassembly will be unreliable DTC arch/arm64/boot/dts/arm/foundation-v8.dtb CHECK arch/arm64/boot/dts/arm/foundation-v8.dtb /home/masahiro/ref/linux/arch/arm64/boot/dts/arm/foundation-v8.dtb: sysreg@10000: '#address-cells' is a required property >From schema: /home/masahiro/ref/linux/Documentation/devicetree/bindings/arm/vexpress-sysreg.yaml /home/masahiro/ref/linux/arch/arm64/boot/dts/arm/foundation-v8.dtb: sysreg@10000: '#size-cells' is a required property >From schema: /home/masahiro/ref/linux/Documentation/devicetree/bindings/arm/vexpress-sysreg.yaml make[1]: *** No rule to make target 'arch/arm64/boot/dts/arm/foundation-v8.dt.yaml'. Stop. make: *** [Makefile:1379: arm/foundation-v8.dtb] Error 2 -- Best Regards Masahiro Yamada
