New patch to review for coreboot: 30f0cba hp/pavilion_m6_1035dx: Do not re-init EC and SMI on S3 resume

[Alexandru Gagniuc]

Alexandru Gagniuc ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5555

-gerrit

commit 30f0cbafcf94e6d5d4b319df3ed81cb304eaea0b
Author: Alexandru Gagniuc 
Date:   Sat Apr 19 15:10:14 2014 -0500

hp/pavilion_m6_1035dx: Do not re-init EC and SMI on S3 resume

It's not needed, and puts the EC back into APM mode. The EC does not
    shut down during S3 sleep, so we don't need to re-initialize it.
    SMI triggers don't need to be set up again, as that resets their state
    and re-enables shutdown on lid close.

Change-Id: I2c06df140f63427dac32ae095d29e68f64135358
    Signed-off-by: Alexandru Gagniuc 
---
 src/mainboard/hp/pavilion_m6_1035dx/mainboard.c | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/src/mainboard/hp/pavilion_m6_1035dx/mainboard.c b/src/mainboard/hp/pavilion_m6_1035dx/mainboard.c
index 2e8436b..c86cfbf 100644
--- a/src/mainboard/hp/pavilion_m6_1035dx/mainboard.c
+++ b/src/mainboard/hp/pavilion_m6_1035dx/mainboard.c
@@ -32,18 +32,18 @@

#include <southbridge/amd/agesa/hudson/smi.h>

-/*************************************************
- * enable the dedicated function in parmer board.
- *************************************************/
-static void mainboard_enable(device_t dev)
-{
-	printk(BIOS_INFO, "Mainboard " CONFIG_MAINBOARD_PART_NUMBER " Enable.\n");
-
-	pavilion_m6_1035dx_ec_init();

+static void pavilion_cold_boot_init(void)
+{
    hudson_configure_gevent_smi(EC_SMI_GEVENT, SMI_MODE_SMI, SMI_LVL_HIGH);
    hudson_configure_gevent_smi(EC_LID_GEVENT, SMI_MODE_SMI, SMI_LVL_LOW);
    hudson_enable_smi_generation();
+	pavilion_m6_1035dx_ec_init();
+}
+
+static void mainboard_enable(device_t dev)
+{
+	printk(BIOS_INFO, "Mainboard " CONFIG_MAINBOARD_PART_NUMBER " Enable.\n");

/*
     * The mainboard is the first place that we get control in ramstage. Check
@@ -53,7 +53,10 @@ static void mainboard_enable(device_t dev)
    acpi_slp_type = acpi_get_sleep_type();
    if (acpi_slp_type == 3)
    	agesawrapper_fchs3earlyrestore();
+	else
 #endif
+		pavilion_cold_boot_init();
+
 }

struct chip_operations mainboard_ops = {