On Wed, 20 Jul 2011 13:42:03 +0200
Stefan Tauner <stefan.tau...@student.tuwien.ac.at> wrote:

> On Sat, 16 Jul 2011 03:10:21 +0200
> Stefan Tauner <stefan.tau...@student.tuwien.ac.at> wrote:
> 
> > On Sat, 16 Jul 2011 02:27:36 +0200
> > Carl-Daniel Hailfinger <c-d.hailfinger.devel.2...@gmx.net> wrote:
> > 
> > > Anything else we need for 0.9.4?
> > > Do we have any other unmerged bugfixes?
> > > Doc fixes? The man page is missing some features.
> 
> another thing that should be dealt with is the situation with ck804 
> boards. basically it seems as if we should clear one byte on all of
> those to disable some chipset-based write protection.
> 
> the board enable committed for the EP-8NPA7I is incomplete. it needs the
> following as well for the oem BIOS(es):
> http://patchwork.coreboot.org/patch/2125/
> 
> if that is applied in one way or another, the same board enable also
> works for the EP-9NPA7I:
> http://patchwork.coreboot.org/patch/3176/
> 
> if we want to distinguish those two, we would need something similar to
> this, because the pci ids are not enough (but we probably do not):
> http://patchwork.coreboot.org/patch/3131/
> 
> another board that relies on the first patch is the ASUS P5N-E SLI
> (although it needs an additional board enable according to joshua).
> it was reported once back when the EP-8NPA7I board enable was developed
> and also recently by Joël Obrecht (who agreed to test any patches btw).
> 

i have applied what i think is the minimum solution that should be done
for a stable release (i.e. 0.9.4) to my tested stuff branch (attached
as independent patch).
- mark the EP-8NPA7I as not working in print.c and add a comment
  pointing to jonathan's patch.
- add the EP-9NPA7I to print.c like the EP-8NPA7I (epox hp is dead btw)
- add the P5N-E SLI to print.c including a note mentioning the patch
  and the need for a board enable.
- add dmi matching to the EP-8NPA7I board enable and mark it as NT...
  there is no "not working" flag, because we usually do not include not
  working board enables in the first place :) i dont want to remove it
  altogether now though and have added a comment instead that also
  points to jonathan's patch. ok?

-- 
Kind regards/Mit freundlichen Grüßen, Stefan Tauner
>From 70d82ab726e19f523e681bd60704fc50eaa05bc2 Mon Sep 17 00:00:00 2001
From: Stefan Tauner <stefan.tau...@student.tuwien.ac.at>
Date: Wed, 20 Jul 2011 15:29:43 +0200
Subject: [PATCH] mark some ck804 boards as not working

P5N-E SLI, EP-8NPA7I and EP-9NPA7I all need at least this patch:
http://patchwork.coreboot.org/patch/2125/
mark them as not working until it is merged.

Signed-off-by: Stefan Tauner <stefan.tau...@student.tuwien.ac.at>
---
 board_enable.c |    2 +-
 print.c        |    4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/board_enable.c b/board_enable.c
