diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2018-12-25 15:26:58 +0100 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2018-12-29 07:18:55 +0000 |
| commit | f0a017fe8288e16adbecf8998e99c2cadc3e12de (patch) | |
| tree | dd058a40a224d25ef0836aa28db2ad1bb188cca2 /src/southbridge/intel/common/smi.c | |
| parent | 9895177517a796093ef11616bff8a35b0437610f (diff) | |
| download | coreboot-f0a017fe8288e16adbecf8998e99c2cadc3e12de.tar.xz | |
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>
Diffstat (limited to 'src/southbridge/intel/common/smi.c')
| -rw-r--r-- | src/southbridge/intel/common/smi.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/southbridge/intel/common/smi.c b/src/southbridge/intel/common/smi.c index 3ce4f4057a..af9dd5d87d 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 @@ void southbridge_smm_init(void) 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..."); |