[coreboot-gerrit] Change in coreboot[master]: AGESA f14/f15tn/f16kb: Factor out PCI MMIO base/size

20 May 2020

Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/41595 )
Change subject: AGESA f14/f15tn/f16kb: Factor out PCI MMIO base/size
......................................................................
AGESA f14/f15tn/f16kb: Factor out PCI MMIO base/size
We set BLDCFG_PCI_MMIO_BASE and BLDCFG_PCI_MMIO_SIZE to the same values
everywhere, so we might as well factor them out. We set them as fallback
so that we can override the settings using BLDCFG options if desired.
Change-Id: I7244c39d2c2aa02a3a9092ddae98e4ac9da89107
Signed-off-by: Angel Pons th3fanbus@gmail.com
---
M src/mainboard/amd/inagua/buildOpts.c
M src/mainboard/amd/olivehill/buildOpts.c
M src/mainboard/amd/parmer/buildOpts.c
M src/mainboard/amd/persimmon/buildOpts.c
M src/mainboard/amd/south_station/buildOpts.c
M src/mainboard/amd/thatcher/buildOpts.c
M src/mainboard/amd/union_station/buildOpts.c
M src/mainboard/asrock/e350m1/buildOpts.c
M src/mainboard/asrock/imb-a180/buildOpts.c
M src/mainboard/asus/am1i-a/buildOpts.c
M src/mainboard/asus/f2a85-m/buildOpts.c
M src/mainboard/bap/ode_e20XX/buildOpts.c
M src/mainboard/biostar/a68n_5200/buildOpts.c
M src/mainboard/biostar/am1ml/buildOpts.c
M src/mainboard/elmex/pcm205400/buildOpts.c
M src/mainboard/gizmosphere/gizmo/buildOpts.c
M src/mainboard/gizmosphere/gizmo2/buildOpts.c
M src/mainboard/hp/abm/buildOpts.c
M src/mainboard/hp/pavilion_m6_1035dx/buildOpts.c
M src/mainboard/jetway/nf81-t56n-lf/buildOpts.c
M src/mainboard/lenovo/g505s/buildOpts.c
M src/mainboard/lippert/frontrunner-af/buildOpts.c
M src/mainboard/lippert/toucan-af/buildOpts.c
M src/mainboard/msi/ms7721/buildOpts.c
M src/mainboard/pcengines/apu1/buildOpts.c
M src/vendorcode/amd/agesa/f14/Config/PlatformInstall.h
M src/vendorcode/amd/agesa/f15tn/Config/PlatformInstall.h
M src/vendorcode/amd/agesa/f16kb/Config/PlatformInstall.h
28 files changed, 6 insertions(+), 67 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/95/41595/1

diff --git a/src/mainboard/amd/inagua/buildOpts.c b/src/mainboard/amd/inagua/buildOpts.c
index 781dde3..4729577 100644
--- a/src/mainboard/amd/inagua/buildOpts.c
+++ b/src/mainboard/amd/inagua/buildOpts.c
@@ -54,9 +54,6 @@
 #define BLDOPT_REMOVE_GFX_RECOVERY		TRUE
 #define BLDOPT_REMOVE_EARLY_SAMPLES		TRUE
-#define BLDCFG_PCI_MMIO_BASE			CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE			CONFIG_MMCONF_BUS_NUMBER
-
 #define BLDCFG_VRM_CURRENT_LIMIT		24000
 //#define BLDCFG_VRM_NB_CURRENT_LIMIT		0
 #define BLDCFG_VRM_LOW_POWER_THRESHOLD		24000
diff --git a/src/mainboard/amd/olivehill/buildOpts.c b/src/mainboard/amd/olivehill/buildOpts.c
index dfcb465..d2bc4e7 100644
--- a/src/mainboard/amd/olivehill/buildOpts.c
+++ b/src/mainboard/amd/olivehill/buildOpts.c
@@ -66,8 +66,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                  15000
diff --git a/src/mainboard/amd/parmer/buildOpts.c b/src/mainboard/amd/parmer/buildOpts.c
index e1c71c3..a3cbb7e 100644
--- a/src/mainboard/amd/parmer/buildOpts.c
+++ b/src/mainboard/amd/parmer/buildOpts.c
@@ -63,8 +63,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                 90000
diff --git a/src/mainboard/amd/persimmon/buildOpts.c b/src/mainboard/amd/persimmon/buildOpts.c
index cda2d9d..5f18c39 100644
--- a/src/mainboard/amd/persimmon/buildOpts.c
+++ b/src/mainboard/amd/persimmon/buildOpts.c
@@ -54,9 +54,6 @@
 #define BLDOPT_REMOVE_GFX_RECOVERY				TRUE
 #define BLDOPT_REMOVE_EARLY_SAMPLES				TRUE
