[U-Boot] [PATCH] dts: force dtb recompilation

[Tom Rini]

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 <patrick.delaunay at st.com>
> ---
> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190308/42645dc6/attachment.sig>