[coreboot-gerrit] [S] Change in coreboot[master]: sb/intel/i82801gx: Move SPIBASE and SPIBARxx to i82801gx.h

[Elyes Haouas (Code Review)]

Elyes Haouas has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69899 )

Change subject: sb/intel/i82801gx: Move SPIBASE and SPIBARxx to i82801gx.h
......................................................................

sb/intel/i82801gx: Move SPIBASE and SPIBARxx to i82801gx.h

Also remove unused DEBUG_PERIODIC_SMIS macro.

Change-Id: Ieb8487c7757b89a09c1cee4a83f94b9077dac87e
Signed-off-by: Elyes Haouas 
---
M src/southbridge/intel/i82801gx/i82801gx.h
M src/southbridge/intel/i82801gx/lpc.c
2 files changed, 17 insertions(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/99/69899/1

diff --git a/src/southbridge/intel/i82801gx/i82801gx.h b/src/southbridge/intel/i82801gx/i82801gx.h
index cf76a40..0625db0 100644
--- a/src/southbridge/intel/i82801gx/i82801gx.h
+++ b/src/southbridge/intel/i82801gx/i82801gx.h
@@ -9,10 +9,13 @@

#include <southbridge/intel/common/rcba.h> /* IWYU pragma: export */

-#ifndef __ACPI__
-#define DEBUG_PERIODIC_SMIS 0
+#define SPIBASE 0x3020
+#define SPIBAR16(x) RCBA16(SPIBASE + x)
+#define SPIBAR32(x) RCBA32(SPIBASE + x)

+#ifndef __ACPI__
 #include <device/device.h>
+
 void i82801gx_enable(struct device *dev);

void i82801gx_lpc_setup(void);
diff --git a/src/southbridge/intel/i82801gx/lpc.c b/src/southbridge/intel/i82801gx/lpc.c
index 05156f7..99972f4 100644
--- a/src/southbridge/intel/i82801gx/lpc.c
+++ b/src/southbridge/intel/i82801gx/lpc.c
@@ -285,7 +285,6 @@
    }
 }

-#define SPIBASE 0x3020
 static void i82801gx_spi_init(void)
 {
    u16 spicontrol;
@@ -418,9 +417,6 @@
    }
 }

-#define SPIBAR16(x) RCBA16(0x3020 + x)
-#define SPIBAR32(x) RCBA32(0x3020 + x)
-
 static void lpc_final(struct device *dev)
 {
    u16 tco1_cnt;