summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAaron Durbin <adurbin@chromium.org>2013-03-27 21:13:02 -0500
committerRonald G. Minnich <rminnich@gmail.com>2013-04-01 23:25:20 +0200
commitd6d6db3717d09f2b6a4590eec6016ca7d417c2f9 (patch)
tree851710c38debc2e4ea2e93fe6d16b145dfe7cff1
parentbab0a0b577ba8cc28aa1e2b2e9ec65fb98edced1 (diff)
downloadcoreboot-d6d6db3717d09f2b6a4590eec6016ca7d417c2f9.tar.xz
lynxpoint: fix enable_pm1() function
The new enable_pm1() function was doing 2 things wrong: 1. It was doing a RMW of the pm1 register. This means we were keeping around the enables from the OS during S3 resume. This is bad in the face of the RTC alarm waking us up because it would cause an infinite stream of SMIs. 2. The register size of PM1_EN is 16-bits. However, the previous implementation was accessing it as a 32-bit register. The PM1 enables should only be set to what we expect to handle in the firmware before the OS changes to ACPI mode. Change-Id: Ib1d3caf6c84a1670d9456ed159420c6cb64f555e Signed-off-by: Aaron Durbin <adurbin@chromium.org> Reviewed-on: http://review.coreboot.org/2978 Tested-by: build bot (Jenkins) Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
-rw-r--r--src/southbridge/intel/lynxpoint/pch.h2
-rw-r--r--src/southbridge/intel/lynxpoint/pmutil.c10
2 files changed, 6 insertions, 6 deletions
diff --git a/src/southbridge/intel/lynxpoint/pch.h b/src/southbridge/intel/lynxpoint/pch.h
index 3535b987ce..659ddb65b3 100644
--- a/src/southbridge/intel/lynxpoint/pch.h
+++ b/src/southbridge/intel/lynxpoint/pch.h
@@ -139,7 +139,7 @@ void enable_pm1_control(u32 mask);
void disable_pm1_control(u32 mask);
/* PM1 */
u16 clear_pm1_status(void);
-void enable_pm1(u32 mask);
+void enable_pm1(u16 events);
u32 clear_smi_status(void);
/* SMI */
void enable_smi(u32 mask);
diff --git a/src/southbridge/intel/lynxpoint/pmutil.c b/src/southbridge/intel/lynxpoint/pmutil.c
index 3a0b70b0b3..386705f00a 100644
--- a/src/southbridge/intel/lynxpoint/pmutil.c
+++ b/src/southbridge/intel/lynxpoint/pmutil.c
@@ -29,6 +29,7 @@
#include <device/pci.h>
#include <device/pci_def.h>
#include <console/console.h>
+#include <pc80/mc146818rtc.h>
#include "pch.h"
#if CONFIG_INTEL_LYNXPOINT_LP
@@ -104,6 +105,7 @@ static u16 reset_pm1_status(void)
{
u16 pm1_sts = inw(get_pmbase() + PM1_STS);
outw(pm1_sts, get_pmbase() + PM1_STS);
+
return pm1_sts;
}
@@ -137,12 +139,10 @@ u16 clear_pm1_status(void)
return print_pm1_status(reset_pm1_status());
}
-/* Enable PM1 event */
-void enable_pm1(u32 mask)
+/* Set the PM1 register to events */
+void enable_pm1(u16 events)
{
- u32 pm1_en = inl(get_pmbase() + PM1_EN);
- pm1_en |= mask;
- outl(pm1_en, get_pmbase() + PM1_EN);
+ outw(events, get_pmbase() + PM1_EN);
}