-#define BLDCFG_PCI_MMIO_BASE					CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE					CONFIG_MMCONF_BUS_NUMBER
-
 #define BLDCFG_VRM_CURRENT_LIMIT				24000
 //#define BLDCFG_VRM_NB_CURRENT_LIMIT			0
 #define BLDCFG_VRM_LOW_POWER_THRESHOLD			24000
diff --git a/src/mainboard/amd/south_station/buildOpts.c b/src/mainboard/amd/south_station/buildOpts.c
index 8bddb38..d3d65713 100644
--- a/src/mainboard/amd/south_station/buildOpts.c
+++ b/src/mainboard/amd/south_station/buildOpts.c
@@ -54,9 +54,6 @@
 #define BLDOPT_REMOVE_GFX_RECOVERY        TRUE
 #define BLDOPT_REMOVE_EARLY_SAMPLES            TRUE
-#define BLDCFG_PCI_MMIO_BASE                    CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE                    CONFIG_MMCONF_BUS_NUMBER
-
 #define BLDCFG_VRM_CURRENT_LIMIT                24000
 //#define BLDCFG_VRM_NB_CURRENT_LIMIT             0
 #define BLDCFG_VRM_LOW_POWER_THRESHOLD          24000
diff --git a/src/mainboard/amd/thatcher/buildOpts.c b/src/mainboard/amd/thatcher/buildOpts.c
index e1c3cd2..53929b8 100644
--- a/src/mainboard/amd/thatcher/buildOpts.c
+++ b/src/mainboard/amd/thatcher/buildOpts.c
@@ -63,8 +63,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                 90000
diff --git a/src/mainboard/amd/union_station/buildOpts.c b/src/mainboard/amd/union_station/buildOpts.c
index 8bddb38..d3d65713 100644
--- a/src/mainboard/amd/union_station/buildOpts.c
+++ b/src/mainboard/amd/union_station/buildOpts.c
@@ -54,9 +54,6 @@
 #define BLDOPT_REMOVE_GFX_RECOVERY        TRUE
 #define BLDOPT_REMOVE_EARLY_SAMPLES            TRUE
-#define BLDCFG_PCI_MMIO_BASE                    CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE                    CONFIG_MMCONF_BUS_NUMBER
-
 #define BLDCFG_VRM_CURRENT_LIMIT                24000
 //#define BLDCFG_VRM_NB_CURRENT_LIMIT             0
 #define BLDCFG_VRM_LOW_POWER_THRESHOLD          24000
diff --git a/src/mainboard/asrock/e350m1/buildOpts.c b/src/mainboard/asrock/e350m1/buildOpts.c
index 0cd14d7..aab03de 100644
--- a/src/mainboard/asrock/e350m1/buildOpts.c
+++ b/src/mainboard/asrock/e350m1/buildOpts.c
@@ -62,9 +62,6 @@
  * needed by the system.
  */
-#define BLDCFG_PCI_MMIO_BASE                    CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE                    CONFIG_MMCONF_BUS_NUMBER
-
 #define BLDCFG_VRM_CURRENT_LIMIT                24000
 //#define BLDCFG_VRM_NB_CURRENT_LIMIT             0
 #define BLDCFG_VRM_LOW_POWER_THRESHOLD          24000
