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

Tim ter Laak timl at scintilla.utwente.nl
Tue Sep 7 21:10:35 CEST 2010


This patch adds support for the Abit BM6 board, using DMI string
identification, and lists it in print.c [v3].

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

On Tue, 7 Sep 2010, Uwe Hermann wrote:

> Looks good to me too, but two small requests:
>
>> + *  - Abit AB-BM6
>
> s/Abit/abit/ for consistency (though we may change the convention later).
>
> And: "AB-BM6" should be just "BM6" as that's the name on the website.
> Where does the "AB-" prefix come from? Does it say so on the PCB or
> something?

Fair enough. It's worth it to keep an eye on grep-ability ;)

And yeah, the AB-BM6 is the marking on the board itself. On closer 
inspection, the AB part must be there just to express the Abit brand, as I 
cannot find their name printed anywhere on the board.

Updated patch is attached.

Tim.

> Either way:
>
> Acked-by: Uwe Hermann <uwe at hermann-uwe.de>
-------------- next part --------------
Index: print.c
===================================================================
--- print.c	(revision 1155)
+++ print.c	(working copy)
@@ -283,6 +283,7 @@
 	B("A-Trend",	"ATC-6220",		1, "http://www.motherboard.cz/mb/atrend/atc6220.htm", NULL),
 	B("abit",	"AN-M2",		1, "http://www.abit.com.tw/page/de/motherboard/motherboard_detail.php?DEFTITLE=Y&fMTYPE=Socket%20AM2&pMODEL_NAME=AN-M2", NULL),
 	B("abit",	"AX8",			1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?DEFTITLE=Y&fMTYPE=Socket%20939&pMODEL_NAME=AX8", NULL),
+	B("abit",	"BM6",			1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?pMODEL_NAME=BM6&fMTYPE=Socket%20370", NULL),
 	B("abit",	"Fatal1ty F-I90HD",	1, "http://www.abit.com.tw/page/de/motherboard/motherboard_detail.php?pMODEL_NAME=Fatal1ty+F-I90HD&fMTYPE=LGA775", NULL),
 	B("abit",	"IC7",			1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?pMODEL_NAME=IC7&fMTYPE=Socket%20478", NULL),
 	B("abit",	"IP35",			1, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?fMTYPE=LGA775&pMODEL_NAME=IP35", NULL),
Index: board_enable.c
===================================================================
--- board_enable.c	(revision 1155)
+++ board_enable.c	(working copy)
@@ -1103,6 +1103,15 @@
 
 /*
  * Suited for:
+ *  - abit BM6
+ */
+static int intel_piix4_gpo26_lower(void)
+{
+        return intel_piix4_gpo_set(26, 0);
+}
+           
+/*
+ * Suited for:
  *  - Intel SE440BX-2
  */
 static int intel_piix4_gpo27_lower(void)
@@ -1773,6 +1782,7 @@
 	/* first pci-id set [4],          second pci-id set [4],          dmi identifier coreboot id [2],             vendor name    board name       max_rom_...  OK? flash enable */
 #if defined(__i386__) || defined(__x86_64__)
 	{0x10DE, 0x0547, 0x147B, 0x1C2F,  0x10DE, 0x0548, 0x147B, 0x1C2F, NULL,          NULL,         NULL,          "abit",        "AN-M2",                 0,   NT, nvidia_mcp_gpio2_raise},
+	{0x8086, 0x7190,      0,      0,  0x8086, 0x7110,      0,      0, "^i440BX-W977 (BM6)$", NULL, NULL,          "abit",        "BM6",                   0,   OK, intel_piix4_gpo26_lower},
 	{0x8086, 0x24d3, 0x147b, 0x1014,  0x8086, 0x2578, 0x147b, 0x1014, NULL,          NULL,         NULL,          "abit",        "IC7",                   0,   NT, intel_ich_gpio23_raise},
 	{0x8086, 0x2930, 0x147b, 0x1084,  0x11ab, 0x4364, 0x147b, 0x1084, NULL,          NULL,         NULL,          "abit",        "IP35",                  0,   OK, intel_ich_gpio16_raise},
 	{0x8086, 0x2930, 0x147b, 0x1083,  0x10ec, 0x8167, 0x147b, 0x1083, NULL,          NULL,         NULL,          "abit",        "IP35 Pro",              0,   OK, intel_ich_gpio16_raise},


More information about the flashrom mailing list