diff options
author | Corey Minyard <minyard@acm.org> | 2009-04-21 15:24:02 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-04-21 16:41:48 -0400 |
commit | 8b32b5d0dca2f5ab632e8bedcd57fe4c109c13fe (patch) | |
tree | f9a4aa93b52d4bb6a326c56974d186788e654cdd /drivers/char/ipmi/ipmi_si_intf.c | |
parent | 4614e6adafa2c5e6c3a9c245af2807fa7bc5117a (diff) |
ipmi: fix platform return check
The wrong return value is being tested when allocating a platform device
in the IPMI SI code. Check the right value.
Signed-off-by: Corey Minyard <minyard@acm.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/char/ipmi/ipmi_si_intf.c')
-rw-r--r-- | drivers/char/ipmi/ipmi_si_intf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi/ipmi_si_intf.c index e58ea4cd55ce..2438fdf889b4 100644 --- a/drivers/char/ipmi/ipmi_si_intf.c +++ b/drivers/char/ipmi/ipmi_si_intf.c | |||
@@ -2863,7 +2863,7 @@ static int try_smi_init(struct smi_info *new_smi) | |||
2863 | */ | 2863 | */ |
2864 | new_smi->pdev = platform_device_alloc("ipmi_si", | 2864 | new_smi->pdev = platform_device_alloc("ipmi_si", |
2865 | new_smi->intf_num); | 2865 | new_smi->intf_num); |
2866 | if (rv) { | 2866 | if (!new_smi->pdev) { |
2867 | printk(KERN_ERR | 2867 | printk(KERN_ERR |
2868 | "ipmi_si_intf:" | 2868 | "ipmi_si_intf:" |
2869 | " Unable to allocate platform device\n"); | 2869 | " Unable to allocate platform device\n"); |