Messages in this thread | | | Date | Wed, 10 Nov 2010 16:40:04 +0000 | From | Sean Young <> | Subject | Re: [PATCH] Fix Winbond CIR driver initialisation |
| |
On Sun, Nov 07, 2010 at 03:46:48PM +0000, Alan Cox wrote: > > A workaround is to boot with argument 8250.nr_uarts=1. This is not really > > desirable, so this patch makes ttyS1 go away. > > setserial can also be used for this surely ? > > > + dev_info(device, "Unregistering phony serial port ttyS1\n"); > > + serial8250_unregister_port(1); > > + ok = request_region(data->sbase, SP_IOMEM_LEN, DRVNAME); > > That's a hack that is only going to work on specific systems where it is > mapped the way you expect and doing stuff behind the back of the serial > driver. > > I'm not averse to a better solution but it needs to be general and > maintainable. Is there a way to identify the presence of the windbond CIR > device as opposed to an 8250 ?
The past couple of days I've been trying to find a way of identifying the winbond cir device correctly in the 8250 driver by its registers, but so far I've had no luck. It is a device which can operate in CEIR, UART and one the irda modes, but all I can find for identifying what is physically connected is querying the Super I/O model or the PNP identifier for CIR.
The Super I/O model might not be enough in cases where different things can be connected to it physically.
The hardware can be flipped between various IR (SIR/CIR/etc) and normal uart modes. We could have an new call which represents this a bit more closely.
To ensure it is mapped as expected, the call should include the I/O port range and irq.
bool serial8250_port_non_uart(int ioport, int ioportlen, int irq);
The 8250 serial driver would unregister the port which matches this, and unregister the corresponding port, returning true if such a port can be found.
This problem here is also one which is encountered by some of the irda drivers, so it would be nice to solve as well for those too.
Would that make more sense?
Regards, Sean
| |