[coreboot-gerrit] Patch set updated for coreboot: amd/pi: Make BottomIo position configurable

[Ricardo Ribalda Delgado (ricardo.ribalda@gmail.com)]

Ricardo Ribalda Delgado (ricardo.ribalda at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17980

-gerrit

commit a2bd1c96fdd2933062f3551ef3022634296da740
Author: Ricardo Ribalda Delgado <ricardo.ribalda at gmail.com>
Date:   Wed Dec 28 15:16:22 2016 +0100

    amd/pi: Make BottomIo position configurable
    
    Some PCI peripherals, such as FPGA accelerators, 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.
    
    Change-Id: Ic590e8aa8b91ff89877cbff6afd10614d33dcf8d
    Credit-to: Kyösti Mälkki <kyosti.malkki at gmail.com>
    Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda at gmail.com>
---
 src/northbridge/amd/pi/Kconfig        | 11 +++++++++++
 src/northbridge/amd/pi/agesawrapper.c |  3 ++-
 2 files changed, 13 insertions(+), 1 deletion(-)

diff --git a/src/northbridge/amd/pi/Kconfig b/src/northbridge/amd/pi/Kconfig
index 122b0e6..a4a3539 100644
--- a/src/northbridge/amd/pi/Kconfig
+++ b/src/northbridge/amd/pi/Kconfig
@@ -20,6 +20,17 @@ config NORTHBRIDGE_AMD_PI
 
 if NORTHBRIDGE_AMD_PI
 
+config BOTTOMIO_POSITION
+	hex "Bottom of 32-bit IO space"
+	default 0xD0000000
+	help
+	  If PCI peripherals with big BARs are connected to the system
+	  the bottom of the IO must be decreased to allocated such
+	  devices.
+
+	  Please note that the last 24 bits of this parameter are ignored
+	  by AGESA.
+
 config CONSOLE_VGA_MULTI
 	bool
 	default n
diff --git a/src/northbridge/amd/pi/agesawrapper.c b/src/northbridge/amd/pi/agesawrapper.c
index 8e16e75..0fe8eab 100644
--- a/src/northbridge/amd/pi/agesawrapper.c
+++ b/src/northbridge/amd/pi/agesawrapper.c
@@ -140,7 +140,8 @@ AGESA_STATUS agesawrapper_amdinitpost(void)
 	// the compiler to flag the error if CONFIG_GFXUMA is not set.
 	PostParams->MemConfig.UmaMode = CONFIG_GFXUMA ? UMA_AUTO : UMA_NONE;
 	PostParams->MemConfig.UmaSize = 0;
-	PostParams->MemConfig.BottomIo = (UINT16)(0xD0000000 >> 24);
+	PostParams->MemConfig.BottomIo = (UINT16)
+					 (CONFIG_BOTTOMIO_POSITION >> 24);
 	status = AmdInitPost (PostParams);
 	printk(
 			BIOS_SPEW,