Commit 634ce97c authored by Michael Ellerman's avatar Michael Ellerman Committed by Shuah Khan

selftests: Fix the .S and .S -> .o rules

Both these rules incorrectly use $< (first prerequisite) rather than
$^ (all prerequisites), meaning they don't work if we're using more than
one .S file as input. Switch them to using $^.

They also don't include $(CPPFLAGS) and other variables used in the
default rules, which breaks targets that require those. Fix that by
using the builtin $(COMPILE.S) and $(LINK.S) rules.

Fixes: a8ba798b ("selftests: enable O and KBUILD_OUTPUT")
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
Tested by: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>
Signed-off-by: default avatarShuah Khan <shuahkh@osg.samsung.com>
parent 2047f1d8
...@@ -54,9 +54,9 @@ $(OUTPUT)/%:%.c ...@@ -54,9 +54,9 @@ $(OUTPUT)/%:%.c
$(LINK.c) $^ $(LDLIBS) -o $@ $(LINK.c) $^ $(LDLIBS) -o $@
$(OUTPUT)/%.o:%.S $(OUTPUT)/%.o:%.S
$(CC) $(ASFLAGS) -c $< -o $@ $(COMPILE.S) $^ -o $@
$(OUTPUT)/%:%.S $(OUTPUT)/%:%.S
$(CC) $(ASFLAGS) $< -o $@ $(LINK.S) $^ $(LDLIBS) -o $@
.PHONY: run_tests all clean install emit_tests .PHONY: run_tests all clean install emit_tests
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment