On Sat, Mar 19, 2005 at 02:13:51PM -0800, Andrew Morton wrote:
au1x00_uart -----------
Maintainer: unknown (akpm - any ideas?)
Ralf.
Actually Pete Popov (ppopov@xxxxxxxxxxxxxxxxx) who I put on the cc.
Thanks :)
This is a complete clone of 8250.c, which includes all the 8250-specific structure names.
Specifically, I'd like to see the following addressed:
- Please clean this up to use au1x00-specific names. - this driver is lagging behind with fixes that the other drivers are getting. Is au1x00_uart actually maintained?
Sort of; much of the Alchemy development effort is still going into 2.4.
It works and no one has complained about any bugs. But you're right, fixes going into other drivers have not made it into this one.
- the usage of UPIO_HUB6 (this driver doesn't support hub6 cards) - __register_serial, register_serial, unregister_serial (this driver doesn't support PCMCIA cards, all of which are based on 8250-compatible devices.) - early_serial_setup (should we really have the function name duplicated across different hardware drivers?)
No argument here. Pete says the AMD Alchemy UART is just different enough to be hard to handle in the 8250 and so the driver is just an ugly chainsawed version of the 8250.c
The main reason is I wish to kill off uart_register_port and uart_unregister_port, but these drivers are using it.
I tried a couple of times to cleanly add support to the 8250 for the Au1x serial. The uart is just different enough to make that hard, though I admit I never spent too much time on it. Sounds like it's time to revisit it again.
Pete
