[patch] spi/xilinx: signedness issue checking platform_get_irq()
Dan Carpenter
dan.carpenter at oracle.com
Thu Jul 18 01:34:26 EST 2013
On Wed, Jul 17, 2013 at 09:03:53AM -0600, Stephen Warren wrote:
> 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...
No. It fixes the problem but Mark is right that it's ugly. Sorry
for that. I will resend.
regards,
dan carpenter
More information about the devicetree-discuss
mailing list