[coreboot-gerrit] Patch set updated for coreboot: amd/thatcher: Removed #include early_serial.c from romstage

[Antonello Dettori (dev@dettori.io)]

Antonello Dettori (dev at dettori.io) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13887

-gerrit

commit f3ffc50c6725ede2fa4a34e7c81a3db3235934bf
Author: Antonello Dettori <dev at dettori.io>
Date:   Thu Mar 3 16:25:35 2016 +0000

    amd/thatcher: Removed #include early_serial.c from romstage
    
    Remove dependency on early_serial.c and instead use the
    Super I/O's header to access the functions needed.
    
    Change-Id: I9edf7fc2501aa832106dda9213e702dbcc1200b4
    Signed-off-by: Antonello Dettori <dev at dettori.io>
---
 src/mainboard/amd/thatcher/romstage.c     | 2 +-
 src/superio/smsc/lpc47n217/Makefile.inc   | 1 +
 src/superio/smsc/lpc47n217/early_serial.c | 2 +-
 src/superio/smsc/lpc47n217/lpc47n217.h    | 5 +++++
 4 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/mainboard/amd/thatcher/romstage.c b/src/mainboard/amd/thatcher/romstage.c
index 10b7df1..4c9c8fa 100644
--- a/src/mainboard/amd/thatcher/romstage.c
+++ b/src/mainboard/amd/thatcher/romstage.c
@@ -30,7 +30,7 @@
 #include <cpu/x86/bist.h>
 #include <cpu/x86/lapic.h>
 #include <southbridge/amd/agesa/hudson/hudson.h>
-#include <superio/smsc/lpc47n217/early_serial.c>
+#include <superio/smsc/lpc47n217/lpc47n217.h>
 #include <cpu/amd/agesa/s3_resume.h>
 #include "cbmem.h"
 
diff --git a/src/superio/smsc/lpc47n217/Makefile.inc b/src/superio/smsc/lpc47n217/Makefile.inc
index f3c5ad4..0bff637 100644
--- a/src/superio/smsc/lpc47n217/Makefile.inc
+++ b/src/superio/smsc/lpc47n217/Makefile.inc
@@ -14,4 +14,5 @@
 ## GNU General Public License for more details.
 ##
 
+romstage-$(CONFIG_SUPERIO_SMSC_LPC47N217) += early_serial.c
 ramstage-$(CONFIG_SUPERIO_SMSC_LPC47N217) += superio.c
diff --git a/src/superio/smsc/lpc47n217/early_serial.c b/src/superio/smsc/lpc47n217/early_serial.c
index dfe06b9..454d77f 100644
--- a/src/superio/smsc/lpc47n217/early_serial.c
+++ b/src/superio/smsc/lpc47n217/early_serial.c
@@ -105,7 +105,7 @@ static void lpc47n217_pnp_set_enable(pnp_devfn_t dev, int enable)
  * @param dev High 8 bits = Super I/O port, low 8 bits = logical device number.
  * @param iobase Processor I/O port address to assign to this serial device.
  */
-static void lpc47n217_enable_serial(pnp_devfn_t dev, u16 iobase)
+void lpc47n217_enable_serial(pnp_devfn_t dev, u16 iobase)
 {
 	/*
 	 * NOTE: Cannot use pnp_set_XXX() here because they assume chip
diff --git a/src/superio/smsc/lpc47n217/lpc47n217.h b/src/superio/smsc/lpc47n217/lpc47n217.h
index 20689d1..1cae5a0 100644
--- a/src/superio/smsc/lpc47n217/lpc47n217.h
+++ b/src/superio/smsc/lpc47n217/lpc47n217.h
@@ -17,6 +17,9 @@
 #ifndef SUPERIO_SMSC_LPC47N217_LPC47N217_H
 #define SUPERIO_SMSC_LPC47N217_LPC47N217_H
 
+#include <arch/io.h>
+#include <stdint.h>
+
 /*
  * These are arbitrary, but must match declarations in the mainboard
  * devicetree.cb file. Values chosen to match SMSC LPC47B37x.
@@ -27,4 +30,6 @@
 
 #define LPC47N217_MAX_CONFIG_REGISTER	0x39
 
+void lpc47n217_enable_serial(pnp_devfn_t dev, u16 iobase);
+
 #endif