From da3087f67d516350249779745927861c4da2173d Mon Sep 17 00:00:00 2001 From: Marc Jones Date: Mon, 5 Nov 2012 17:25:52 -0700 Subject: Mainboard SMI S state handler was using the wrong defines The PCH register bit definition for sleep type is a little confusing. For example, 7 is S5. To make this simpler for the mainbaord developer, the mainboard smi sleep hander is called as mainboard_sleep(slp_typ-2). A couple mainboard SMI handlers were using the PCH define for slp_ty, so S3 code would be run for S5 and S5 code would never be run. Change-Id: Iaecf96bfd48cf00153600cd119760364fbdfc29e Signed-off-by: Marc Jones Reviewed-on: http://review.coreboot.org/2514 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Ronald G. Minnich --- src/mainboard/intel/emeraldlake2/smihandler.c | 6 +++--- src/mainboard/samsung/stumpy/smihandler.c | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) (limited to 'src/mainboard') diff --git a/src/mainboard/intel/emeraldlake2/smihandler.c b/src/mainboard/intel/emeraldlake2/smihandler.c index 03c505b816..acc1fde93d 100644 --- a/src/mainboard/intel/emeraldlake2/smihandler.c +++ b/src/mainboard/intel/emeraldlake2/smihandler.c @@ -59,11 +59,11 @@ void mainboard_smi_sleep(u8 slp_typ) u8 reg8; switch (slp_typ) { - case SLP_TYP_S3: - case SLP_TYP_S4: + case 3: + case 4: break; - case SLP_TYP_S5: + case 5: /* Turn off LED */ reg8 = inb(SIO_GPIO_BASE_SET4); reg8 |= (1 << 5); diff --git a/src/mainboard/samsung/stumpy/smihandler.c b/src/mainboard/samsung/stumpy/smihandler.c index 660bb31cd4..5eda0a1072 100644 --- a/src/mainboard/samsung/stumpy/smihandler.c +++ b/src/mainboard/samsung/stumpy/smihandler.c @@ -62,8 +62,8 @@ void mainboard_smi_sleep(u8 slp_typ) u8 reg8; switch (slp_typ) { - case SLP_TYP_S3: - case SLP_TYP_S4: + case 3: + case 4: /* Blink LED */ it8772f_enter_conf(); it8772f_sio_write(IT8772F_CONFIG_REG_LDN, IT8772F_GPIO); @@ -79,7 +79,7 @@ void mainboard_smi_sleep(u8 slp_typ) it8772f_exit_conf(); break; - case SLP_TYP_S5: + case 5: /* Turn off LED */ reg8 = inb(SIO_GPIO_BASE_SET4); reg8 |= (1 << 5); -- cgit v1.2.3