diff --git a/src/mainboard/asrock/imb-a180/buildOpts.c b/src/mainboard/asrock/imb-a180/buildOpts.c
index 97d49c7..99b6180 100644
--- a/src/mainboard/asrock/imb-a180/buildOpts.c
+++ b/src/mainboard/asrock/imb-a180/buildOpts.c
@@ -66,8 +66,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                  15000
diff --git a/src/mainboard/asus/am1i-a/buildOpts.c b/src/mainboard/asus/am1i-a/buildOpts.c
index e2c01fc..a364982 100644
--- a/src/mainboard/asus/am1i-a/buildOpts.c
+++ b/src/mainboard/asus/am1i-a/buildOpts.c
@@ -82,8 +82,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                 15000
diff --git a/src/mainboard/asus/f2a85-m/buildOpts.c b/src/mainboard/asus/f2a85-m/buildOpts.c
index 6c368cb..38d302e 100644
--- a/src/mainboard/asus/f2a85-m/buildOpts.c
+++ b/src/mainboard/asus/f2a85-m/buildOpts.c
@@ -76,8 +76,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                 90000
diff --git a/src/mainboard/bap/ode_e20XX/buildOpts.c b/src/mainboard/bap/ode_e20XX/buildOpts.c
index 0d98c02..2bcabf4 100644
--- a/src/mainboard/bap/ode_e20XX/buildOpts.c
+++ b/src/mainboard/bap/ode_e20XX/buildOpts.c
@@ -66,8 +66,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                  15000
diff --git a/src/mainboard/biostar/a68n_5200/buildOpts.c b/src/mainboard/biostar/a68n_5200/buildOpts.c
index dfcb465..d2bc4e7 100644
--- a/src/mainboard/biostar/a68n_5200/buildOpts.c
+++ b/src/mainboard/biostar/a68n_5200/buildOpts.c
@@ -66,8 +66,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                  15000
diff --git a/src/mainboard/biostar/am1ml/buildOpts.c b/src/mainboard/biostar/am1ml/buildOpts.c
index 5e9f256..b9a3efe 100644
--- a/src/mainboard/biostar/am1ml/buildOpts.c
+++ b/src/mainboard/biostar/am1ml/buildOpts.c
@@ -66,8 +66,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                  15000
diff --git a/src/mainboard/elmex/pcm205400/buildOpts.c b/src/mainboard/elmex/pcm205400/buildOpts.c
index bbc8f00..8c34cd6 100644
--- a/src/mainboard/elmex/pcm205400/buildOpts.c
+++ b/src/mainboard/elmex/pcm205400/buildOpts.c
@@ -54,9 +54,6 @@
 #define BLDOPT_REMOVE_GFX_RECOVERY				TRUE
 #define BLDOPT_REMOVE_EARLY_SAMPLES				TRUE
-#define BLDCFG_PCI_MMIO_BASE					CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE					CONFIG_MMCONF_BUS_NUMBER
-
 #define BLDCFG_VRM_CURRENT_LIMIT				24000
 //#define BLDCFG_VRM_NB_CURRENT_LIMIT			0
 #define BLDCFG_VRM_LOW_POWER_THRESHOLD			24000
diff --git a/src/mainboard/gizmosphere/gizmo/buildOpts.c b/src/mainboard/gizmosphere/gizmo/buildOpts.c
index cda2d9d..5f18c39 100644
--- a/src/mainboard/gizmosphere/gizmo/buildOpts.c
+++ b/src/mainboard/gizmosphere/gizmo/buildOpts.c
@@ -54,9 +54,6 @@
 #define BLDOPT_REMOVE_GFX_RECOVERY				TRUE
 #define BLDOPT_REMOVE_EARLY_SAMPLES				TRUE
-#define BLDCFG_PCI_MMIO_BASE					CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE					CONFIG_MMCONF_BUS_NUMBER
-
 #define BLDCFG_VRM_CURRENT_LIMIT				24000
 //#define BLDCFG_VRM_NB_CURRENT_LIMIT			0
 #define BLDCFG_VRM_LOW_POWER_THRESHOLD			24000
diff --git a/src/mainboard/gizmosphere/gizmo2/buildOpts.c b/src/mainboard/gizmosphere/gizmo2/buildOpts.c
index 0d98c02..2bcabf4 100644
--- a/src/mainboard/gizmosphere/gizmo2/buildOpts.c
+++ b/src/mainboard/gizmosphere/gizmo2/buildOpts.c
@@ -66,8 +66,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                  15000
diff --git a/src/mainboard/hp/abm/buildOpts.c b/src/mainboard/hp/abm/buildOpts.c
index 7ebf119..a649c73 100644
--- a/src/mainboard/hp/abm/buildOpts.c
+++ b/src/mainboard/hp/abm/buildOpts.c
@@ -71,8 +71,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                  15000
diff --git a/src/mainboard/hp/pavilion_m6_1035dx/buildOpts.c b/src/mainboard/hp/pavilion_m6_1035dx/buildOpts.c
index 17b5804..e7ff996 100644
--- a/src/mainboard/hp/pavilion_m6_1035dx/buildOpts.c
+++ b/src/mainboard/hp/pavilion_m6_1035dx/buildOpts.c
@@ -77,8 +77,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                 90000
diff --git a/src/mainboard/jetway/nf81-t56n-lf/buildOpts.c b/src/mainboard/jetway/nf81-t56n-lf/buildOpts.c
index 02a416a..a20d133 100644
--- a/src/mainboard/jetway/nf81-t56n-lf/buildOpts.c
+++ b/src/mainboard/jetway/nf81-t56n-lf/buildOpts.c
@@ -68,9 +68,6 @@
 #define BLDOPT_REMOVE_GFX_RECOVERY				TRUE
 #define BLDOPT_REMOVE_EARLY_SAMPLES				TRUE
