Date: Fri, 20 May 2005 18:04:49 +0200 From: "Walter C. Pelissero" <walter@pelissero.de> To: Larry Baird <lab@gta.com> Cc: freebsd-hackers@freebsd.org Subject: Re: WRAP.1E and geode.c Message-ID: <17038.2721.616026.745710@zaphod.home.loc> In-Reply-To: <20050520134441.75330.qmail@gta.com> References: <17037.14274.229768.369886@zaphod.home.loc> <20050520134441.75330.qmail@gta.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Larry Baird writes:
> In article <17037.14274.229768.369886@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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?17038.2721.616026.745710>