2003-03-21 18:33:49

by Alan

Subject: PATCH: PC9800 has a slight funny on 8250_pnp

diff -u --new-file --recursive --exclude-from /usr/src/exclude linux-2.5.65/drivers/serial/8250_pnp.c linux-2.5.65-ac2/drivers/serial/8250_pnp.c
--- linux-2.5.65/drivers/serial/8250_pnp.c 2003-03-03 19:20:12.000000000 +0000
+++ linux-2.5.65-ac2/drivers/serial/8250_pnp.c 2003-03-14 01:07:59.000000000 +0000
@@ -188,6 +188,8 @@
{ "MVX00A1", 0 },
/* PC Rider K56 Phone System PnP */
{ "MVX00F2", 0 },
+ /* NEC 98NOTE SPEAKER PHONE FAX MODEM(33600bps) */
+ { "nEC8241", 0 },
/* Pace 56 Voice Internal Plug & Play Modem */
{ "PMC2430", 0 },
/* Generic */
@@ -373,6 +375,9 @@
((port->min == 0x2f8) ||
(port->min == 0x3f8) ||
(port->min == 0x2e8) ||
+#ifdef CONFIG_X86_PC9800
+ (port->min == 0x8b0) ||
+#endif
(port->min == 0x3e8)))
return 0;
}

2003-03-23 00:08:10

by Osamu Tomita

Subject: Re: PATCH: PC9800 has a slight funny on 8250_pnp

Joe Perches wrote:
>
> On Fri, 2003-03-21 at 11:47, Alan Cox wrote:
> > diff -u --new-file --recursive --exclude-from /usr/src/exclude linux-2.5.65/drivers/serial/8250_pnp.c linux-2.5.65-ac2/drivers/serial/8250_pnp.c
> > --- linux-2.5.65/drivers/serial/8250_pnp.c 2003-03-03 19:20:12.000000000 +0000
> > +++ linux-2.5.65-ac2/drivers/serial/8250_pnp.c 2003-03-14 01:07:59.000000000 +0000
> > @@ -188,6 +188,8 @@
> > { "MVX00A1", 0 },
> > /* PC Rider K56 Phone System PnP */
> > { "MVX00F2", 0 },
> > + /* NEC 98NOTE SPEAKER PHONE FAX MODEM(33600bps) */
> > + { "nEC8241", 0 },
>
> Lowercase "n"? Shouldn't it be "NEC8241"?
This onboard modem returns lowercase 'n'. I don't know it's a bug
or any other reason. I tested it on many PC-98s, it works fine.

Thanks,
Osamu Tomita