diff options
| author | Nico Huber <nico.h@gmx.de> | 2019-08-05 21:12:33 +0200 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2019-08-09 09:37:32 +0000 |
| commit | 3e786b55463fc656fd3c23823b42b72591eb3d21 (patch) | |
| tree | 76946c7658392d6ee5f046ef7edce521bc849ee0 /src/soc/intel/common/block/smm | |
| parent | 2fe596e6778ca1e9bc5a1f0f585604d000297426 (diff) | |
| download | coreboot-3e786b55463fc656fd3c23823b42b72591eb3d21.tar.xz | |
soc/intel: Drop pmc_soc_restore_power_failure()
Get rid of this function and its dangerous, weak implementation.
Instead, call pmc_set_power_failure_state() directly from the SMI
handler.
Change-Id: I0718afc5db66447c93289643f9097a4257b10934
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/34727
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Diffstat (limited to 'src/soc/intel/common/block/smm')
| -rw-r--r-- | src/soc/intel/common/block/smm/smihandler.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/soc/intel/common/block/smm/smihandler.c b/src/soc/intel/common/block/smm/smihandler.c index 7aa69c5c65..abd699ac2b 100644 --- a/src/soc/intel/common/block/smm/smihandler.c +++ b/src/soc/intel/common/block/smm/smihandler.c @@ -29,6 +29,7 @@ #include <intelblocks/uart.h> #include <smmstore.h> #include <soc/nvs.h> +#include <soc/pci_devs.h> #include <soc/pm.h> #include <soc/gpio.h> #include <soc/iomap.h> @@ -221,7 +222,7 @@ void smihandler_southbridge_sleep( /* Disable all GPE */ pmc_disable_all_gpe(); /* Set which state system will be after power reapplied */ - pmc_soc_restore_power_failure(); + pmc_set_power_failure_state(false); /* also iterates over all bridges on bus 0 */ busmaster_disable_on_bus(0); |