[coreboot-gerrit] New patch to review for coreboot: 9dc44d5 util/superiotool: Register fix for Fintek F81865F/F-I

[W. Duijvenvoorde (w.a.n.duijvenvoorde@gmail.com)]

W. Duijvenvoorde (w.a.n.duijvenvoorde at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5404

-gerrit

commit 9dc44d55b0917959c24bcca9e4a503734e1a5bc1
Author: Wilbert Duijvenvoorde <w.a.n.duijvenvoorde at gmail.com>
Date:   Mon Mar 24 09:50:25 2014 +0100

    util/superiotool: Register fix for Fintek F81865F/F-I
    
    Datasheet: http://www.fintek.com.tw/files/productfiles/F81865_V028P.pdf
    
    There is a multi-function select register listed as 0x2a-1 and 0x2a-2.
    These are the original names in the datasheet, but superiotool will
    display register 0x29 and 0x28 and their values.
    This patch renames them both to 0x2a and shows both of the default values
    for them. They are both 0x00, so one of them could be dropped though.
    
    Change-Id: Iad91f9e4755d2d1a123e56ab0fa9257be7ea9978
    Signed-off-by: Wilbert Duijvenvoorde <w.a.n.duijvenvoorde at gmail.com>
---
 util/superiotool/fintek.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/superiotool/fintek.c b/util/superiotool/fintek.c
index aba0353..b4fac9f 100644
--- a/util/superiotool/fintek.c
+++ b/util/superiotool/fintek.c
@@ -322,7 +322,7 @@ static const struct superio_registers reg_table[] = {
 		{EOT}}},
 	{0x0407, "F81865F/F-I", {
 		{NOLDN, NULL,
-			{0x02,0x07,0x20,0x21,0x23,0x24,0x25,0x26,0x27,0x28,0x29,0x2a-1,0x2a-2,0x2b,0x2c,0x2d,EOT},
+			{0x02,0x07,0x20,0x21,0x23,0x24,0x25,0x26,0x27,0x28,0x29,0x2a,0x2a,0x2b,0x2c,0x2d,EOT},
 			{NANA,0x00,0x07,0x04,0x19,0x34,NANA,NANA,NANA,0x00,0x00,0x00,0x00,0x1f,0x00,0x08,EOT}},
 		{0x00, "FDC",
 			{0x30,0x60,0x61,0x70,0x74,0xf0,0xf2,0xf4,EOT},