diff options
author | Dave Jones <davej@redhat.com> | 2006-10-19 23:29:09 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-20 10:26:44 -0700 |
commit | 1cd441f99819abdd2d919ff13e8c75af58a0fd9c (patch) | |
tree | f4f7f41354305d526cffaf5718ecbeddb0fda786 /drivers | |
parent | c4ec7b0de4bc18ccb4380de638550984d9a65c25 (diff) | |
download | op-kernel-dev-1cd441f99819abdd2d919ff13e8c75af58a0fd9c.zip op-kernel-dev-1cd441f99819abdd2d919ff13e8c75af58a0fd9c.tar.gz |
[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 24825bd..e5cfb1f 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, info = kzalloc(sizeof(*info), GFP_KERNEL); if (!info) - return ENOMEM; + return -ENOMEM; info->addr_source = "PCI"; @@ -1810,7 +1810,7 @@ static int __devinit ipmi_pci_probe(struct pci_dev *pdev, kfree(info); printk(KERN_INFO "ipmi_si: %s: Unknown IPMI type: %d\n", pci_name(pdev), class_type); - return ENOMEM; + return -ENOMEM; } rv = pci_enable_device(pdev); |