Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32751 )
Change subject: Set ENV_PAYLOAD_LOADER to ENV_POSTCAR when CONFIG_RAMPAYLOAD is enabled ......................................................................
Patch Set 11:
Patch Set 11: Code-Review+2
seems fine now. I can't wait to use it :-)
yes, i guess we are making some progress now in order to build coreboot without ramstage (thanks to add reviewers specifically Aaron and Julius)
now we need to add basic functionality into postcar to boot with full feature. Now here is 2 challenge
1. Dynamic generation of ASL code as we are not executing BS machine. I'm planning to create a smaller devicetree.cb with only required devices entry for dynamic ASL creation in postcar
2. Dynamic PCI BAR allocation based on free main memory to boot from boot media as depthcharge will expect to have BAR assigned. In absence of BS machine, dynamic BAR allocation will be key. hence a limited devicetree.cb in postcar to allocated #1 and #2 might solve these problem.
Like to hear more thoughts. remaining i have good visibility about how much code needs to put into postcar apart from #1 and #2 hack right now i'm having in code.