[coreboot-gerrit] New patch to review for coreboot: 4f66a17 superio/fintek/f81216h: Add the correct unlock key values

Dave Frodin (dave.frodin@se-eng.com) gerrit at coreboot.org
Fri Feb 6 15:45:18 CET 2015


Dave Frodin (dave.frodin at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8375

-gerrit

commit 4f66a17124a4ba7140d128e4fcbda369595c45da
Author: Dave Frodin <dave.frodin at se-eng.com>
Date:   Fri Feb 6 07:25:40 2015 -0700

    superio/fintek/f81216h: Add the correct unlock key values
    
    Change-Id: I8f20e3d0ce83dd87b6c233ee98c3e77a6b7c1b03
    Signed-off-by: Dave Frodin <dave.frodin at se-eng.com>
---
 src/superio/fintek/f81216h/superio.c | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/superio/fintek/f81216h/superio.c b/src/superio/fintek/f81216h/superio.c
index 89204c4..84116a0 100644
--- a/src/superio/fintek/f81216h/superio.c
+++ b/src/superio/fintek/f81216h/superio.c
@@ -46,15 +46,21 @@ static void pnp_enter_ext_func_mode(struct device *dev)
 	 */
 	switch(conf->conf_key_mode) {
 	case MODE_6767:
+		key = 0x67;
+		break;
 	case MODE_7777:
+		key = 0x77;
+		break;
 	case MODE_8787:
+		key = 0x87;
+		break;
 	case MODE_A0A0:
-		key = conf->conf_key_mode;
+		key = 0xA0;
 		break;
 	default:
 		printk(BIOS_WARNING, "Warning: Undefined F81216 unlock key assignment!\n");
 		printk(BIOS_WARNING, "Setting conf_key_mode to default\n");
-		key = MODE_7777; /* try the hw default */
+		key = 0x77; /* try the hw default */
 		break;
 	}
 



More information about the coreboot-gerrit mailing list