On Wed, Mar 06, 2019 at 02:30:59PM +0100, Patrick Delaunay wrote: > The dependency for .dtb = .dts is not enough in dts Makefile, > as the dts files are dts pre-proprocessed (include dtsi files, > as specific *-u-boot.dtsi). > > For arm architecture, the dependency is correctly managed in > makefile of arch/arm/dts with .cmd files, it is needed to > execute make in this directory. > > Signed-off-by: Patrick Delaunay <[email protected]> > --- > Issue see on stm32mp1 board > > stm32mp157c-ed1.dts is included in stm32mp157c-ev1.dts > and dependency is not correctly managed > (ev1 is not recompiled when ed1 is modified) > > > dts/Makefile | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/dts/Makefile b/dts/Makefile > index a7a6043..bcd611f 100644 > --- a/dts/Makefile > +++ b/dts/Makefile > @@ -17,7 +17,6 @@ ifneq ($(EXT_DTB),) > DTB := $(EXT_DTB) > else > DTB := $(ARCH_PATH)/$(DEVICE_TREE).dtb > -dtb_depends += $(DTB:.dtb=.dts) > endif > > $(obj)/dt-spl.dtb: $(DTB) $(objtree)/tools/fdtgrep FORCE > @@ -28,7 +27,7 @@ $(obj)/dt.dtb: $(DTB) FORCE > > targets += dt.dtb dt-spl.dtb > > -$(DTB): $(dtb_depends) > +$(DTB): $(dtb_depends) FORCE > ifeq ($(EXT_DTB),) > $(Q)$(MAKE) $(build)=$(ARCH_PATH) $@ > endif
Yamada-san, what do you think of this? And I'm grabbing https://patchwork.ozlabs.org/patch/1052939/ shortly, thanks! -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
