commit f0a017fe8288e16adbecf8998e99c2cadc3e12de
author Arthur Heymans <arthur@aheymans.xyz> Tue Dec 25 15:26:58 2018 +0100
committer Patrick Georgi <pgeorgi@google.com> Sat Dec 29 07:18:55 2018 +0000
tree dd058a40a224d25ef0836aa28db2ad1bb188cca2
parent 9895177517a796093ef11616bff8a35b0437610f

sb/intel/bd828x6x: Make CONFIG_ELOG=y compile

The function pch_log_state() was overlooked when making the smi
relocation code common.

Change-Id: I878772f1a93105b828e50f37e105d04988ba0bdf
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/30426
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Rudolph <siro@das-labor.org>

src/southbridge/intel/common/smi.c

diff --git a/src/southbridge/intel/common/smi.c b/src/southbridge/intel/common/smi.c
index 3ce4f40..af9dd5d 100644
--- a/src/southbridge/intel/common/smi.c
+++ b/src/southbridge/intel/common/smi.c
@@ -23,6 +23,7 @@
 #include <cpu/x86/smm.h>
 #include <cpu/intel/smm/gen1/smi.h>
 #include <southbridge/intel/common/pmbase.h>
+#include <southbridge/intel/common/pmutil.h>
 
 #include "pmutil.h"
 
@@ -39,10 +40,9 @@
 	u16 pm1_en;
 	u32 gpe0_en;
 
-#if IS_ENABLED(CONFIG_ELOG)
+	if (IS_ENABLED(CONFIG_ELOG))
 	/* Log events from chipset before clearing */
-	pch_log_state();
-#endif
+		pch_log_state();
 
 	printk(BIOS_DEBUG, "Initializing southbridge SMI...");