[U-Boot] [PATCH] dts: force dtb recompilation
Masahiro Yamada
yamada.masahiro at socionext.com
Sat Mar 9 15:57:18 UTC 2019
Hi Patrick,
On Wed, Mar 6, 2019 at 10:31 PM Patrick Delaunay
<patrick.delaunay at st.com> 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)
Do you still see this issue?
I do not see the problem any more
because the following commit fixed it.
commit 2737dfe096b6c34654734a5a4dc5f4b4962c5617
Author: Stephen Warren <swarren at nvidia.com>
Date: Tue Feb 26 12:20:25 2019 -0700
kbuild: make arch-dtbs target PHONY
Could you test it on the latest git version?
>
> 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
> --
> 2.7.4
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot
--
Best Regards
Masahiro Yamada
More information about the U-Boot
mailing list