diff options
author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2021-01-08 16:41:02 +0200 |
---|---|---|
committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2021-01-25 10:35:46 +0000 |
commit | c1d524b8c60981d43153a8e0112d8224ba43a889 (patch) | |
tree | 15fa1bf3fd868c6e685a0a142a3e29ee4adc4b73 | |
parent | 9a1620f4ed5a17bde351142d3ca5489f3c95c61f (diff) |
sb/intel/common: Change some SMI logging
Change-Id: Ief0c3d36e6de6e18b7f2613f043ac4d31a193f9d
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49249
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r-- | src/mainboard/dell/optiplex_9010/smihandler.c | 10 | ||||
-rw-r--r-- | src/mainboard/google/butterfly/smihandler.c | 7 | ||||
-rw-r--r-- | src/mainboard/google/parrot/smihandler.c | 2 | ||||
-rw-r--r-- | src/southbridge/intel/common/smihandler.c | 3 |
4 files changed, 2 insertions, 20 deletions
diff --git a/src/mainboard/dell/optiplex_9010/smihandler.c b/src/mainboard/dell/optiplex_9010/smihandler.c index f3ba57cdd2..1c7d3288ed 100644 --- a/src/mainboard/dell/optiplex_9010/smihandler.c +++ b/src/mainboard/dell/optiplex_9010/smihandler.c @@ -6,11 +6,6 @@ #include <cpu/x86/smm.h> #include <superio/smsc/sch5545/sch5545.h> -void mainboard_smi_gpi(u32 gpi_sts) -{ - printk(BIOS_SPEW, "%s: gpi_sts: %08x\n", __func__, gpi_sts); -} - int mainboard_smi_apmc(u8 data) { u8 val; @@ -38,8 +33,3 @@ int mainboard_smi_apmc(u8 data) } return 0; } - -void mainboard_smi_sleep(u8 slp_typ) -{ - printk(BIOS_SPEW, "%s: SMI sleep: %02x\n", __func__, slp_typ); -} diff --git a/src/mainboard/google/butterfly/smihandler.c b/src/mainboard/google/butterfly/smihandler.c index f826bcf194..6b9ec5a54e 100644 --- a/src/mainboard/google/butterfly/smihandler.c +++ b/src/mainboard/google/butterfly/smihandler.c @@ -13,15 +13,8 @@ #include <ec/quanta/ene_kb3940q/ec.h> #include "ec.h" -void mainboard_smi_gpi(u32 gpi_sts) -{ - printk(BIOS_DEBUG, "warn: unknown mainboard_smi_gpi: %x\n", gpi_sts); -} - void mainboard_smi_sleep(u8 slp_typ) { - printk(BIOS_DEBUG, "mainboard_smi_sleep: %x\n", slp_typ); - /* Tell the EC to Enable USB power for S3 if requested */ if (gnvs->s3u0 != 0 || gnvs->s3u1 != 0) ec_mem_write(EC_EC_PSW, ec_mem_read(EC_EC_PSW) | EC_PSW_USB); diff --git a/src/mainboard/google/parrot/smihandler.c b/src/mainboard/google/parrot/smihandler.c index 95dce5057a..451c3a93df 100644 --- a/src/mainboard/google/parrot/smihandler.c +++ b/src/mainboard/google/parrot/smihandler.c @@ -36,7 +36,6 @@ static u8 mainboard_smi_ec(void) void mainboard_smi_gpi(u32 gpi_sts) { - printk(BIOS_DEBUG, "%s: %x\n", __func__, gpi_sts); if (gpi_sts & (1 << EC_SMI_GPI)) { /* Process all pending events from EC */ do {} while (mainboard_smi_ec() != EC_NO_EVENT); @@ -50,7 +49,6 @@ void mainboard_smi_gpi(u32 gpi_sts) void mainboard_smi_sleep(u8 slp_typ) { - printk(BIOS_DEBUG, "%s: %x\n", __func__, slp_typ); /* Disable SCI and SMI events */ /* Clear pending events that may trigger immediate wake */ diff --git a/src/southbridge/intel/common/smihandler.c b/src/southbridge/intel/common/smihandler.c index 7256497c4f..07cd662b3f 100644 --- a/src/southbridge/intel/common/smihandler.c +++ b/src/southbridge/intel/common/smihandler.c @@ -113,9 +113,10 @@ static void southbridge_smi_sleep(void) /* Figure out SLP_TYP */ reg32 = read_pmbase32(PM1_CNT); - printk(BIOS_SPEW, "SMI#: SLP = 0x%08x\n", reg32); slp_typ = acpi_sleep_from_pm1(reg32); + printk(BIOS_SPEW, "SMI#: SLP = 0x%08x, TYPE = 0x%02x\n", reg32, slp_typ); + southbridge_smm_xhci_sleep(slp_typ); /* Do any mainboard sleep handling */ |