On Fri, Sep 15, 2017 at 09:52:41AM -0600, Stephen Warren wrote: > On 09/15/2017 09:50 AM, Stephen Warren wrote: > >On 09/04/2017 02:12 PM, Pantelis Antoniou wrote: > >>Special rule for dtbo generation > > > >>diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib > > > >>+quiet_cmd_dtco = DTCO $@ > >>+# Rule for objects only; does not put specific u-boot include at the end > >>+# No generation of assembly file either > >>+# Modified for U-Boot > >>+cmd_dtco = mkdir -p $(dir ${dtc-tmp}) ; \ > >>+ $(CPP) $(dtc_cpp_flags) -x assembler-with-cpp -o $(dtc-tmp) - ; \ > >>+ $(DTC) -@ -O dtb -o $@ -b 0 \ > >>+ -i $(dir $<) $(DTC_FLAGS) \ > >>+ -d $(depfile).dtc.tmp $(dtc-tmp) ; \ > > > >As of u-boot-fdt/master commit 503afc3ca1e5 "test: overlay: Add > >unit test for stacked overlay", the U-Boot build fails for sandbox > >with the error below. I guess it's due to this commit. Does U-Boot > >now intentionally rely on a newer dtc or is this an accident? > > > > DTC test/overlay/test-fdt-base.dtb > > LD lib/libfdt/built-in.o > > CC lib/charset.o > > CC lib/rsa/rsa-verify.o > >dtc: invalid option -- '@' > >Usage: dtc [options] <input file> > > > >Options: -[qI:O:o:V:d:R:S:p:fb:i:H:sW:E:hv] > > -q, --quiet > > Quiet: -q suppress warnings, -qq errors, -qqq all > >... > > Related, the following check in Makefile doesn't trigger: > > checkdtc: > @if test $(call dtc-version) -lt 0104; then \ > echo '*** Your dtc is too old, please upgrade to dtc 1.4 or > newer'; \ > false; \ > fi > > ... because I have dtc 1.4.0. If U-Boot requires a new version now, > that Makefile chunk should be updated.
Ah, this is what needs updating, yes, it needs to cehck for 1.4.3 or newer. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot