On Fri, May 20, 2005 at 08:51:05PM +0200, Francois Romieu wrote: > /* is this our interrupt? */ > - if ( 0 == isr ) { > + if (isr) { This appears to invert the logic. Later you have: > - if( 0 != isr ) { > + if (isr) { The rest looks fine. Hamish -- Hamish Moffatt VK3SB <hamish@xxxxxxxxxx> <hamish@xxxxxxxxxxxx>