[coreboot-gerrit] Change in coreboot[master]: nb/intel/i440bx: add resources during read_resources()

[Aaron Durbin (Code Review)]

Aaron Durbin has submitted this change. ( https://review.coreboot.org/c/coreboot/+/41368 )

Change subject: nb/intel/i440bx: add resources during read_resources()
......................................................................

nb/intel/i440bx: add resources during read_resources()

The chipset code was incorrectly adding memory resources
to the domain device after resource allocation occurred.
It's not possible to get the correct view of the address space,
and it's generally incorrect to not add resources during
read_resources(). This change fixes the order by adding resources in
read_resources().

Signed-off-by: Furquan Shaikh 
Change-Id: I84c1ba8645b548248a8bb8bf5bc4953d3be12475
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41368
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
Reviewed-by: Paul Menzel 
Reviewed-by: Keith Hui 
Reviewed-by: Aaron Durbin 
---
M src/northbridge/intel/i440bx/northbridge.c
1 file changed, 5 insertions(+), 4 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Paul Menzel: Looks good to me, but someone else must approve
  Aaron Durbin: Looks good to me, approved
  Keith Hui: Looks good to me, but someone else must approve
  Angel Pons: Looks good to me, but someone else must approve

diff --git a/src/northbridge/intel/i440bx/northbridge.c b/src/northbridge/intel/i440bx/northbridge.c
index aefd026..2659288 100644
--- a/src/northbridge/intel/i440bx/northbridge.c
+++ b/src/northbridge/intel/i440bx/northbridge.c
@@ -27,11 +27,13 @@
    .device = 0x7190,
 };

-static void i440bx_domain_set_resources(struct device *dev)
+static void i440bx_domain_read_resources(struct device *dev)
 {
    struct device *mc_dev;
    uint32_t pci_tolm;

+	pci_domain_read_resources(dev);
+
    pci_tolm = find_pci_tolm(dev->link_list);
    mc_dev = dev->link_list->children;
    if (mc_dev) {
@@ -62,12 +64,11 @@
    	ram_resource(dev, idx++, 0, 640);
    	ram_resource(dev, idx++, 768, tolmk - 768);
    }
-	assign_resources(dev->link_list);
 }

static struct device_operations pci_domain_ops = {
-	.read_resources		= pci_domain_read_resources,
-	.set_resources		= i440bx_domain_set_resources,
+	.read_resources		= i440bx_domain_read_resources,
+	.set_resources		= pci_domain_set_resources,
    .scan_bus		= pci_domain_scan_bus,
 };