[coreboot-gerrit] Change in coreboot[master]: security/vboot/Makefile.inc: Update regions-for-file function

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak 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 1:

(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... these should all be indented another two levels, as this is the 'else' clause of line 174 (line 175 is the `then` clause)