On Mon, 13 Jun 2011 10:19:26 +0100 Jamie Iles <jamie@xxxxxxxxxxxxx> wrote: > On Fri, Jun 10, 2011 at 02:57:24PM +0100, Alan Cox wrote: > > > I found this series from Alan > > > (http://www.spinics.net/lists/linux-serial/msg03484.html) which looks > > > like it would do the job if we added the extra irq callback. Ideally > > > we just remove both of the UPIO_DWAPB and UPIO_DWAPB32 and let the > > > platform specify the ops. > > > > I've not yet had time to go back and revisit those patches and debug > > them so they actually work but as and when someone gets time I think > > it's the right basic path to follow, and the irq callback looks > > sensible too. > > As an intermediate step, how about adding the irq callback to > uart_8250_port and removing UPIO_DWAPB and UPIO_DWAPB32 from the driver? Feel free to - it makes the later job easier.