| To: | Jamie Iles <jamie@jamieiles.com> |
|---|---|
| Subject: | Re: [PATCH] tty: 8250: handle USR for DesignWare 8250 with correct accessors |
| From: | Alan Cox <alan@linux.intel.com> |
| Date: | Fri, 10 Jun 2011 14:57:24 +0100 |
| Cc: | Ralf Baechle <ralf@linux-mips.org>, linux-serial@vger.kernel.org, Greg Kroah-Hartman <gregkh@suse.de>, linux-mips@linux-mips.org, Marc St-Jean <bluezzer@gmail.com>, Shane McDonald <mcdonald.shane@gmail.com>, Anoop P A <anoop.pa@gmail.com> |
| In-reply-to: | <20110610075426.GM3711@pulham.picochip.com> |
| Organisation: | Intel Corporation UK Ltd, registered no. 1134945 (England), Registered office Pipers Way, Swindon, SN3 1RJ |
| Organization: | Intel |
| References: | <1307616525-22028-1-git-send-email-jamie@jamieiles.com> <20110610035817.GA6740@linux-mips.org> <20110610075426.GM3711@pulham.picochip.com> |
| Sender: | linux-mips-bounce@linux-mips.org |
> 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. Ultimately yes I'd also like to see all board specific ops banished from 8250.c |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH] mips, exec: remove redundant addr_limit assignment, Mathias Krause |
|---|---|
| Next by Date: | [PATCH] PCI: Make the struct pci_dev * argument of pci_fixup_irqs const., Ralf Baechle |
| Previous by Thread: | Re: [PATCH] tty: 8250: handle USR for DesignWare 8250 with correct accessors, Jamie Iles |
| Next by Thread: | Re: [PATCH] tty: 8250: handle USR for DesignWare 8250 with correct accessors, Jamie Iles |
| Indexes: | [Date] [Thread] [Top] [All Lists] |