[coreboot-gerrit] Change in ...coreboot[master]: Kconfig: Enable RAMPAYLOAD for x86

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33859 )

Change subject: Kconfig: Enable RAMPAYLOAD for x86
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/#/c/33859/3//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/33859/3//COMMIT_MSG@10 
PS3, Line 10: of HAVE_RAMPAYLOAD kconfig from mainboard for x86 platform.
> i mean in case of HAVE_FSP_GOP selection is enough to select RUN_FSP_GOP, mainboard doesn't really n […]

I don't think those equivalent for comparison purposes. In this CL you are changing the whole boot flow for a mainboard once HAVE is selected. It would require someone to deselect that option if they didn't want the behavior. But maybe it's fine. I was thinking HAVE_RAMPAYLOAD as a "has support" thing, but it's really more of a 'there is a rampayload'. So I guess it's fine?