diff mbox series

Arm: arrange to have linker script honor dependencies (again)

Message ID 5c39e5c3-6e9e-e009-177b-495d80b6ea62@suse.com (mailing list archive)
State New, archived
Headers show
Series Arm: arrange to have linker script honor dependencies (again) | expand

Commit Message

Jan Beulich March 14, 2022, 5:16 p.m. UTC 
The if_changed macro leads to dependencies being recorded in .*.d, which
aren't loaded anymore. Use if_changed_dep instead, like x86 does.

Fixes: 06ef696c85a7 ("build: introduce if_changed_deps")
Signed-off-by: Jan Beulich <jbeulich@suse.com>

Comments

Julien Grall March 14, 2022, 5:51 p.m. UTC | #1 
Hi Jan,

On 14/03/2022 17:16, Jan Beulich wrote:
> The if_changed macro leads to dependencies being recorded in .*.d, which
> aren't loaded anymore. Use if_changed_dep instead, like x86 does.
> 
> Fixes: 06ef696c85a7 ("build: introduce if_changed_deps")
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Acked-by: Julien Grall <jgrall@amazon.com>

Cheers,
diff mbox series

Patch

--- a/xen/arch/arm/Makefile
+++ b/xen/arch/arm/Makefile
@@ -112,7 +112,7 @@  $(TARGET)-syms: $(BASEDIR)/prelink.o $(o
 include:
 
 $(obj)/xen.lds: $(src)/xen.lds.S FORCE
-	$(call if_changed,cpp_lds_S)
+	$(call if_changed_dep,cpp_lds_S)
 
 $(obj)/dtb.o: $(patsubst "%",%,$(CONFIG_DTB_FILE))