[coreboot-gerrit] [S] Change in coreboot[master]: top/Makefile.inc: Define regions-for-file with a flexibility

[Bao Zheng (Code Review)]

Attention is currently required from: Martin L Roth, Zheng Bao.

Hello Zheng Bao,

I'd like you to do a code review.
Please visit

https://review.coreboot.org/c/coreboot/+/70313

to review the following change.

Change subject: top/Makefile.inc: Define regions-for-file with a flexibility
......................................................................

top/Makefile.inc: Define regions-for-file with a flexibility

If we need to put a CBFS chunk into a specific region, add a line in
any Makefile.inc

regions-for-file-xxx=region_name

Change-Id: Ie37a8a9230dc8b8e5664be8806f047afb94fba69
Signed-off-by: Zheng Bao 
---
M Makefile.inc
1 file changed, 16 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/13/70313/1

diff --git a/Makefile.inc b/Makefile.inc
index ef565c2..0c2d3c0 100644
--- a/Makefile.inc
+++ b/Makefile.inc
@@ -814,7 +814,7 @@
 #
 # This is the default implementation. When using a boot strategy employing
 # multiple CBFSes in fmap regions, override it.
-regions-for-file ?= COREBOOT
+regions-for-file ?= $(if $(value regions-for-file-$(1)), $(regions-for-file-$(1)), COREBOOT)

ifeq ($(CONFIG_CBFS_AUTOGEN_ATTRIBUTES),y)
    cbfs-autogen-attributes=-g