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>
This commit is contained in:
Arthur Heymans 2018-12-25 15:26:58 +01:00 committed by Patrick Georgi
parent 9895177517
commit f0a017fe82
4 changed files with 5 additions and 6 deletions

View File

@ -21,6 +21,7 @@
#include <stdint.h>
#include <string.h>
#include <elog.h>
#include <southbridge/intel/common/pmutil.h>
#include "pch.h"
void pch_log_state(void)

View File

@ -66,9 +66,6 @@ int pch_silicon_revision(void);
int pch_silicon_type(void);
int pch_silicon_supported(int type, int rev);
void pch_iobp_update(u32 address, u32 andvalue, u32 orvalue);
#if IS_ENABLED(CONFIG_ELOG)
void pch_log_state(void);
#endif
#else /* __PRE_RAM__ */
void enable_smbus(void);
void enable_usb_bar(void);

View File

@ -149,5 +149,6 @@ void southbridge_update_gnvs(u8 apm_cnt, int *smm_done);
void southbridge_finalize_all(void);
void southbridge_smi_monitor(void);
em64t101_smm_state_save_area_t *smi_apmc_find_state_save(u8 cmd);
void pch_log_state(void);
#endif /*INTEL_COMMON_PMUTIL_H */

View File

@ -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...");