Re: [PATCH 1/2] tty serial: xr17c15x driver

[One Thousand Gnomes]

On Wed, 9 Apr 2014 23:10:35 -0500
Michael Welling <mwelling@xxxxxxxx> wrote:

> The 8250_core driver is supposed to support these devices but it has
> soft lockups on some hardware. This driver was ported from the vendors
> 2.6.37 driver and has been proven to work.
> 
> I do not expect it to be merged, I am looking for feedback and
> possible fixes.

The thing to do is going to be to compare the various methods in the
driver with the 8250.c driver.

It appeas to be based upon an ancient 8250.c so if you want to pick up
all the changes then you need to compare with 8250.c, and if you want to
fix 8250.c instead you need to do exactly the same job so that seems a
more productive way to go about it IMHO.

There are bits of the driver you've pasted like the loop over ports for
shared IRQs you can't fix however, because they assume there is one
driver for all the ports. Not sure it matters though given your devices
are all going to be PCI bus.

Other thing to do would be to instrument serial_in/out and run the two
drivers briefly and compare the sequence of accesses and settings used.

Alan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/