[coreboot-gerrit] Change in coreboot[master]: pci_mmio_cfg.h: Add a compile time error if MMCONF_BASE_ADDRESS is un...

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/35883 )

Change subject: pci_mmio_cfg.h: Add a compile time error if MMCONF_BASE_ADDRESS is undefined
......................................................................

pci_mmio_cfg.h: Add a compile time error if MMCONF_BASE_ADDRESS is undefined

if CONFIG_MMCONF_SUPPORT is set, add a compiletime error if
CONFIG_MMCONF_BASE_ADDRESS is not defined.

Change-Id: I0439e994d170e8ec564ce188e82a850e2a286a66
Signed-off-by: Arthur Heymans 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/35883
Reviewed-by: Nico Huber 
Tested-by: build bot (Jenkins) 
---
M src/include/device/pci_mmio_cfg.h
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Nico Huber: Looks good to me, approved

diff --git a/src/include/device/pci_mmio_cfg.h b/src/include/device/pci_mmio_cfg.h
index 5567ed8..e3c5fe4 100644
--- a/src/include/device/pci_mmio_cfg.h
+++ b/src/include/device/pci_mmio_cfg.h
@@ -90,6 +90,10 @@

#if CONFIG(MMCONF_SUPPORT)

+#if CONFIG_MMCONF_BASE_ADDRESS == 0
+#error "CONFIG_MMCONF_BASE_ADDRESS undefined!"
+#endif
+
 /* Avoid name collisions as different stages have different signature
  * for these functions. The _s_ stands for simple, fundamental IO or
  * MMIO variant.