[coreboot-gerrit] Change in coreboot[master]: WIP: Add configurable ramstage support for minimal PCI scanning

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36221 )

Change subject: WIP: Add configurable ramstage support for minimal PCI scanning
......................................................................

Patch Set 14:
> Patch Set 14:
> > Patch Set 14:
> > > Patch Set 14: Code-Review+1
> >
> > Are we heading towards creating all PCI APIs with and without _always like been done for read_resources()?
> >
> > I believe our original goal is to block probe function for all other devices except the one marked as "mandatory" in devicetree.cb right ? @Ron
>
> Some of us (Aaron, Nico, me) seem to be much against (that original goal or approach) of partial PCI probing/enumeration due the undesirable side-effects mentioned along the review comments here. Furthermore, some of us seem to agree the real problem is with the resource allocation and not the PCI enumeration part.

if we divide the original definition of PCI enumeration then it combination of discovering the device, marking devices numbering and allocate the resource, enabling resource.
Not in static world we really don't have to do PCI device discover unless its on PCI-E root ports so resource allocation might be only significant things we have in hand right now. isn't it?