Thread (52 messages) 52 messages, 6 authors, 2019-02-14

Re: [PATCH] net: phylink: dsa: mv88e6xxx: Revise irq setup ordering

From: John David Anglin <hidden>
Date: 2019-02-04 19:52:12

Hi Andrew,

On 2019-02-04 2:35 p.m., Andrew Lunn wrote:
The change to the interrupt handler i'm pretty sure is wrong. You have
to accept with edge interrupts you are going to loose interrupts.
Can you be more specific regarding what you think is wrong with this hunk?

I can see that an interrupt might be handled twice if the source wasn't
cleared before interrupts
are re-enabled but I think that would also occur with level interrupts.

Dave

-- 
John David Anglin  dave.anglin@bell.net

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