diff options
author | Mike Banon <mikebdp2@gmail.com> | 2020-01-19 21:42:09 +0300 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2020-04-22 13:49:05 +0000 |
commit | e7f176cd61a7ccb78ab9c015bb79b83ae6aaf071 (patch) | |
tree | e50f8edb9d792869da5050a6a01ce7743ca6af34 /src/northbridge/amd/agesa/family14/state_machine.c | |
parent | 4ac376a67b0be2a93a05f4fbf6e72ffab30c700f (diff) | |
download | coreboot-e7f176cd61a7ccb78ab9c015bb79b83ae6aaf071.tar.xz |
amd/agesa: Make BottomIo position configurable
Some PCI peripherals, such as discrete VGA adapters, require a great
amount of memory mapped IO. This patch allows the user to select at
build time the bottom IO to leave enough space for such devices.
We cannot calculate this value at runtime because it has to be set
before the PCI devices are enumerated. 0x80000000 has been successfully
boot-tested on A88XM-E (fam15tn), G505S (fam15tn) and AM1I-A (fam16kb).
Signed-off-by: Mike Banon <mikebdp2@gmail.com>
Change-Id: Ie235631231bcb4aeebaff2e0026da2ea9d82f9d0
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38472
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Michał Żygowski <michal.zygowski@3mdeb.com>
Diffstat (limited to 'src/northbridge/amd/agesa/family14/state_machine.c')
-rw-r--r-- | src/northbridge/amd/agesa/family14/state_machine.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/northbridge/amd/agesa/family14/state_machine.c b/src/northbridge/amd/agesa/family14/state_machine.c index db923c5ec9..5b0040899a 100644 --- a/src/northbridge/amd/agesa/family14/state_machine.c +++ b/src/northbridge/amd/agesa/family14/state_machine.c @@ -48,6 +48,8 @@ void platform_BeforeInitEarly(struct sysinfo *cb, AMD_EARLY_PARAMS *Early) void platform_BeforeInitPost(struct sysinfo *cb, AMD_POST_PARAMS *Post) { + Post->MemConfig.BottomIo = (UINT16)(MIN(0xE0000000, + MAX(0x28000000, CONFIG_BOTTOMIO_POSITION)) >> 24) & 0xF8; } void platform_AfterInitPost(struct sysinfo *cb, AMD_POST_PARAMS *Post) |