[coreboot-gerrit] Change in ...coreboot[master]: Rampayload: Able to build coreboot without ramstage

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33142 )

Change subject: Rampayload: Able to build coreboot without ramstage
......................................................................

Patch Set 7:

(2 comments)

https://review.coreboot.org/#/c/33142/7/src/Kconfig 
File src/Kconfig:

https://review.coreboot.org/#/c/33142/7/src/Kconfig@1195 
PS7, Line 1195: config ENABLE_STAGE_BOOTBLOCK
It would be nice to align the naming with the already established
HAVE_* pattern in coreboot. e.g. HAVE_BOOTBLOCK, HAVE_VERSTAGE,
etc.

https://review.coreboot.org/#/c/33142/7/src/Kconfig@1199 
PS7, Line 1199: 	  This option is enabled to select bootblock architecture.
I don't understand what `architecture` refers to here? Maybe
this doesn't need a help text at all?