[coreboot-gerrit] Patch set updated for coreboot: build system: Add option for auto generated cbfs attributes

[Werner Zeh (werner.zeh@siemens.com)]

Werner Zeh (werner.zeh at siemens.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/12968

-gerrit

commit 571e2771675c9122592a5e043a30fd2566205484
Author: Werner Zeh <werner.zeh at siemens.com>
Date:   Thu Jan 14 15:08:36 2016 +0100

    build system: Add option for auto generated cbfs attributes
    
    Add a new Kconfig variable to enable the generation of
    position and alignment attributes for cbfs files which
    has constraints on this parameters.
    In addition,  modify Makefile.inc to support that option.
    
    Change-Id: Ibd725fe69a4de35964bdb2dde106d9a7c37ffb47
    Signed-off-by: Werner Zeh <werner.zeh at siemens.com>
---
 Makefile.inc | 11 ++++++++---
 src/Kconfig  |  8 ++++++++
 2 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/Makefile.inc b/Makefile.inc
index d7bb018..cd60c7e 100644
--- a/Makefile.inc
+++ b/Makefile.inc
@@ -647,6 +647,10 @@ $(objcbfs)/%.elf: $(objcbfs)/%.debug
 # 4) replace all '*' characters with spaces
 extract_nth=$(subst *,$(spc),$(patsubst -%-,%,$(word $(1), $(subst |,- -,-$(2)-))))
 
+ifeq ($(CONFIG_CBFS_AUTOGEN_ATTRIBUTES),y)
+	cbfs-autogen-attributes=-g
+endif
+
 cbfs-add-cmd = \
                printf "    CBFS       $(call extract_nth,2,$(file))\n"; \
 	       $(CBFSTOOL) $@.tmp \
@@ -654,7 +658,7 @@ cbfs-add-cmd = \
 	       -f $(call extract_nth,1,$(file)) \
 	       -n $(call extract_nth,2,$(file)) \
 	       $(if $(filter-out stage,$(call extract_nth,3,$(file))),-t $(call extract_nth,3,$(file))) \
-	       $(if $(call extract_nth,4,$(file)),-c $(call extract_nth,4,$(file))) \
+	       $(if $(call extract_nth,4,$(file)),-c $(call extract_nth,4,$(file))) $(cbfs-autogen-attributes)\
 	       $(call extract_nth,7,$(file))
 
 cbfs-files=$(cbfs-files-fixed) $(cbfs-files-aligned) $(cbfs-files-regular)
@@ -730,7 +734,7 @@ ifeq ($(CONFIG_ARCH_X86),y)
 		-f $(objcbfs)/bootblock.bin \
 		-n bootblock \
 		-t bootblock \
-		-b -$(call file-size,$(objcbfs)/bootblock.bin)
+		-b -$(call file-size,$(objcbfs)/bootblock.bin) $(cbfs-autogen-attributes)
 else
 	# don't add bootblock to cbfs yet, it's just a waste of space
 	true $(CBFSTOOL) $@.tmp add \
@@ -761,7 +765,8 @@ prebuild-files = \
 	$(foreach file,$(cbfs-files), \
 	$(CBFSTOOL) $@.tmp remove -n $(call extract_nth,2,$(file)) 2>/dev/null ; \
 	       $(if $(call extract_nth,6,$(file)),$(cbfs-add-cmd) -a $(call extract_nth,6,$(file)) &&,\
-	       $(cbfs-add-cmd) $(if $(call extract_nth,5,$(file)),-b $(call extract_nth,5,$(file))) &&))
+	       $(cbfs-add-cmd) $(if $(call extract_nth,5,$(file)),-b $(call extract_nth,5,$(file))) \
+	       $(cbfs-autogen-attributes) &&))
 
 .PHONY: $(obj)/coreboot.pre
 $(obj)/coreboot.pre: $$(prebuilt-files) $(CBFSTOOL)
diff --git a/src/Kconfig b/src/Kconfig
index f6702ea..3ff28c9 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -389,6 +389,14 @@ config SYSTEM_TYPE_LAPTOP
 	default n
 	bool
 
+config CBFS_AUTOGEN_ATTRIBUTES
+	default n
+	bool
+	help
+	  If this option is selected, every file in cbfs which has a constraint
+	  regarding position or alignment will get an additional file attribute
+	  which describes this constraint.
+
 menu "Chipset"
 
 comment "SoC"