Re: [PATCH] 3c59x: Fix shared IRQ handling

2015-07-08 Thread David Miller
From: Denys Vlasenko Date: Tue, 7 Jul 2015 20:48:55 +0200 > As its first order of business, boomerang_interrupt() checks whether > the device really has any pending interrupts. If it does not, > it does nothing and returns, but it still returns IRQ_HANDLED. > > This is wrong: interrupt was not

[PATCH] 3c59x: Fix shared IRQ handling

2015-07-07 Thread Denys Vlasenko
As its first order of business, boomerang_interrupt() checks whether the device really has any pending interrupts. If it does not, it does nothing and returns, but it still returns IRQ_HANDLED. This is wrong: interrupt was not handled, IRQ handlers of other devices sharing this IRQ line need to be