Thread (8 messages) 8 messages, 4 authors, 2012-09-21
DORMANTno replies

Re: [PATCH] serial: omap: fix the overrun case

From: Felipe Balbi <hidden>
Date: 2012-09-21 12:07:53
Also in: linux-arm-kernel, linux-omap

On Fri, Sep 21, 2012 at 05:09:56PM +0530, Poddar, Sourav wrote:
Hi Felipe,

On Fri, Sep 21, 2012 at 4:30 PM, Felipe Balbi [off-list ref] wrote:
quoted
On Fri, Sep 21, 2012 at 03:52:56PM +0530, Shubhrajyoti D wrote:
quoted
Overrun also causes an internal flag to be set, which disables further
reception. Before the next frame can
be received, the MPU must:
• Reset the RX FIFO.
• clear the internal flag.

In the uart mode a dummy read is needed. Add the same.
Very nice patch but I think commit log can be a bit more verbose.

Please make the problem a little clearer. Why do we even get that
interrupt fired if BRK_ERROR_BITS aren't set ?
According to LSR registers, there are few other bits like RX_FIFO_E( atleast 1
character in RX_FIFO or TX FIFO Empty), which might be the cause of an
interrupt. ?
right. In that case, is it really correct to just return if
BRK_ERROR_BITS aren't set ? IRQ line will not toggle and we just might
end up with an IRQ storm, right ?

-- 
balbi

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help