[coreboot-gerrit] Change in coreboot[master]: soc/amd/common: Handle I2C resource only if base address is defined

[Patrick Georgi (Code Review)]

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

Change subject: soc/amd/common: Handle I2C resource only if base address is defined
......................................................................

soc/amd/common: Handle I2C resource only if base address is defined

Change-Id: I767ad58f442bc5561bda4dd1de2d9593c8434615
Signed-off-by: Zheng Bao 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49959
Tested-by: build bot (Jenkins) 
Reviewed-by: Raul Rangel 
---
M src/soc/amd/common/block/lpc/lpc.c
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Raul Rangel: Looks good to me, approved

diff --git a/src/soc/amd/common/block/lpc/lpc.c b/src/soc/amd/common/block/lpc/lpc.c
index 0be5b74..b3f0e2d 100644
--- a/src/soc/amd/common/block/lpc/lpc.c
+++ b/src/soc/amd/common/block/lpc/lpc.c
@@ -125,11 +125,13 @@
    res->size = 0x00001000;
    res->flags = IORESOURCE_MEM | IORESOURCE_ASSIGNED | IORESOURCE_FIXED;

+#ifdef I2C_BASE_ADDRESS
    /* I2C devices */
    res = new_resource(dev, 4);
    res->base = I2C_BASE_ADDRESS;
    res->size = I2C_DEVICE_SIZE * I2C_DEVICE_COUNT;
    res->flags = IORESOURCE_MEM | IORESOURCE_ASSIGNED | IORESOURCE_FIXED;
+#endif

compact_resources(dev);
 }