[lkml]   [2004]   [Nov]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Patch in this message
SubjectExar ST16C2550 rev A2 bug
Hi Russell,

There seem to be an increasing number of the above UARTs floating
around and I'm wondering if we can do something to better detect and
work around their flaw. Exar has documented the problem and their
proposed serial driver changes to work around the issue here:

In short, they had a mask problem that inadvertently exposes the DVID,
DREV and EFR registers on the chip. This causes us to detect the device
as a 16650V2 and try to make use of the FIFO as if it were 32 bytes.
The previous version of the UART detected correctly as a 16550A and
worked fine. Exar's proposed solution is simply to only set the port
type to 16650V2 if size_fifo() == 32, or simply:

===== drivers/serial/8250.c 1.91 vs edited =====
--- 1.91/drivers/serial/8250.c 2004-11-13 17:43:56 -07:00
+++ edited/drivers/serial/8250.c 2004-11-17 11:13:05 -07:00
@@ -570,7 +570,7 @@
if (size_fifo(up) == 64)
up->port.type = PORT_16654;
- else
+ else if (size_fifo(up) == 32)
up->port.type = PORT_16650V2;

The proposed 2.4 solution is quite similar. It seems reasonable to
verify the reported FIFO size, but perhaps you have a better solution.
For anyone currently impacted by this bug, a simple yet limited
workaround is to change the uart type using setserial. Thanks,

Alex Williamson HP Linux & Open Source Lab

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-03-22 14:08    [W:0.040 / U:10.004 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site