WRAP.1E and geode.c

[Walter C. Pelissero]

Larry Baird writes:
 > In article <17037.14274.229768.369886 at zaphod.home.loc> you wrote:
 > > Reading geode.c it appears (at least to me) that the led devices are
 > > created for the WRAP.1C but not for the WRAP.1E.
 > > 
 > > Reading the PC-Engines documentation it looks to me as WRAP.1C and
 > > WRAP1.E were identical when it comes to the LEDs, thus it would be
 > > nice to have the same entries in /dev.
 > > 
 > > Anyone with a better understanding of this platform cares to comment
 > > on this?
 > The BIOS id string on the wrap board has changed.  Following patch
 > for geode.c should fix the problem.

Some more good old trial-and-error activity let me come up with the
answer to my problem.  The BIOS id string has indeed moved.  The
following patch makes geode.c play nicely with newer versions of the
WARP boards:

--- geode.c	16 Jun 2004 09:47:07 -0000	1.5
+++ geode.c	20 May 2005 15:42:28 -0000
@@ -145,8 +145,8 @@
 			led1b = 20;
 			led1 = led_create(led_func, &led1b, "error");
 		} else if (NULL !=
-		    bios_string(0xf9000, 0xf9000, "PC Engines WRAP.1C ", 0)) {
-			printf("PC Engines WRAP.1C platfrom\n");
+		    bios_string(0xf9000, 0xf9010, "PC Engines WRAP", 0)) {
+			printf("PC Engines WRAP platfrom\n");
 			led1b = -2;
 			led2b = -3;
 			led3b = -18;


If no one has objections I'd file a PR.

-- 
walter pelissero
http://www.pelissero.de