diff options
author | Dave Jones <davej@redhat.com> | 2006-10-20 02:29:09 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-20 13:26:44 -0400 |
commit | 1cd441f99819abdd2d919ff13e8c75af58a0fd9c (patch) | |
tree | f4f7f41354305d526cffaf5718ecbeddb0fda786 /drivers | |
parent | c4ec7b0de4bc18ccb4380de638550984d9a65c25 (diff) |
[PATCH] ipmi: fix return codes in failure case
These returns should be negative, like the others in this function.
Signed-off-by: Dave Jones <davej@redhat.com>
Acked-by: Corey Minyard <minyard@acm.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/char/ipmi/ipmi_si_intf.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi/ipmi_si_intf.c index 24825bdca8f4..e5cfb1fa47d1 100644 --- a/drivers/char/ipmi/ipmi_si_intf.c +++ b/drivers/char/ipmi/ipmi_si_intf.c | |||
@@ -1789,7 +1789,7 @@ static int __devinit ipmi_pci_probe(struct pci_dev *pdev, | |||
1789 | 1789 | ||
1790 | info = kzalloc(sizeof(*info), GFP_KERNEL); | 1790 | info = kzalloc(sizeof(*info), GFP_KERNEL); |
1791 | if (!info) | 1791 | if (!info) |
1792 | return ENOMEM; | 1792 | return -ENOMEM; |
1793 | 1793 | ||
1794 | info->addr_source = "PCI"; | 1794 | info->addr_source = "PCI"; |
1795 | 1795 | ||
@@ -1810,7 +1810,7 @@ static int __devinit ipmi_pci_probe(struct pci_dev *pdev, | |||
1810 | kfree(info); | 1810 | kfree(info); |
1811 | printk(KERN_INFO "ipmi_si: %s: Unknown IPMI type: %d\n", | 1811 | printk(KERN_INFO "ipmi_si: %s: Unknown IPMI type: %d\n", |
1812 | pci_name(pdev), class_type); | 1812 | pci_name(pdev), class_type); |
1813 | return ENOMEM; | 1813 | return -ENOMEM; |
1814 | } | 1814 | } |
1815 | 1815 | ||
1816 | rv = pci_enable_device(pdev); | 1816 | rv = pci_enable_device(pdev); |