Sridhar Siricilla has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43766 )
Change subject: security/vboot/Makefile.inc: Update regions-for-file function ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/43766/1/src/security/vboot/Makefile... File src/security/vboot/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/43766/1/src/security/vboot/Makefile... PS1, Line 176: $(if $(filter \ : $(if $(filter y,$(CONFIG_VBOOT_STARTS_IN_ROMSTAGE)), \ : %/romstage,) \ : mts \ : %/verstage \ : locales \ : locale_%.bin \ : font.bin \ : vbgfx.bin \ : rmu.bin \ : cmos_layout.bin \ : cmos.default \ : $(call strip_quotes,$(CONFIG_RO_REGION_ONLY)) \ : ,$(1)),COREBOOT,\ : $(if $(filter \ : $(call strip_quotes,$(CONFIG_RWA_REGION_ONLY)) \ : ,$(1)), FW_MAIN_A, \ : $(if $(filter \ : $(call strip_quotes,$(CONFIG_RWB_REGION_ONLY)) \ : ,$(1)), FW_MAIN_B, \ : $(if $(filter \ : $(call strip_quotes,$(CONFIG_RW_REGION_ONLY)) \ : ,$(1)), $(RW_PARTITIONS), $(VBOOT_PARTITIONS) ) \
sorry, my pseudopatch in the comment was cheating... […]
Ack