Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36221 )
Change subject: Add configurable ramstage support for minimal PCI scanning ......................................................................
Patch Set 9:
I'm a little nonplussed by the comments.
I think the biggest problem is that people don't stay on topic. Some say it's about ENV_PAYLOAD_LOADER, some say it isn't. Some even say this is about resource allocation, which is actually orthogonal.
That shows that there is a lot interest in related things. I guess the mailing list would be a much better place because one can follow threads on the individual topics more easily there. And maybe we could even find the problem to your solution :)
Multiple vendors have shown a need for what this CL does.
I'd assume they work with hardware, while this only works in Qemu?
It's simple. It's off by default. It fixes a problem.
What's the issue here?
We're trying to help. But if you don't want to explain the problem you're trying to solve, we can't. End of discussion?
Nobody has given a -2 here, that means you can go on. Not every patch has to make everybody happy. If you want to evaluate this further and rather have the big bang discussion on the first change that affects more than a few lines, that's ok too. But don't blame anyone but your- self then, if this turns out as a dead end.