From c1d524b8c60981d43153a8e0112d8224ba43a889 Mon Sep 17 00:00:00 2001 From: Kyösti Mälkki Date: Fri, 8 Jan 2021 16:41:02 +0200 Subject: sb/intel/common: Change some SMI logging MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Change-Id: Ief0c3d36e6de6e18b7f2613f043ac4d31a193f9d Signed-off-by: Kyösti Mälkki Reviewed-on: https://review.coreboot.org/c/coreboot/+/49249 Reviewed-by: Angel Pons Tested-by: build bot (Jenkins) --- src/mainboard/dell/optiplex_9010/smihandler.c | 10 ---------- src/mainboard/google/butterfly/smihandler.c | 7 ------- src/mainboard/google/parrot/smihandler.c | 2 -- 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 #include -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 #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 */ -- cgit v1.2.3