> Shouldn't this be> > > - if (client->irq <= NO_IRQ)> > instead ofNO_IRQ is obsolete. client->irq != is the test and IRQ numbers areunsigned.Alan