[coreboot-gerrit] Change in coreboot[master]: 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: Add configurable ramstage support for minimal PCI scanning
......................................................................

Patch Set 17:

(3 comments)

https://review.coreboot.org/c/coreboot/+/36221/9/src/Kconfig 
File src/Kconfig:

https://review.coreboot.org/c/coreboot/+/36221/9/src/Kconfig@340 
PS9, Line 340: 	default y if ARCH_X86
> Why the arch dependency?

i can clean this up later

https://review.coreboot.org/c/coreboot/+/36221/9/src/Kconfig@348 
PS9, Line 348: 	help
> This option should only be visible for boards with devicetree. […]

i can clean this up later

https://review.coreboot.org/c/coreboot/+/36221/11/src/device/pci_device.c 
File src/device/pci_device.c:

https://review.coreboot.org/c/coreboot/+/36221/11/src/device/pci_device.c@94... 
PS11, Line 943: 	if (CONFIG(MINIMAL_PCI_SCANNING) && driver && !driver->always_scan) {
> I think we also want […]

Ack