summaryrefslogtreecommitdiff
path: root/src/southbridge/intel
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2022-11-25 09:07:56 +0200
committerKyösti Mälkki <kyosti.malkki@gmail.com>2022-11-28 10:26:27 +0000
commit99166482fe4baa00780ca3b45fce6ae75c4cdbfb (patch)
tree91dbc6ee21db8427ed8c82aeb079af2a74c3be6b /src/southbridge/intel
parentac23f9da757e8e47a6bdcfd619c54e9b4c2b906c (diff)
sb,soc/intel: Drop spurious SMI entry message
The message only makes sense if ACPI PM base address is allowed to be dynamic. If requested, it can be logged in common code. Change-Id: Iad7a60098c0391cc23384035af49e373dad90233 Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/70047 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Elyes Haouas <ehaouas@noos.fr> Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Diffstat (limited to 'src/southbridge/intel')
-rw-r--r--src/southbridge/intel/common/smi.c5
-rw-r--r--src/southbridge/intel/lynxpoint/smi.c5
2 files changed, 1 insertions, 9 deletions
diff --git a/src/southbridge/intel/common/smi.c b/src/southbridge/intel/common/smi.c
index 20926e579e..a6e000b233 100644
--- a/src/southbridge/intel/common/smi.c
+++ b/src/southbridge/intel/common/smi.c
@@ -23,15 +23,12 @@ static int smi_enabled(void)
if (CONFIG(ELOG))
pch_log_state();
- printk(BIOS_DEBUG, "Initializing southbridge SMI...");
- printk(BIOS_SPEW, " ... pmbase = 0x%04x\n", lpc_get_pmbase());
-
smi_en = read_pmbase32(SMI_EN);
if (smi_en & APMC_EN) {
printk(BIOS_INFO, "SMI# handler already enabled?\n");
return 1;
}
- printk(BIOS_DEBUG, "\n");
+
return 0;
}
diff --git a/src/southbridge/intel/lynxpoint/smi.c b/src/southbridge/intel/lynxpoint/smi.c
index 6edf5c1836..9ae78793a2 100644
--- a/src/southbridge/intel/lynxpoint/smi.c
+++ b/src/southbridge/intel/lynxpoint/smi.c
@@ -17,17 +17,12 @@ void smm_southbridge_clear_state(void)
if (CONFIG(ELOG))
pch_log_state();
- printk(BIOS_DEBUG, "Initializing Southbridge SMI...");
- printk(BIOS_SPEW, " ... pmbase = 0x%04x\n", get_pmbase());
-
smi_en = inl(get_pmbase() + SMI_EN);
if (smi_en & APMC_EN) {
printk(BIOS_INFO, "SMI# handler already enabled?\n");
return;
}
- printk(BIOS_DEBUG, "\n");
-
/* Dump and clear status registers */
clear_smi_status();
clear_pm1_status();