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:
Patch Set 3:
Well my criticism was mostly about the process, not the commit context.
I would have preferred to see the fundamental change of RAMPAYLOAD to be submitted, with documentation,
We are planning to write some document for sure. Thanks for feedback.
after the (pending) release. The concept is now submitted scattered over period of several months, and yet there is not a single 'select HAVE_RAMPAYLOAD' line.
Its WIP and hopefully we should see some sample RVP board to select HAVE_RAMPAYLOAD as POC.
So currently in upstream tree none of this concept is even build-tested?
Mainboard to select this feature is not available in upsteam thats true, but i have my POC running.
Is there published/pending work in gerrit that validates these changes on some level at least?
You can refer to below patch as mentioned previously. https://review.coreboot.org/c/coreboot/+/30985
idea is to divide this entire CL into smaller review worthy CL and submit to enable this feature, I think i have started with 75 files in initial CL and now its 63 files.