[coreboot-gerrit] Change in coreboot[master]: northbridge/amd/stoney: Reserve c0000-fffff

[Marc Jones (Code Review)]

Marc Jones has uploaded a new change for review. ( https://review.coreboot.org/19156 )

Change subject: northbridge/amd/stoney: Reserve c0000-fffff
......................................................................

northbridge/amd/stoney: Reserve c0000-fffff

Claim memory-mapped regions in the legacy area.

When the APU graphics is used, or if any add-in card is present and its
VBIOS initialized, claim all of 0xc0000-0xdffff.

If any BIOS tables are generated, claim all of 0xe0000-0xfffff.  This
specifically preserves the RSDP structure.

These changes allow code and information to be retained in the event
unused regions get wiped.

Original-Signed-off-by: Marshall Dawson <marshalldawson3rd at gmail.com>
Reviewed-by: Marc Jones <marcj303 at gmail.com>
(cherry picked from commit d612d4fe69881609d42053496409c452e1014947)

Change-Id: I9c47c919bbfd0edccf752e052f32d1e47c1a1324
Signed-off-by: Marc Jones <marcj303 at gmail.com>
---
M src/northbridge/amd/pi/00670F00/northbridge.c
1 file changed, 10 insertions(+), 0 deletions(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/56/19156/1

diff --git a/src/northbridge/amd/pi/00670F00/northbridge.c b/src/northbridge/amd/pi/00670F00/northbridge.c
index f9e986b..e2d1801 100644
--- a/src/northbridge/amd/pi/00670F00/northbridge.c
+++ b/src/northbridge/amd/pi/00670F00/northbridge.c
@@ -818,6 +818,16 @@
 			assign_resources(link);
 		}
 	}
+#if IS_ENABLED(CONFIG_GFXUMA) || IS_ENABLED(CONFIG_VGA_ROM_RUN)
+	mmio_resource(dev, 0xc000, 0xc0000 >> 10, 0x20000 >> 10);
+#endif
+#if (IS_ENABLED(CONFIG_GENERATE_PIRQ_TABLE) || \
+		IS_ENABLED(CONFIG_GENERATE_MP_TABLE) || \
+		IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)) || \
+		IS_ENABLED(CONFIG_GENERATE_SMBIOS_TABLES)
+	mmio_resource(dev, 0xe000, 0xe0000 >> 10, 0x20000 >> 10);
+#endif
+
 }
 
 static struct device_operations pci_domain_ops = {

-- 
To view, visit https://review.coreboot.org/19156
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9c47c919bbfd0edccf752e052f32d1e47c1a1324
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Marc Jones <marc at marcjonesconsulting.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd at gmail.com>