index ffa522a..e2772dc 100644
--- a/board_enable.c
+++ b/board_enable.c
@@ -2027,7 +2027,7 @@ const struct board_pciid_enable board_pciid_enables[] = {
 	{0x10de, 0x03ea, 0x1019, 0x2602,  0x10de, 0x03e0, 0x1019, 0x2602, NULL,         NULL, NULL,           P3, "Elitegroup",  "GeForce6100SM-M",       0,   OK, board_ecs_geforce6100sm_m},
 	{0x1106, 0x3038, 0x1019, 0x0996,  0x1106, 0x3177, 0x1019, 0x0996, NULL,         NULL, NULL,           P3, "Elitegroup",  "K7VTA3",                256, OK, NULL},
 	{0x1106, 0x3177, 0x1106, 0x3177,  0x1106, 0x3059, 0x1695, 0x3005, NULL,         NULL, NULL,           P3, "EPoX",        "EP-8K5A2",              0,   OK, w836xx_memw_enable_2e},
-	{0x10DE, 0x005E, 0x1695, 0x1010,  0x10DE, 0x0050, 0x1695, 0x1010, NULL,         NULL, NULL,           P3, "EPoX",        "EP-8NPA7I",             0,   OK, nvidia_mcp_gpio4_raise},
+	{0x10DE, 0x005E, 0x1695, 0x1010,  0x10DE, 0x0050, 0x1695, 0x1010, "8NPA7I",     NULL, NULL,           P3, "EPoX",        "EP-8NPA7I",             0,   NT, nvidia_mcp_gpio4_raise}, /* needs http://patchwork.coreboot.org/patch/2125/ */
 	{0x10EC, 0x8139, 0x1695, 0x9001,  0x11C1, 0x5811, 0x1695, 0x9015, NULL,         NULL, NULL,           P3, "EPoX",        "EP-8RDA3+",             0,   OK, nvidia_mcp_gpio31_raise},
 	{0x8086, 0x7110,      0,      0,  0x8086, 0x7190,      0,      0, NULL,         "epox", "ep-bx3",     P3, "EPoX",        "EP-BX3",                0,   NT, intel_piix4_gpo22_raise},
 	{0x10de, 0x02f0, 0x105b, 0x0d01,  0x10de, 0x0264, 0x105b, 0x0d01, NULL,         NULL, NULL,           P3, "Foxconn",     "6150K8MD-8EKRSH",       0,   NT, nvidia_mcp_gpio2_raise},
diff --git a/print.c b/print.c
index 4a5885b..f159d8b 100644
--- a/print.c
+++ b/print.c
@@ -481,6 +481,7 @@ const struct board_info boards_known[] = {
 	B("ASUS",	"P5KC",			1, "http://www.asus.com/product.aspx?P_ID=fFZ8oUIGmLpwNMjj";, NULL),
 	B("ASUS",	"P5L-MX",		1, "http://www.asus.com/product.aspx?P_ID=X70d3NCzH2DE9vWH";, NULL),
 	B("ASUS",	"P5GD1 Pro",		1, "http://www.asus.com/product.aspx?P_ID=50M49xQh71EZOeM1";, NULL),
+	B("ASUS",	"P5N-E SLI",		0, "http://www.asus.com/product.aspx?P_ID=KyHOsOKWujC2QguJ";, "Needs a patch (http://patchwork.coreboot.org/patch/2125/) and a board enable."),
 	B("ASUS",	"P5N32-E SLI",		1, "http://www.asus.com/product.aspx?P_ID=vBZLIBtPzYB2bLcb";, NULL),
 	B("ASUS",	"P5ND2-SLI Deluxe",	1, "http://www.asus.com/product.aspx?P_ID=WY7XroDuUImVbgp5";, NULL),
 	B("ASUS",	"P5PE-VM",		1, "http://www.asus.com/product.aspx?P_ID=k3h0ZFVu9Lo1dUvk";, NULL),
@@ -510,7 +511,8 @@ const struct board_info boards_known[] = {
 	B("Elitegroup", "RS485M-M",		1, "http://www.ecs.com.tw/ECSWebSite_2007/Products/ProductsDetail.aspx?CategoryID=1&DetailID=654&DetailName=Feature&MenuID=1&LanID=0";, NULL),
 	B("Emerson",	"ATCA-7360",		1, "http://www.emerson.com/sites/Network_Power/en-US/Products/Product_Detail/Product1/Pages/EmbCompATCA-7360.aspx";, NULL),
 	B("EPoX",	"EP-8K5A2",		1, "http://www.epox.com/product.asp?ID=EP-8K5A2";, NULL),
-	B("EPoX",	"EP-8NPA7I",		1, "http://epox.com/product.asp?ID=EP-8NPA7I";, NULL),
+	B("EPoX",	"EP-8NPA7I",		0, "http://www.epox.com/product.asp?ID=EP-8NPA7I";, "Needs a patch (http://patchwork.coreboot.org/patch/2125/)."),
+	B("EPoX",	"EP-9NPA7I",		0, "http://www.epox.com/product.asp?ID=EP-9NPA7I";, "Needs a patch (http://patchwork.coreboot.org/patch/2125/) and the same board enable as the EP-8NPA7I."),
 	B("EPoX",	"EP-8RDA3+",		1, "http://www.epox.com/product.asp?ID=EP-8RDA3plus";, NULL),
 	B("EPoX",	"EP-BX3",		1, "http://www.epox.com/product.asp?ID=EP-BX3";, NULL),
 	B("EVGA",	"132-CK-NF78",		1, "http://http://www.evga.com/articles/385.asp";, NULL),
-- 
1.7.1

_______________________________________________
flashrom mailing list
flashrom@flashrom.org
http://www.flashrom.org/mailman/listinfo/flashrom

Reply via email to