Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41553 )
Change subject: device/pci: Don't enabled unassigned resources ......................................................................
Patch Set 1:
(2 comments)
This change is ready for review.
https://review.coreboot.org/c/coreboot/+/41553/1/src/device/pci_device.c File src/device/pci_device.c:
https://review.coreboot.org/c/coreboot/+/41553/1/src/device/pci_device.c@517 PS1, Line 517: resource->size = 0; can we just call pci_store_bridge_resource() here and return?
https://review.coreboot.org/c/coreboot/+/41553/1/src/device/pci_device.c@558 PS1, Line 558: resource->flags |= IORESOURCE_STORED; Shouldn't we set this in the respective functions (pci_store_bridge_resource() and pci_store_resource()) as well as the accompanying report_resource_stored()?