Re: [PATCH] free_irq(): Actually handle DEBUG_SHIRQ

2007-09-12 Thread David Woodhouse
On Wed, 2007-09-12 at 12:17 +0100, Maciej W. Rozycki wrote: > Code used for DEBUG_SHIRQ in free_irq() is unreachable -- the for() > loop within never terminates otherwise than by return. This is an > obvious fix. Yeah... except that it enables new debugging code which (due to my stupidity)

[PATCH] free_irq(): Actually handle DEBUG_SHIRQ

2007-09-12 Thread Maciej W. Rozycki
Code used for DEBUG_SHIRQ in free_irq() is unreachable -- the for() loop within never terminates otherwise than by return. This is an obvious fix. Signed-off-by: Maciej W. Rozycki <[EMAIL PROTECTED]> --- Please apply. While at it, I have a question about the complementary code within

[PATCH] free_irq(): Actually handle DEBUG_SHIRQ

2007-09-12 Thread Maciej W. Rozycki
Code used for DEBUG_SHIRQ in free_irq() is unreachable -- the for() loop within never terminates otherwise than by return. This is an obvious fix. Signed-off-by: Maciej W. Rozycki [EMAIL PROTECTED] --- Please apply. While at it, I have a question about the complementary code within

Re: [PATCH] free_irq(): Actually handle DEBUG_SHIRQ

2007-09-12 Thread David Woodhouse
On Wed, 2007-09-12 at 12:17 +0100, Maciej W. Rozycki wrote: Code used for DEBUG_SHIRQ in free_irq() is unreachable -- the for() loop within never terminates otherwise than by return. This is an obvious fix. Yeah... except that it enables new debugging code which (due to my stupidity) wasn't