diff options
author | Roel Kluin <12o3l@tiscali.nl> | 2008-04-23 17:44:03 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@redhat.com> | 2008-04-29 01:56:37 -0400 |
commit | 770f867991155f9c9e36a845a142f770d55ee67c (patch) | |
tree | 9c43c3069288a9cc94ceb8fda8e75ee58b01dab9 /drivers/net/arm | |
parent | 0a0a83107c01a8dba04fa40ddb29ff021d4f8112 (diff) |
ARM: am79c961a: platform_get_irq() may return signed unnoticed
dev->irq is unsigned, platform_get_irq() may return signed unnoticed
Signed-off-by: Roel Kluin <12o3l@tiscali.nl>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'drivers/net/arm')
-rw-r--r-- | drivers/net/arm/am79c961a.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/drivers/net/arm/am79c961a.c b/drivers/net/arm/am79c961a.c index ba6bd03a015f..a637910b02dd 100644 --- a/drivers/net/arm/am79c961a.c +++ b/drivers/net/arm/am79c961a.c | |||
@@ -693,11 +693,15 @@ static int __init am79c961_probe(struct platform_device *pdev) | |||
693 | * done by the ether bootp loader. | 693 | * done by the ether bootp loader. |
694 | */ | 694 | */ |
695 | dev->base_addr = res->start; | 695 | dev->base_addr = res->start; |
696 | dev->irq = platform_get_irq(pdev, 0); | 696 | ret = platform_get_irq(pdev, 0); |
697 | 697 | ||
698 | ret = -ENODEV; | 698 | if (ret < 0) { |
699 | if (dev->irq < 0) | 699 | ret = -ENODEV; |
700 | goto nodev; | 700 | goto nodev; |
701 | } | ||
702 | dev->irq = ret; | ||
703 | |||
704 | ret = -ENODEV; | ||
701 | if (!request_region(dev->base_addr, 0x18, dev->name)) | 705 | if (!request_region(dev->base_addr, 0x18, dev->name)) |
702 | goto nodev; | 706 | goto nodev; |
703 | 707 | ||