[coreboot-gerrit] New patch to review for coreboot: Makefile.inc: Include build/dsdt.d if it exists

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Wed Dec 9 21:05:43 CET 2015


Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/12699

-gerrit

commit ccaf446287457c5d5b7e1d9217702e7f68f4c1c6
Author: Martin Roth <martinroth at google.com>
Date:   Wed Dec 9 13:05:00 2015 -0700

    Makefile.inc: Include build/dsdt.d if it exists
    
    The dsdt dependency file is created, but wasn't being used to determine
    whether or not to update the dsdt file.  If it's present, include it
    into the makefile so dsdt.aml gets rebuilt if any of the depencencies
    change.
    
    Change-Id: I76bc22541c6b9740841bda891a5b88030cb949cd
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 Makefile.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Makefile.inc b/Makefile.inc
index efdc238..2edd0ca 100644
--- a/Makefile.inc
+++ b/Makefile.inc
@@ -197,6 +197,7 @@ $(CONFIG_CBFS_PREFIX)/$(1).aml-file = $(obj)/$(1).aml
 $(CONFIG_CBFS_PREFIX)/$(1).aml-type = raw
 $(CONFIG_CBFS_PREFIX)/$(1).aml-compression = none
 cbfs-files-$(if $(2),$(2),y) += $(CONFIG_CBFS_PREFIX)/$(1).aml
+$(if $(wildcard $(obj)/$(1).d),$(eval include $(obj)/$(1).d))
 $(obj)/$(1).aml: $(src)/mainboard/$(MAINBOARDDIR)/$(1).asl $(obj)/config.h
 	@printf "    IASL       $$(subst $(top)/,,$$(@))\n"
 	$(CC_ramstage) -x assembler-with-cpp -E -MMD -MT $$(@) $$(CPPFLAGS_ramstage) -D__ACPI__ -P -include $(src)/include/kconfig.h -I$(obj) -I$(src) -I$(src)/include -I$(src)/arch/$(ARCHDIR-$(ARCH-ramstage-y))/include -I$(src)/mainboard/$(MAINBOARDDIR) $$< -o $$@



More information about the coreboot-gerrit mailing list