Change in coreboot[master]: [TESTME]asrock/imb-a180: fix the VGA-related PCI resource allocation ...
Mike Banon has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/38217 )
Change subject: [TESTME]asrock/imb-a180: fix the VGA-related PCI resource allocation problems ......................................................................
[TESTME]asrock/imb-a180: fix the VGA-related PCI resource allocation problems
This workaround helps to avoid the PCI resource allocation problems which happen if a discrete VGA adapter has been plugged in to ASRock IMB-A180 board. It does not bring any downsides if there is no discrete VGA adapter.
Signed-off-by: Mike Banon mikebdp2@gmail.com Change-Id: I00d509b0ad6b215fc257875687a2d645e6dfd00a --- M src/mainboard/asrock/imb-a180/OemCustomize.c 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/17/38217/1
diff --git a/src/mainboard/asrock/imb-a180/OemCustomize.c b/src/mainboard/asrock/imb-a180/OemCustomize.c index eaa27b7..ac7c9da 100644 --- a/src/mainboard/asrock/imb-a180/OemCustomize.c +++ b/src/mainboard/asrock/imb-a180/OemCustomize.c @@ -144,6 +144,8 @@
void board_BeforeInitPost(struct sysinfo *cb, AMD_POST_PARAMS *InitPost) { + /* Set to 0xD0 instead of 0xE0 to avoid the PCI resource allocation problems. */ + InitPost->MemConfig.BottomIo = 0xD0; InitPost->MemConfig.PlatformMemoryConfiguration = (PSO_ENTRY *)PlatformMemoryTable; }
Mike Banon has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38217 )
Change subject: [TESTME]asrock/imb-a180: fix the VGA-related PCI resource allocation problems ......................................................................
Patch Set 1:
Please review, hopefully you have a spare discrete GPU to test this.
Mike Banon has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38217 )
Change subject: [TESTME]asrock/imb-a180: fix the VGA-related PCI resource allocation problems ......................................................................
Patch Set 1:
Please review, hopefully you have a spare discrete GPU to test this.
Hello Stefan T, Kyösti Mälkki, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/38217
to look at the new patch set (#2).
Change subject: [TESTME]asrock/imb-a180: fix the VGA-related PCI resource allocation problems ......................................................................
[TESTME]asrock/imb-a180: fix the VGA-related PCI resource allocation problems
This workaround helps to avoid the PCI resource allocation problems which happen if a discrete VGA adapter has been plugged in to ASRock IMB-A180 board. It does not bring any downsides if there is no discrete VGA adapter.
Signed-off-by: Mike Banon mikebdp2@gmail.com Change-Id: I00d509b0ad6b215fc257875687a2d645e6dfd00a --- M src/mainboard/asrock/imb-a180/OemCustomize.c 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/17/38217/2
Mike Banon has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38217 )
Change subject: [TESTME]asrock/imb-a180: fix the VGA-related PCI resource allocation problems ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/38217/2/src/mainboard/asrock/imb-a1... File src/mainboard/asrock/imb-a180/OemCustomize.c:
https://review.coreboot.org/c/coreboot/+/38217/2/src/mainboard/asrock/imb-a1... PS2, Line 147: /* Set to 0x40 instead of 0xE0 to avoid the PCI resource allocation problems. */ : InitPost->MemConfig.BottomIo = 0x40; See CB:30987 comment here for more info about this change from 0xD0 to 0x40 - https://review.coreboot.org/c/coreboot/+/30987/45/src/mainboard/asus/a88xm-e...
Mike Banon has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/38217 )
Change subject: [TESTME]asrock/imb-a180: fix the VGA-related PCI resource allocation problems ......................................................................
Abandoned
Superseded by CB:38472 (amd/agesa: Make BottomIo position configurable)
-
Mike Banon (Code Review)