On Sun, Jun 06, 2010 at 05:12:27PM -0400, Kevin O'Connor wrote:
On Fri, May 28, 2010 at 07:01:35PM +0900, Isaku Yamahata wrote:
Surprisingly iasl creates output file even when compilation error. So typing make after an error will succeed. This patch prevents it by removing the output file when error. And adds related dependencies to compile when .hex is missing.
[...]
####### dsdt build rules -src/acpi-dsdt.hex: src/acpi-dsdt.dsl +$(OUT)%.dsl.i: src/%.dsl
- $(Q)cpp -P $< > $@
+src/%.hex: $(OUT)%.dsl.i @echo "Compiling DSDT"
- $(Q)cpp -P $< > $(OUT)acpi-dsdt.dsl.i
- $(Q)iasl -tc -p $@ $(OUT)acpi-dsdt.dsl.i
- $(Q)rm $(OUT)acpi-dsdt.dsl.i
- $(Q)iasl -tc -p $@ $< || (e=$$?; rm $@; exit $$e)
+src/acpi.c: src/acpi-dsdt.hex +$(OUT)ccode32flat.o: src/acpi.c +$(OUT)../src/acpi-dsdt.hex: src/acpi-dsdt.hex
This doesn't look right - gcc should be emitting file dependencies on its own. How about:
Works for me. Thank you. I generalized it a bit so that the rule can be applied to other dsl. I'm working on qemu q35 chipset which requires its own dsdt other than the existing one.
--- a/Makefile +++ b/Makefile @@ -187,8 +187,10 @@ $(OUT)vgabios.bin: $(OUT)vgabios.bin.raw tools/buildrom.py src/acpi-dsdt.hex: src/acpi-dsdt.dsl @echo "Compiling DSDT" $(Q)cpp -P $< > $(OUT)acpi-dsdt.dsl.i
$(Q)iasl -tc -p $@ $(OUT)acpi-dsdt.dsl.i$(Q)rm $(OUT)acpi-dsdt.dsl.i
$(Q)iasl -tc -p $(OUT)acpi-dsdt $(OUT)acpi-dsdt.dsl.i$(Q)cp $(OUT)acpi-dsdt.hex $@+$(OUT)ccode32flat.o: src/acpi-dsdt.hex
####### Generic rules clean:
-Kevin
From 9c95b8e2c5379b5f4f06e7eb13d490418480a1ae Mon Sep 17 00:00:00 2001
Message-Id: 9c95b8e2c5379b5f4f06e7eb13d490418480a1ae.1275898861.git.yamahata@valinux.co.jp In-Reply-To: cover.1275898861.git.yamahata@valinux.co.jp References: cover.1275898861.git.yamahata@valinux.co.jp From: Isaku Yamahata yamahata@valinux.co.jp Date: Mon, 7 Jun 2010 17:19:27 +0900 Subject: [PATCH] seabios: remove iasl output file when error.
Surprisingly iasl creates output file even when compilation error. So typing make after an error will succeed. This patch prevents it by removing the output file when error. And adds related dependencies to compile when .hex is missing.
Signed-off-by: Kevin O'Connor kevin@koconnor.net Signed-off-by: Isaku Yamahata yamahata@valinux.co.jp
--- Changes v1 -> v2 - followed suggestion by Kevin. --- Makefile | 10 ++++++---- 1 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/Makefile b/Makefile index 72d711d..927999c 100644 --- a/Makefile +++ b/Makefile @@ -184,11 +184,13 @@ $(OUT)vgabios.bin: $(OUT)vgabios.bin.raw tools/buildrom.py $(Q)./tools/buildrom.py $< $@
####### dsdt build rules -src/acpi-dsdt.hex: src/acpi-dsdt.dsl +src/%.hex: src/%.dsl @echo "Compiling DSDT" - $(Q)cpp -P $< > $(OUT)acpi-dsdt.dsl.i - $(Q)iasl -tc -p $@ $(OUT)acpi-dsdt.dsl.i - $(Q)rm $(OUT)acpi-dsdt.dsl.i + $(Q)cpp -P $< > $(OUT)$*.dsl.i + $(Q)iasl -tc -p $(OUT)$* $(OUT)$*.dsl.i + $(Q)cp $(OUT)$*.hex $@ + +$(OUT)ccode32flat.o: src/acpi-dsdt.hex
####### Generic rules clean: