diff options
author | Roel Kluin <12o3l@tiscali.nl> | 2008-07-24 00:29:53 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-07-24 13:47:30 -0400 |
commit | 166a375b657b7af494f4ce3f72c4d2002180da44 (patch) | |
tree | 02a3c51a478535b582b4d38833755fc3cfa82d3e /drivers/spi | |
parent | a61f5345eba34772a71523227de890a28410f320 (diff) |
xilinx_spi: test below 0 on unsigned irq in xilinx_spi_probe()
xilinx_spi->irq is unsigned, so the test fails
Signed-off-by: Roel Kluin <12o3l@tiscali.nl>
Cc: David Brownell <dbrownell@users.sourceforge.net>
Cc: Andrei Konovalov <akonovalov@ru.mvista.com>
Cc: Yuri Frolov <yfrolov@ru.mvista.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/spi')
-rw-r--r-- | drivers/spi/xilinx_spi.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/spi/xilinx_spi.c b/drivers/spi/xilinx_spi.c index 113a0468ffcb..68d6f4988fb5 100644 --- a/drivers/spi/xilinx_spi.c +++ b/drivers/spi/xilinx_spi.c | |||
@@ -353,11 +353,12 @@ static int __init xilinx_spi_probe(struct platform_device *dev) | |||
353 | goto put_master; | 353 | goto put_master; |
354 | } | 354 | } |
355 | 355 | ||
356 | xspi->irq = platform_get_irq(dev, 0); | 356 | ret = platform_get_irq(dev, 0); |
357 | if (xspi->irq < 0) { | 357 | if (ret < 0) { |
358 | ret = -ENXIO; | 358 | ret = -ENXIO; |
359 | goto unmap_io; | 359 | goto unmap_io; |
360 | } | 360 | } |
361 | xspi->irq = ret; | ||
361 | 362 | ||
362 | master->bus_num = pdata->bus_num; | 363 | master->bus_num = pdata->bus_num; |
363 | master->num_chipselect = pdata->num_chipselect; | 364 | master->num_chipselect = pdata->num_chipselect; |