Attention is currently required from: Arthur Heymans.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80465?usp=email )
Change subject: [UNTESTED]nb/intel/gm45: Use ssdt PCI root bridge generator ......................................................................
Patch Set 3:
(1 comment)
File src/northbridge/intel/gm45/northbridge.c:
https://review.coreboot.org/c/coreboot/+/80465/comment/65263bb3_fea5f68f : PS3, Line 111: struct resource *upper_pci = find_resource(dev, IOINDEX_SUBTRACTIVE(2, 0)); : if (upper_pci) : upper_pci->base = touud; : : struct resource *lower_pci = find_resource(dev, IOINDEX_SUBTRACTIVE(1, 0)); : if (lower_pci) : lower_pci->base = tolud;
It might be less confusing to just declare the bridge resources locally rather than patching them up […]
+1