Re: [PATCH] net: phylink: dsa: mv88e6xxx: Revise irq setup ordering
From: John David Anglin <hidden>
Date: 2019-02-04 19:52:12
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