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

[Subrata Banik (Code Review)]

Subrata Banik 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 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?

yes, i also think so