[coreboot] flashrom patch: add support for Abit AB-BM6 board

Tim ter Laak timl at scintilla.utwente.nl
Wed Sep 3 11:42:50 CEST 2008


Hello all,

This patch adds support for the Abit AB-BM6 mainboard to flashrom.

The biggest part is a generic function to lower a GPIO line on the PIIX4E 
southbridge, copied and adapted from its ich_gpio_raise() counterpart 
(mostly lower instead of raise, followed names from the PIIX4E datasheet). 
The board specific function then uses this to lower GPO 26.

Signed-off-by: Tim ter Laak <timl at scintilla.utwente.nl>
---

The patch was made from the flashrom subdir, because svn diff from the 
top-level coreboot dir (as per the directions in the Development 
Guidelines on the wiki) didn't seem to work for me. Probably because 
flashrom is fetched externally when checking out coreboot? Anyway, I hope 
this is okay. If you really prefer a diff from the top-level dir just let 
me know, and I'll regenerate it with gnu diff.

Kind regards,
 	Tim.


Index: board_enable.c
===================================================================
--- board_enable.c	(revision 3559)
+++ board_enable.c	(working copy)
@@ -349,6 +349,58 @@
  }

  /**
+ * Set the specified GPIO on the specified PIIX4 southbridge to low.
+ *
+ * @param name The name of this board.
+ * @param piix_vendor PCI vendor ID of the specified PIIX4 southbridge. (0x8086)
+ * @param piix_device PCI device ID of the specified PIIX4 southbridge, function Power Management (0x7113 for PIIX4E)
+ * @param pmbase_reg PMBASE register offset in the bridge. (0x40 for PIIX4E)
+ * @param pmbase_mask PMBASE bitmask (0xFFC0)
+ * @param gporeg Offset of GPOREG register in I/O space, relative to GPIOBASE. (0x34)
+ * @param gpio_bit The bit (GPIO) which shall be set to low.
+ * @return If the write-enable was successful return 0, otherwise return -1.
+ */
+static int piix4_gpio_lower(const char *name, uint16_t piix_vendor,
+			  uint16_t piix_device, uint8_t pmbase_reg,
+			  uint8_t gporeg, uint32_t pmbase_mask,
+			  unsigned int gpio_bit)
+{
+	struct pci_dev *dev;
+	uint16_t pmbar;
+	uint32_t reg32;
+
+	dev = pci_dev_find(piix_vendor, piix_device);     /* Intel PIIX4 ACPI function */
+	if (!dev) {
+		fprintf(stderr, "\nERROR: PIIX4 dev %4x:%4x not found.\n",
+			piix_vendor, piix_device);
+		return -1;
+	}
+
+	/* Use PMBASE register to find the I/O space for GPIO. */
+	pmbar = pci_read_word(dev, pmbase_reg) & pmbase_mask;
+
+	/* Set specified GPIO to high. */
+	reg32 = INL(pmbar + gporeg);
+	OUTB(0x00, 0xEB);	/* dummy write to unused port as delay */
+ 
+	reg32 &= ~(1 << gpio_bit);
+	OUTL(reg32, pmbar + gporeg);
+	OUTB(0x00, 0xEB);	/* another delay */
+
+	return 0;
+}
+
+/**
+ * Suited for Abit AB-BM6.
+ */
+static int piix4_gpio26_lower(const char *name)
+{
+	return piix4_gpio_lower(name, 0x8086, 0x7113, 0x40, 0x34, 0xffc0, 26);
+}
+
+
+
+/**
   * Suited for Acorp 6A815EPD.
   */
  static int board_acorp_6a815epd(const char *name)
@@ -672,6 +724,8 @@
   	 NULL, NULL, "GIGABYTE GA-7VT600", board_biostar_p4m80_m4},
  	{0x1106, 0x3149, 0x1462, 0x7094, 0x10ec, 0x8167, 0x1462, 0x094c,
  	 NULL, NULL, "MSI K8T Neo2", w83627thf_gpio4_4_raise_2e},
+	{0x8086, 0x7190, 0x0000, 0x0000, 0x8086, 0x7110, 0x0000, 0x0000,
+	 "abit", "ab-bm6", "Abit AB-BM6", piix4_gpio26_lower},
  	{0, 0, 0, 0, 0, 0, 0, 0, NULL, NULL}	/* Keep this */
  };





More information about the coreboot mailing list