-#define BLDCFG_PCI_MMIO_BASE					CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE					CONFIG_MMCONF_BUS_NUMBER
-
 #define BLDCFG_VRM_CURRENT_LIMIT				24000
 //#define BLDCFG_VRM_NB_CURRENT_LIMIT			0
 #define BLDCFG_VRM_LOW_POWER_THRESHOLD			24000
diff --git a/src/mainboard/lenovo/g505s/buildOpts.c b/src/mainboard/lenovo/g505s/buildOpts.c
index 789bb40..f4993d6 100644
--- a/src/mainboard/lenovo/g505s/buildOpts.c
+++ b/src/mainboard/lenovo/g505s/buildOpts.c
@@ -77,8 +77,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                 90000
diff --git a/src/mainboard/lippert/frontrunner-af/buildOpts.c b/src/mainboard/lippert/frontrunner-af/buildOpts.c
index cda2d9d..5f18c39 100644
--- a/src/mainboard/lippert/frontrunner-af/buildOpts.c
+++ b/src/mainboard/lippert/frontrunner-af/buildOpts.c
@@ -54,9 +54,6 @@
 #define BLDOPT_REMOVE_GFX_RECOVERY				TRUE
 #define BLDOPT_REMOVE_EARLY_SAMPLES				TRUE
-#define BLDCFG_PCI_MMIO_BASE					CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE					CONFIG_MMCONF_BUS_NUMBER
-
 #define BLDCFG_VRM_CURRENT_LIMIT				24000
 //#define BLDCFG_VRM_NB_CURRENT_LIMIT			0
 #define BLDCFG_VRM_LOW_POWER_THRESHOLD			24000
diff --git a/src/mainboard/lippert/toucan-af/buildOpts.c b/src/mainboard/lippert/toucan-af/buildOpts.c
index cda2d9d..5f18c39 100644
--- a/src/mainboard/lippert/toucan-af/buildOpts.c
+++ b/src/mainboard/lippert/toucan-af/buildOpts.c
@@ -54,9 +54,6 @@
 #define BLDOPT_REMOVE_GFX_RECOVERY				TRUE
 #define BLDOPT_REMOVE_EARLY_SAMPLES				TRUE
-#define BLDCFG_PCI_MMIO_BASE					CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE					CONFIG_MMCONF_BUS_NUMBER
-
 #define BLDCFG_VRM_CURRENT_LIMIT				24000
 //#define BLDCFG_VRM_NB_CURRENT_LIMIT			0
 #define BLDCFG_VRM_LOW_POWER_THRESHOLD			24000
diff --git a/src/mainboard/msi/ms7721/buildOpts.c b/src/mainboard/msi/ms7721/buildOpts.c
index aaa88b6a..4029bc3 100644
--- a/src/mainboard/msi/ms7721/buildOpts.c
+++ b/src/mainboard/msi/ms7721/buildOpts.c
@@ -76,8 +76,6 @@
//#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT  TRUE
-#define BLDCFG_PCI_MMIO_BASE	CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE	CONFIG_MMCONF_BUS_NUMBER
 /* Build configuration values here.
  */
 #define BLDCFG_VRM_CURRENT_LIMIT                 90000
diff --git a/src/mainboard/pcengines/apu1/buildOpts.c b/src/mainboard/pcengines/apu1/buildOpts.c
index 4780c33..2bf1437 100644
--- a/src/mainboard/pcengines/apu1/buildOpts.c
+++ b/src/mainboard/pcengines/apu1/buildOpts.c
@@ -54,9 +54,6 @@
 #define BLDOPT_REMOVE_GFX_RECOVERY				TRUE
 #define BLDOPT_REMOVE_EARLY_SAMPLES				TRUE
-#define BLDCFG_PCI_MMIO_BASE					CONFIG_MMCONF_BASE_ADDRESS
-#define BLDCFG_PCI_MMIO_SIZE					CONFIG_MMCONF_BUS_NUMBER
-
 #define BLDCFG_VRM_CURRENT_LIMIT				24000
 //#define BLDCFG_VRM_NB_CURRENT_LIMIT			0
 #define BLDCFG_VRM_LOW_POWER_THRESHOLD			24000
