First attempt to clean up SPI probing and create a common
[coreboot.git] / util / flashrom / board_enable.c
index 78749634ae222e9fa81178e4d72c629cf496f5c8..a3eae3353d3109f4ca2ba4c84b3614810794994d 100644 (file)
@@ -3,7 +3,7 @@
  *
  * Copyright (C) 2005-2007 coresystems GmbH <stepan@coresystems.de>
  * Copyright (C) 2006 Uwe Hermann <uwe@hermann-uwe.de>
- * Copyright (C) 2007 Luc Verhaegen <libv@skynet.be>
+ * Copyright (C) 2007-2008 Luc Verhaegen <libv@skynet.be>
  * Copyright (C) 2007 Carl-Daniel Hailfinger
  *
  * This program is free software; you can redistribute it and/or modify
@@ -28,6 +28,7 @@
 #include <pci/pci.h>
 #include <stdint.h>
 #include <string.h>
+#include <fcntl.h>
 #include "flash.h"
 
 /*
 /* Enter extended functions */
 static void w836xx_ext_enter(uint16_t port)
 {
-       outb(0x87, port);
-       outb(0x87, port);
+       OUTB(0x87, port);
+       OUTB(0x87, port);
 }
 
 /* Leave extended functions */
 static void w836xx_ext_leave(uint16_t port)
 {
-       outb(0xAA, port);
+       OUTB(0xAA, port);
 }
 
 /* General functions for reading/writing Winbond Super I/Os. */
 static unsigned char wbsio_read(uint16_t index, uint8_t reg)
 {
-       outb(reg, index);
-       return inb(index+1);
+       OUTB(reg, index);
+       return INB(index + 1);
 }
 
 static void wbsio_write(uint16_t index, uint8_t reg, uint8_t data)
 {
-       outb(reg, index);
-       outb(data, index+1);
+       OUTB(reg, index);
+       OUTB(data, index + 1);
 }
 
 static void wbsio_mask(uint16_t index, uint8_t reg, uint8_t data, uint8_t mask)
 {
        uint8_t tmp;
 
-       outb(reg, index);
-       tmp = inb(index+1) & ~mask;
-       outb(tmp | (data & mask), index+1);
+       OUTB(reg, index);
+       tmp = INB(index + 1) & ~mask;
+       OUTB(tmp | (data & mask), index + 1);
 }
 
 /**
@@ -79,8 +80,8 @@ static int w83627hf_gpio24_raise(uint16_t index, const char *name)
 {
        w836xx_ext_enter(index);
 
-       /* Is this the w83627hf? */
-       if (wbsio_read(index, 0x20) != 0x52) {  /* Super I/O device ID register */
+       /* Is this the W83627HF? */
+       if (wbsio_read(index, 0x20) != 0x52) {  /* Super I/O device ID reg. */
                fprintf(stderr, "\nERROR: %s: W83627HF: Wrong ID: 0x%02X.\n",
                        name, wbsio_read(index, 0x20));
                w836xx_ext_leave(index);
@@ -90,14 +91,12 @@ static int w83627hf_gpio24_raise(uint16_t index, const char *name)
        /* PIN89S: WDTO/GP24 multiplex -> GPIO24 */
        wbsio_mask(index, 0x2B, 0x10, 0x10);
 
-       wbsio_write(index, 0x07, 0x08); /* Select logical device 8: GPIO port 2 */
+       /* Select logical device 8: GPIO port 2 */
+       wbsio_write(index, 0x07, 0x08);
 
        wbsio_mask(index, 0x30, 0x01, 0x01);    /* Activate logical device. */
-
        wbsio_mask(index, 0xF0, 0x00, 0x10);    /* GPIO24 -> output */
-
        wbsio_mask(index, 0xF2, 0x00, 0x10);    /* Clear GPIO24 inversion */
-
        wbsio_mask(index, 0xF1, 0x10, 0x10);    /* Raise GPIO24 */
 
        w836xx_ext_leave(index);
@@ -107,6 +106,7 @@ static int w83627hf_gpio24_raise(uint16_t index, const char *name)
 
 static int w83627hf_gpio24_raise_2e(const char *name)
 {
+       /* TODO: Typo? Shouldn't this be 0x2e? */
        return w83627hf_gpio24_raise(0x2d, name);
 }
 
@@ -119,8 +119,9 @@ static int w83627hf_gpio24_raise_2e(const char *name)
 static int w83627thf_gpio4_4_raise(uint16_t index, const char *name)
 {
        w836xx_ext_enter(index);
-       /* Is this the w83627thf? */
-       if (wbsio_read(index, 0x20) != 0x82) {  /* Super I/O device ID register */
+
+       /* Is this the W83627THF? */
+       if (wbsio_read(index, 0x20) != 0x82) {  /* Super I/O device ID reg. */
                fprintf(stderr, "\nERROR: %s: W83627THF: Wrong ID: 0x%02X.\n",
                        name, wbsio_read(index, 0x20));
                w836xx_ext_leave(index);
@@ -129,15 +130,11 @@ static int w83627thf_gpio4_4_raise(uint16_t index, const char *name)
 
        /* PINxxxxS: GPIO4/bit 4 multiplex -> GPIOXXX */
 
-       wbsio_write(index, 0x07, 0x09); /* Select logical device 9: GPIO port 4 */
-
-       wbsio_mask(index, 0x30, 0x02, 0x02);    /* Activate logical device. */
-
-       wbsio_mask(index, 0xF4, 0x00, 0x10);    /* GPIO4 bit 4 -> output */
-
-       wbsio_mask(index, 0xF6, 0x00, 0x10);    /* Clear GPIO4 bit 4 inversion */
-
-       wbsio_mask(index, 0xF5, 0x10, 0x10);    /* Raise GPIO4 bit 4 */
+       wbsio_write(index, 0x07, 0x09);      /* Select LDN 9: GPIO port 4 */
+       wbsio_mask(index, 0x30, 0x02, 0x02); /* Activate logical device. */
+       wbsio_mask(index, 0xF4, 0x00, 0x10); /* GPIO4 bit 4 -> output */
+       wbsio_mask(index, 0xF6, 0x00, 0x10); /* Clear GPIO4 bit 4 inversion */
+       wbsio_mask(index, 0xF5, 0x10, 0x10); /* Raise GPIO4 bit 4 */
 
        w836xx_ext_leave(index);
 
@@ -146,17 +143,18 @@ static int w83627thf_gpio4_4_raise(uint16_t index, const char *name)
 
 static int w83627thf_gpio4_4_raise_4e(const char *name)
 {
-       return w83627thf_gpio4_4_raise(0x4E, name);
+       return w83627thf_gpio4_4_raise(0x4e, name);
 }
+
 /**
  * Suited for VIAs EPIA M and MII, and maybe other CLE266 based EPIAs.
  *
- * We don't need to do this when using LinuxBIOS, GPIO15 is never lowered there.
+ * We don't need to do this when using coreboot, GPIO15 is never lowered there.
  */
 static int board_via_epia_m(const char *name)
 {
        struct pci_dev *dev;
-       unsigned int base;
+       uint16_t base;
        uint8_t val;
 
        dev = pci_dev_find(0x1106, 0x3177);     /* VT8235 ISA bridge */
@@ -173,10 +171,10 @@ static int board_via_epia_m(const char *name)
        /* Get Power Management IO address. */
        base = pci_read_word(dev, 0x88) & 0xFF80;
 
-       /* enable GPIO15 which is connected to write protect. */
-       val = inb(base + 0x4D);
+       /* Enable GPIO15 which is connected to write protect. */
+       val = INB(base + 0x4D);
        val |= 0x80;
-       outb(val, base + 0x4D);
+       OUTB(val, base + 0x4D);
 
        return 0;
 }
@@ -199,22 +197,44 @@ static int board_asus_a7v8x_mx(const char *name)
                return -1;
        }
 
-       /* This bit is marked reserved actually */
+       /* This bit is marked reserved actually. */
        val = pci_read_byte(dev, 0x59);
        val &= 0x7F;
        pci_write_byte(dev, 0x59, val);
 
-       /* Raise ROM MEMW# line on Winbond w83697 SuperIO */
+       /* Raise ROM MEMW# line on Winbond W83697 Super I/O. */
        w836xx_ext_enter(0x2E);
 
-       if (!(wbsio_read(0x2E, 0x24) & 0x02))   /* flash rom enabled? */
-               wbsio_mask(0x2E, 0x24, 0x08, 0x08);     /* enable MEMW# */
+       if (!(wbsio_read(0x2E, 0x24) & 0x02))   /* Flash ROM enabled? */
+               wbsio_mask(0x2E, 0x24, 0x08, 0x08);     /* Enable MEMW#. */
 
        w836xx_ext_leave(0x2E);
 
        return 0;
 }
 
+/**
+ * Suited for VIAs EPIA SP.
+ */
+static int board_via_epia_sp(const char *name)
+{
+       struct pci_dev *dev;
+       uint8_t val;
+
+       dev = pci_dev_find(0x1106, 0x3227);     /* VT8237R ISA bridge */
+       if (!dev) {
+               fprintf(stderr, "\nERROR: VT8237R ISA bridge not found.\n");
+               return -1;
+       }
+
+       /* All memory cycles, not just ROM ones, go to LPC */
+       val = pci_read_byte(dev, 0x59);
+       val &= ~0x80;
+       pci_write_byte(dev, 0x59, val);
+
+       return 0;
+}
+
 /**
  * Suited for ASUS P5A.
  *
@@ -229,14 +249,14 @@ static int board_asus_p5a(const char *name)
 
 #define ASUSP5A_LOOP 5000
 
-       outb(0x00, 0xE807);
-       outb(0xEF, 0xE803);
+       OUTB(0x00, 0xE807);
+       OUTB(0xEF, 0xE803);
 
-       outb(0xFF, 0xE800);
+       OUTB(0xFF, 0xE800);
 
        for (i = 0; i < ASUSP5A_LOOP; i++) {
-               outb(0xE1, 0xFF);
-               if (inb(0xE800) & 0x04)
+               OUTB(0xE1, 0xFF);
+               if (INB(0xE800) & 0x04)
                        break;
        }
 
@@ -245,13 +265,13 @@ static int board_asus_p5a(const char *name)
                return -1;
        }
 
-       outb(0x20, 0xE801);
-       outb(0x20, 0xE1);
+       OUTB(0x20, 0xE801);
+       OUTB(0x20, 0xE1);
 
-       outb(0xFF, 0xE802);
+       OUTB(0xFF, 0xE802);
 
        for (i = 0; i < ASUSP5A_LOOP; i++) {
-               tmp = inb(0xE800);
+               tmp = INB(0xE800);
                if (tmp & 0x70)
                        break;
        }
@@ -261,24 +281,24 @@ static int board_asus_p5a(const char *name)
                return -1;
        }
 
-       tmp = inb(0xE804);
+       tmp = INB(0xE804);
        tmp &= ~0x02;
 
-       outb(0x00, 0xE807);
-       outb(0xEE, 0xE803);
+       OUTB(0x00, 0xE807);
+       OUTB(0xEE, 0xE803);
 
-       outb(tmp, 0xE804);
+       OUTB(tmp, 0xE804);
 
-       outb(0xFF, 0xE800);
-       outb(0xE1, 0xFF);
+       OUTB(0xFF, 0xE800);
+       OUTB(0xE1, 0xFF);
 
-       outb(0x20, 0xE801);
-       outb(0x20, 0xE1);
+       OUTB(0x20, 0xE801);
+       OUTB(0x20, 0xE1);
 
-       outb(0xFF, 0xE802);
+       OUTB(0xFF, 0xE802);
 
        for (i = 0; i < ASUSP5A_LOOP; i++) {
-               tmp = inb(0xE800);
+               tmp = INB(0xE800);
                if (tmp & 0x70)
                        break;
        }
@@ -296,9 +316,9 @@ static int board_ibm_x3455(const char *name)
        uint8_t byte;
 
        /* Set GPIO lines in the Broadcom HT-1000 southbridge. */
-       outb(0x45, 0xcd6);
-       byte = inb(0xcd7);
-       outb(byte | 0x20, 0xcd7);
+       OUTB(0x45, 0xcd6);
+       byte = INB(0xcd7);
+       OUTB(byte | 0x20, 0xcd7);
 
        return 0;
 }
@@ -311,13 +331,212 @@ static int board_epox_ep_bx3(const char *name)
        uint8_t tmp;
 
        /* Raise GPIO22. */
-       tmp = inb(0x4036);
-       outb(tmp, 0xEB);
+       tmp = INB(0x4036);
+       OUTB(tmp, 0xEB);
 
        tmp |= 0x40;
 
-       outb(tmp, 0x4036);
-       outb(tmp, 0xEB);
+       OUTB(tmp, 0x4036);
+       OUTB(tmp, 0xEB);
+
+       return 0;
+}
+
+/**
+ * Suited for Acorp 6A815EPD.
+ */
+static int board_acorp_6a815epd(const char *name)
+{
+       struct pci_dev *dev;
+       uint16_t port;
+       uint8_t val;
+
+       dev = pci_dev_find(0x8086, 0x2440);     /* Intel ICH2 LPC */
+       if (!dev) {
+               fprintf(stderr, "\nERROR: ICH2 LPC bridge not found.\n");
+               return -1;
+       }
+
+       /* Use GPIOBASE register to find where the GPIO is mapped. */
+       port = (pci_read_word(dev, 0x58) & 0xFFC0) + 0xE;
+
+       val = INB(port);
+       val |= 0x80; /* Top Block Lock -- pin 8 of PLCC32 */
+       val |= 0x40; /* Lower Blocks Lock -- pin 7 of PLCC32 */
+       OUTB(val, port);
+
+       return 0;
+}
+
+/**
+ * Suited for Artec Group DBE61 and DBE62.
+ */
+static int board_artecgroup_dbe6x(const char *name)
+{
+#define DBE6x_MSR_DIVIL_BALL_OPTS      0x51400015
+#define DBE6x_PRI_BOOT_LOC_SHIFT       (2)
+#define DBE6x_BOOT_OP_LATCHED_SHIFT    (8)
+#define DBE6x_SEC_BOOT_LOC_SHIFT       (10)
+#define DBE6x_PRI_BOOT_LOC             (3 << DBE6x_PRI_BOOT_LOC_SHIFT)
+#define DBE6x_BOOT_OP_LATCHED          (3 << DBE6x_BOOT_OP_LATCHED_SHIFT)
+#define DBE6x_SEC_BOOT_LOC             (3 << DBE6x_SEC_BOOT_LOC_SHIFT)
+#define DBE6x_BOOT_LOC_FLASH           (2)
+#define DBE6x_BOOT_LOC_FWHUB           (3)
+
+       unsigned long msr[2];
+       int msr_fd;
+       unsigned long boot_loc;
+
+       msr_fd = open("/dev/cpu/0/msr", O_RDWR);
+       if (msr_fd == -1) {
+               perror("open /dev/cpu/0/msr");
+               return -1;
+       }
+
+       if (lseek(msr_fd, DBE6x_MSR_DIVIL_BALL_OPTS, SEEK_SET) == -1) {
+               perror("lseek");
+               close(msr_fd);
+               return -1;
+       }
+
+       if (read(msr_fd, (void*) msr, 8) != 8) {
+               perror("read");
+               close(msr_fd);
+               return -1;
+       }
+
+       if ((msr[0] & (DBE6x_BOOT_OP_LATCHED)) ==
+           (DBE6x_BOOT_LOC_FWHUB << DBE6x_BOOT_OP_LATCHED_SHIFT))
+               boot_loc = DBE6x_BOOT_LOC_FWHUB;
+       else
+               boot_loc = DBE6x_BOOT_LOC_FLASH;
+
+       msr[0] &= ~(DBE6x_PRI_BOOT_LOC | DBE6x_SEC_BOOT_LOC);
+       msr[0] |= ((boot_loc << DBE6x_PRI_BOOT_LOC_SHIFT) |
+           (boot_loc << DBE6x_SEC_BOOT_LOC_SHIFT));
+
+       if (lseek(msr_fd, DBE6x_MSR_DIVIL_BALL_OPTS, SEEK_SET) == -1) {
+               perror("lseek");
+               close(msr_fd);
+               return -1;
+       }
+
+       if (write(msr_fd, (void*) msr, 8) != 8) {
+               perror("write");
+               close(msr_fd);
+               return -1;
+       }
+
+       close(msr_fd);
+       return 0;
+}
+
+/**
+ * Set the specified GPIO on the specified ICHx southbridge to high.
+ *
+ * @param name The name of this board.
+ * @param ich_vendor PCI vendor ID of the specified ICHx southbridge.
+ * @param ich_device PCI device ID of the specified ICHx southbridge.
+ * @param gpiobase_reg GPIOBASE register offset in the LPC bridge.
+ * @param gp_lvl Offset of GP_LVL register in I/O space, relative to GPIOBASE.
+ * @param gp_lvl_bitmask GP_LVL bitmask (set GPIO bits to 1, all others to 0).
+ * @param gpio_bit The bit (GPIO) which shall be set to high.
+ * @return If the write-enable was successful return 0, otherwise return -1.
+ */
+static int ich_gpio_raise(const char *name, uint16_t ich_vendor,
+                         uint16_t ich_device, uint8_t gpiobase_reg,
+                         uint8_t gp_lvl, uint32_t gp_lvl_bitmask,
+                         unsigned int gpio_bit)
+{
+       struct pci_dev *dev;
+       uint16_t gpiobar;
+       uint32_t reg32;
+
+       dev = pci_dev_find(ich_vendor, ich_device);     /* Intel ICHx LPC */
+       if (!dev) {
+               fprintf(stderr, "\nERROR: ICHx LPC dev %4x:%4x not found.\n",
+                       ich_vendor, ich_device);
+               return -1;
+       }
+
+       /* Use GPIOBASE register to find the I/O space for GPIO. */
+       gpiobar = pci_read_word(dev, gpiobase_reg) & gp_lvl_bitmask;
+
+       /* Set specified GPIO to high. */
+       reg32 = INL(gpiobar + gp_lvl);
+       reg32 |= (1 << gpio_bit);
+       OUTL(reg32, gpiobar + gp_lvl);
+
+       return 0;
+}
+
+/**
+ * Suited for ASUS P4B266.
+ */
+static int ich2_gpio22_raise(const char *name)
+{
+       return ich_gpio_raise(name, 0x8086, 0x2440, 0x58, 0x0c, 0xffc0, 22);
+}
+
+static int board_kontron_986lcd_m(const char *name)
+{
+       struct pci_dev *dev;
+       uint16_t gpiobar;
+       uint32_t val;
+
+#define ICH7_GPIO_LVL2 0x38
+
+       dev = pci_dev_find(0x8086, 0x27b8);     /* Intel ICH7 LPC */
+       if (!dev) {
+               // This will never happen on this board
+               fprintf(stderr, "\nERROR: ICH7 LPC bridge not found.\n");
+               return -1;
+       }
+
+       /* Use GPIOBASE register to find where the GPIO is mapped. */
+       gpiobar = pci_read_word(dev, 0x48) & 0xfffc;
+
+       val = INL(gpiobar + ICH7_GPIO_LVL2);    /* GP_LVL2 */
+       printf_debug("\nGPIOBAR=0x%04x GP_LVL: 0x%08x\n", gpiobar, val);
+
+       /* bit 2 (0x04) = 0 #TBL --> bootblock locking = 1
+        * bit 2 (0x04) = 1 #TBL --> bootblock locking = 0
+        * bit 3 (0x08) = 0 #WP --> block locking = 1
+        * bit 3 (0x08) = 1 #WP --> block locking = 0
+        *
+        * To enable full block locking, you would do:
+        *     val &= ~ ((1 << 2) | (1 << 3));
+        */
+       val |= (1 << 2) | (1 << 3);
+
+       OUTL(val, gpiobar + ICH7_GPIO_LVL2);
+
+       return 0;
+}
+
+/**
+ * Suited for:
+ *   - BioStar P4M80-M4: Intel P4 + VIA P4M800 + VT8237
+ *   - GIGABYTE GA-7VT600: AMD K7 + VIA KT600 + VT8237
+ */
+static int board_biostar_p4m80_m4(const char *name)
+{
+       /* enter IT87xx conf mode */
+       OUTB(0x87, 0x2e);
+       OUTB(0x01, 0x2e);
+       OUTB(0x55, 0x2e);
+       OUTB(0x55, 0x2e);
+
+       /* select right flash chip */
+       wbsio_mask(0x2e, 0x22, 0x80, 0x80);
+
+       /* bit 3: flash chip write enable
+        * bit 7: map flash chip at 1MB-128K (why though? ignoring this.)
+        */
+       wbsio_mask(0x2e, 0x24, 0x04, 0x04);
+
+       /* exit IT87xx conf mode */
+       wbsio_write(0x2, 0x2e, 0x2);
 
        return 0;
 }
@@ -329,35 +548,41 @@ static int board_epox_ep_bx3(const char *name)
  * subsystem/card IDs in a sane manner.
  *
  * Keep the second set NULLed if it should be ignored.
+ *
+ * Keep the subsystem IDs NULLed if they don't identify the board fully.
  */
 struct board_pciid_enable {
-       /* Any device, but make it sensible, like the isa bridge. */
+       /* Any device, but make it sensible, like the ISA bridge. */
        uint16_t first_vendor;
        uint16_t first_device;
        uint16_t first_card_vendor;
        uint16_t first_card_device;
 
        /* Any device, but make it sensible, like 
-        * the host bridge. May be NULL
+        * the host bridge. May be NULL.
         */
        uint16_t second_vendor;
        uint16_t second_device;
        uint16_t second_card_vendor;
        uint16_t second_card_device;
 
-       /* From linuxbios table */
-       char *lb_vendor;
-       char *lb_part;
+       /* The vendor / part name from the coreboot table. */
+       const char *lb_vendor;
+       const char *lb_part;
 
-       char *name;
+       const char *name;
        int (*enable) (const char *name);
 };
 
 struct board_pciid_enable board_pciid_enables[] = {
+       {0x8086, 0x1a30, 0x1043, 0x8070, 0x8086, 0x244b, 0x1043, 0x8028,
+        NULL, NULL, "ASUS P4B266", ich2_gpio22_raise},
        {0x10de, 0x0360, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
         "gigabyte", "m57sli", "GIGABYTE GA-M57SLI-S4", it87xx_probe_spi_flash},
        {0x10de, 0x03e0, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
         "gigabyte", "m61p", "GIGABYTE GA-M61P-S3", it87xx_probe_spi_flash},
+       {0x1039, 0x0761, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
+        "gigabyte", "2761gxdk", "GIGABYTE GA-2761GXDK", it87xx_probe_spi_flash},
        {0x1022, 0x7468, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
         "iwill", "dk8_htx", "IWILL DK8-HTX", w83627hf_gpio24_raise_2e},
        {0x10de, 0x005e, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
@@ -368,6 +593,8 @@ struct board_pciid_enable board_pciid_enables[] = {
         NULL, NULL, "VIA EPIA M/MII/...", board_via_epia_m},
        {0x1106, 0x3177, 0x1043, 0x80A1, 0x1106, 0x3205, 0x1043, 0x8118,
         NULL, NULL, "ASUS A7V8-MX SE", board_asus_a7v8x_mx},
+       {0x1106, 0x3227, 0x1106, 0xAA01, 0x1106, 0x0259, 0x1106, 0xAA01,
+        NULL, NULL, "VIA EPIA SP", board_via_epia_sp},
        {0x8086, 0x1076, 0x8086, 0x1176, 0x1106, 0x3059, 0x10f1, 0x2498,
         NULL, NULL, "Tyan Tomcat K7M", board_asus_a7v8x_mx},
        {0x10B9, 0x1541, 0x0000, 0x0000, 0x10B9, 0x1533, 0x0000, 0x0000,
@@ -376,20 +603,52 @@ struct board_pciid_enable board_pciid_enables[] = {
         "ibm", "x3455", "IBM x3455", board_ibm_x3455},
        {0x8086, 0x7110, 0x0000, 0x0000, 0x8086, 0x7190, 0x0000, 0x0000,
         "epox", "ep-bx3", "EPoX EP-BX3", board_epox_ep_bx3},
+       {0x8086, 0x1130, 0x0000, 0x0000, 0x105a, 0x0d30, 0x105a, 0x4d33,
+        "acorp", "6a815epd", "Acorp 6A815EPD", board_acorp_6a815epd},
+       {0x1022, 0x2090, 0x0000, 0x0000, 0x1022, 0x2080, 0x0000, 0x0000,
+        "artecgroup", "dbe61", "Artec Group DBE61", board_artecgroup_dbe6x},
+       {0x1022, 0x2090, 0x0000, 0x0000, 0x1022, 0x2080, 0x0000, 0x0000,
+        "artecgroup", "dbe62", "Artec Group DBE62", board_artecgroup_dbe6x},
+       {0x8086, 0x27b8, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
+        "kontron", "986lcd-m", "Kontron 986LCD-M", board_kontron_986lcd_m},
+       {0x1106, 0x3149, 0x1565, 0x3206, 0x1106, 0x3344, 0x1565, 0x1202,
+        NULL, NULL, "BioStar P4M80-M4", board_biostar_p4m80_m4},
+       {0x1106, 0x3227, 0x1458, 0x5001, 0x10ec, 0x8139, 0x1458, 0xe000,
+        NULL, NULL, "GIGABYTE GA-7VT600", board_biostar_p4m80_m4},
        {0, 0, 0, 0, 0, 0, 0, 0, NULL, NULL}    /* Keep this */
 };
 
+void print_supported_boards(void)
+{
+       int i;
+
+       printf("\nSupported mainboards (this list is not exhaustive!):\n\n");
+
+       for (i = 0; board_pciid_enables[i].name != NULL; i++) {
+               if (board_pciid_enables[i].lb_vendor != NULL) {
+                       printf("%s (-m %s:%s)\n", board_pciid_enables[i].name,
+                              board_pciid_enables[i].lb_vendor,
+                              board_pciid_enables[i].lb_part);
+               } else {
+                       printf("%s (autodetected)\n",
+                              board_pciid_enables[i].name);
+               }
+       }
+
+       printf("\nSee also: http://coreboot.org/Flashrom\n");
+}
+
 /**
- * Match boards on LinuxBIOS table gathered vendor and part name.
+ * Match boards on coreboot table gathered vendor and part name.
  * Require main PCI IDs to match too as extra safety.
  */
-static struct board_pciid_enable *board_match_linuxbios_name(char *vendor, 
-                                                               char *part)
+static struct board_pciid_enable *board_match_coreboot_name(const char *vendor, const char *part)
 {
        struct board_pciid_enable *board = board_pciid_enables;
+       struct board_pciid_enable *partmatch = NULL;
 
        for (; board->name; board++) {
-               if (!board->lb_vendor || strcmp(board->lb_vendor, vendor))
+               if (vendor && (!board->lb_vendor || strcmp(board->lb_vendor, vendor)))
                        continue;
 
                if (!board->lb_part || strcmp(board->lb_part, part))
@@ -401,9 +660,26 @@ static struct board_pciid_enable *board_match_linuxbios_name(char *vendor,
                if (board->second_vendor &&
                        !pci_dev_find(board->second_vendor, board->second_device))
                        continue;
-               return board;
+
+               if (vendor)
+                       return board;
+
+               if (partmatch) {
+                       /* a second entry has a matching part name */
+                       printf("AMBIGUOUS BOARD NAME: %s\n", part);
+                       printf("At least vendors '%s' and '%s' match.\n",
+                               partmatch->lb_vendor, board->lb_vendor);
+                       printf("Please use the full -m vendor:part syntax.\n");
+                       return NULL;
+               }
+               partmatch = board;
        }
+
+       if (partmatch)
+               return partmatch;
+
        printf("NOT FOUND %s:%s\n", vendor, part);
+
        return NULL;
 }
 
@@ -444,19 +720,19 @@ static struct board_pciid_enable *board_match_pci_card_ids(void)
        return NULL;
 }
 
-int board_flash_enable(char *vendor, char *part)
+int board_flash_enable(const char *vendor, const char *part)
 {
        struct board_pciid_enable *board = NULL;
        int ret = 0;
 
-       if (vendor && part)
-               board = board_match_linuxbios_name(vendor, part);
+       if (part)
+               board = board_match_coreboot_name(vendor, part);
 
        if (!board)
                board = board_match_pci_card_ids();
 
        if (board) {
-               printf("Found board \"%s\": enabling flash write... ",
+               printf("Found board \"%s\", enabling flash write... ",
                        board->name);
 
                ret = board->enable(board->name);