Message ID | 20220803120826.1.I1f0fa17bc2141cbfac75a841e1e86d79516a07b3@changeid |
---|---|
State | Accepted |
Commit | c252fa5cdbb421bbcca750866e5c5f9f6c8c1c34 |
Delegated to: | Tom Rini |
Headers | show |
Series | Makefile: Correct the rule removing old of-platdata files | expand |
On Wed, Aug 03, 2022 at 12:08:29PM -0600, Simon Glass wrote: > This makes use of makefile variables that don't exist anymore. Fix it and > also remove the object files in that directory. > > Also add FORCE as a dependency as required by the if_changed macro. > > Fixes 354d2324635 ("Makefile: Remove old of-platdata files before regenerating") > Reported-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com> > Signed-off-by: Simon Glass <sjg@chromium.org> Applied to u-boot/master, thanks!
diff --git a/scripts/Makefile.spl b/scripts/Makefile.spl index 1cfb8115e31..3bafeb4fe98 100644 --- a/scripts/Makefile.spl +++ b/scripts/Makefile.spl @@ -358,13 +358,14 @@ $(obj)/dts/dt-%.o: $(obj)/dts/dt-%.c $(platdata-hdr) $(call if_changed,plat) # Don't use dts_dir here, since it forces running this expensive rule every time -$(platdata-hdr) $(u-boot-spl-platdata_c) &: $(obj)/$(SPL_BIN).dtb +$(platdata-hdr) $(u-boot-spl-platdata_c) &: $(obj)/$(SPL_BIN).dtb FORCE @[ -d $(obj)/dts ] || mkdir -p $(obj)/dts @# Remove old files since which ones we generate depends on the setting @# of OF_PLATDATA_INST and this might change between builds. Leaving old @# ones around is confusing and it is possible that switching the @# setting again will use the old one instead of regenerating it. - @rm -f $(u-boot-spl-all-platdata_c) $(u-boot-spl-all-platdata) + @rm -f $(u-boot-spl-old-platdata_c) $(u-boot-spl-platdata_c) \ + $(u-boot-spl-old-platdata) $(call if_changed,dtoc) ifneq ($(CONFIG_ARCH_EXYNOS)$(CONFIG_ARCH_S5PC1XX),)
This makes use of makefile variables that don't exist anymore. Fix it and also remove the object files in that directory. Also add FORCE as a dependency as required by the if_changed macro. Fixes 354d2324635 ("Makefile: Remove old of-platdata files before regenerating") Reported-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com> Signed-off-by: Simon Glass <sjg@chromium.org> --- scripts/Makefile.spl | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)