On 07/17/2013 06:26 AM, Dan Carpenter wrote: > "xspi->irq" is unsigned int so it's never less than zero. I have added > a cast. Um. Doesn't this just hide a warning, not solve the problem? If the value can't be negative, so the if condition never fires, then the cast won't change that...