diff options
| author | Luc Verhaegen <libv@skynet.be> | 2007-03-02 22:16:38 +0000 |
|---|---|---|
| committer | Uwe Hermann <uwe@hermann-uwe.de> | 2007-03-02 22:16:38 +0000 |
| commit | 47ce57acd769a0ec6858402fdf9900d69ab51592 (patch) | |
| tree | 9231359ef7864784b7992f81aa19a926f3dacddc /util | |
| parent | a81bb03b797ed5ed35e874692526482cdcb7c12f (diff) | |
| download | coreboot-47ce57acd769a0ec6858402fdf9900d69ab51592.tar.xz | |
flashrom: Fix wrong VT8235 flash enable failed warning.
* Fix harmless but worrying warning where the return value of
pci_write_byte is misinterpreted.
* Hash together VT8231 and VT8235 code into VT823x. VT8231 is the better
implementation, but lacked the write protect disable code that's
apparently needed for VT8235.
Signed-off-by: Luc Verhaegen <libv@skynet.be>
Signed-off-by: Uwe Hermann <uwe@hermann-uwe.de>
Acked-by: Stefan Reinauer <stepan@coresystems.de>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@2568 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
Diffstat (limited to 'util')
| -rw-r--r-- | util/flashrom/flash_enable.c | 77 |
1 files changed, 37 insertions, 40 deletions
diff --git a/util/flashrom/flash_enable.c b/util/flashrom/flash_enable.c index c557b22734..097cfbe272 100644 --- a/util/flashrom/flash_enable.c +++ b/util/flashrom/flash_enable.c @@ -158,52 +158,49 @@ static int enable_flash_ich_dc(struct pci_dev *dev, char *name) return enable_flash_ich(dev, name, 0xdc); } -static int enable_flash_vt8235(struct pci_dev *dev, char *name) -{ - uint8_t old, new, val; - unsigned int base; - int ok; - - old = pci_read_byte(dev, 0x40); - - new = old | 0x10; - - if (new == old) - return 0; - - ok = pci_write_byte(dev, 0x40, new); - if (ok != 0) { - printf("tried to set 0x%x to 0x%x on %s failed (WARNING ONLY)\n", - old, new, name); - } - - /* enable GPIO15 which is connected to write protect. */ - base = ((pci_read_byte(dev, 0x88) & 0x80) | pci_read_byte(dev, 0x89) << 8); - val = inb(base + 0x4d); - val |= 0x80; - outb(val, base + 0x4d); - - if (ok != 0) { - return -1; - } else { - return 0; - } -} - -static int enable_flash_vt8231(struct pci_dev *dev, char *name) +static int enable_flash_vt823x(struct pci_dev *dev, char *name) { uint8_t val; + int ret = 0; + /* ROM Write enable */ val = pci_read_byte(dev, 0x40); val |= 0x10; pci_write_byte(dev, 0x40, val); if (pci_read_byte(dev, 0x40) != val) { - printf("tried to set 0x%x to 0x%x on %s failed (WARNING ONLY)\n", - 0x40, val, name); - return -1; + printf("Warning: Failed to enable ROM Write on %s\n", name); + ret = -1; } - return 0; + + if (dev->device_id == 0x3177) { /* VT8235 */ + if (!iopl(3)) { /* enable full IO access */ + unsigned int base; + + /* GPIO12-15 -> output */ + val = pci_read_byte(dev, 0xE4); + val |= 0x38; + pci_write_byte(dev, 0xE4, val); + + /* Get Power Management IO address. */ + base = pci_read_word(dev, 0x88) & 0xFF80; + + /* enable GPIO15 which is connected to write protect. */ + val = inb(base + 0x4d); + val |= 0xFF; + outb(val, base + 0x4d); + + val = inb(base + 0x4E); + val |= 0x0F; + outb(val, base + 0x4E); + } else { + printf("Warning; Failed to disable Write Protect" + " on %s (iopl failed)\n", name); + return -1; + } + } + + return ret; } static int enable_flash_cs5530(struct pci_dev *dev, char *name) @@ -445,9 +442,9 @@ static FLASH_ENABLE enables[] = { {0x8086, 0x2810, "ICH8/ICH8R", enable_flash_ich_dc}, {0x8086, 0x2812, "ICH8DH", enable_flash_ich_dc}, {0x8086, 0x2814, "ICH8DO", enable_flash_ich_dc}, - {0x1106, 0x8231, "VT8231", enable_flash_vt8231}, - {0x1106, 0x3177, "VT8235", enable_flash_vt8235}, - {0x1106, 0x3227, "VT8237", enable_flash_vt8231}, + {0x1106, 0x8231, "VT8231", enable_flash_vt823x}, + {0x1106, 0x3177, "VT8235", enable_flash_vt823x}, + {0x1106, 0x3227, "VT8237", enable_flash_vt823x}, {0x1106, 0x0686, "VT82C686", enable_flash_amd8111}, {0x1078, 0x0100, "CS5530", enable_flash_cs5530}, {0x100b, 0x0510, "SC1100", enable_flash_sc1100}, |