diff --git a/src/vendorcode/amd/agesa/f14/Config/PlatformInstall.h b/src/vendorcode/amd/agesa/f14/Config/PlatformInstall.h
index eee3a4b..d5060d6 100644
--- a/src/vendorcode/amd/agesa/f14/Config/PlatformInstall.h
+++ b/src/vendorcode/amd/agesa/f14/Config/PlatformInstall.h
@@ -1084,13 +1084,13 @@
 #ifdef BLDCFG_PCI_MMIO_BASE
   #define CFG_PCI_MMIO_BASE               (BLDCFG_PCI_MMIO_BASE)
 #else
-  #define CFG_PCI_MMIO_BASE               (0)
+  #define CFG_PCI_MMIO_BASE               (CONFIG_MMCONF_BASE_ADDRESS)
 #endif
#ifdef BLDCFG_PCI_MMIO_SIZE
   #define CFG_PCI_MMIO_SIZE               (BLDCFG_PCI_MMIO_SIZE)
 #else
-  #define CFG_PCI_MMIO_SIZE               (0)
+  #define CFG_PCI_MMIO_SIZE               (CONFIG_MMCONF_BUS_NUMBER)
 #endif
#ifdef BLDCFG_AP_MTRR_SETTINGS_LIST
diff --git a/src/vendorcode/amd/agesa/f15tn/Config/PlatformInstall.h b/src/vendorcode/amd/agesa/f15tn/Config/PlatformInstall.h
index 70c2135..6442064 100644
--- a/src/vendorcode/amd/agesa/f15tn/Config/PlatformInstall.h
+++ b/src/vendorcode/amd/agesa/f15tn/Config/PlatformInstall.h
@@ -2421,13 +2421,13 @@
 #ifdef BLDCFG_PCI_MMIO_BASE
   #define CFG_PCI_MMIO_BASE               (BLDCFG_PCI_MMIO_BASE)
 #else
-  #define CFG_PCI_MMIO_BASE               (0)
+  #define CFG_PCI_MMIO_BASE               (CONFIG_MMCONF_BASE_ADDRESS)
 #endif
#ifdef BLDCFG_PCI_MMIO_SIZE
   #define CFG_PCI_MMIO_SIZE               (BLDCFG_PCI_MMIO_SIZE)
 #else
-  #define CFG_PCI_MMIO_SIZE               (0)
+  #define CFG_PCI_MMIO_SIZE               (CONFIG_MMCONF_BUS_NUMBER)
 #endif
#ifdef BLDCFG_AP_MTRR_SETTINGS_LIST
diff --git a/src/vendorcode/amd/agesa/f16kb/Config/PlatformInstall.h b/src/vendorcode/amd/agesa/f16kb/Config/PlatformInstall.h
index 90c4e3f..b74753d 100644
--- a/src/vendorcode/amd/agesa/f16kb/Config/PlatformInstall.h
+++ b/src/vendorcode/amd/agesa/f16kb/Config/PlatformInstall.h
@@ -1429,13 +1429,13 @@
 #ifdef BLDCFG_PCI_MMIO_BASE
   #define CFG_PCI_MMIO_BASE               (BLDCFG_PCI_MMIO_BASE)
 #else
-  #define CFG_PCI_MMIO_BASE               (0)
+  #define CFG_PCI_MMIO_BASE               (CONFIG_MMCONF_BASE_ADDRESS)
 #endif
#ifdef BLDCFG_PCI_MMIO_SIZE
   #define CFG_PCI_MMIO_SIZE               (BLDCFG_PCI_MMIO_SIZE)
 #else
-  #define CFG_PCI_MMIO_SIZE               (0)
+  #define CFG_PCI_MMIO_SIZE               (CONFIG_MMCONF_BUS_NUMBER)
 #endif
#ifdef BLDCFG_AP_MTRR_SETTINGS_LIST
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/41595
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I7244c39d2c2aa02a3a9092ddae98e4ac9da89107
Gerrit-Change-Number: 41595
Gerrit-PatchSet: 1
Gerrit-Owner: Angel Pons th3fanbus@gmail.com
Gerrit-Reviewer: Alexander Couzens lynxis@fe80.eu
Gerrit-Reviewer: Michał Żygowski michal.zygowski@3mdeb.com
Gerrit-Reviewer: Patrick Rudolph siro@das-labor.org
Gerrit-Reviewer: Piotr Król piotr.krol@3mdeb.com
Gerrit-MessageType: newchange



    

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

[coreboot-gerrit] Change in coreboot[master]: AGESA f14/f15tn/f16kb: Factor out PCI MMIO base/size