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

login
register
about
Submitter Tim ter Laak
Date 2010-09-07 19:10:35
Message ID <Pine.LNX.4.64.1009072059020.28655@virtuscin>
Download mbox | patch
Permalink /patch/1883/
State Not Applicable
Headers show

Comments

Tim ter Laak - 2010-09-07 19:10:35
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@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@hermann-uwe.de>
Uwe Hermann - 2010-09-13 23:00:24
On Tue, Sep 07, 2010 at 09:10:35PM +0200, Tim ter Laak wrote:
> 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@scintilla.utwente.nl>

Thanks a lot, committed as r1163.


Uwe.

Patch

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},