diff options
author | Joel Stanley <joel@jms.id.au> | 2016-09-21 19:35:53 +0930 |
---|---|---|
committer | Corey Minyard <cminyard@mvista.com> | 2016-09-29 19:05:06 -0500 |
commit | 1a377a79211a08c5c8a05c0b6dee6d5b13ef4107 (patch) | |
tree | d16340a09b16def477699549642afb50181701fc /drivers/char | |
parent | 54f9c4d0778b3f9ab791b1b7eb1a5d2809f02f50 (diff) | |
download | op-kernel-dev-1a377a79211a08c5c8a05c0b6dee6d5b13ef4107.zip op-kernel-dev-1a377a79211a08c5c8a05c0b6dee6d5b13ef4107.tar.gz |
ipmi: Fix ioremap error handling in bt-bmc
devm_ioremap_resource returns ERR_PTR so we can't check for NULL.
Signed-off-by: Joel Stanley <joel@jms.id.au>
Acked-by: Cédric Le Goater <clg@kaod.org>
Signed-off-by: Corey Minyard <cminyard@mvista.com>
Diffstat (limited to 'drivers/char')
-rw-r--r-- | drivers/char/ipmi/bt-bmc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/char/ipmi/bt-bmc.c b/drivers/char/ipmi/bt-bmc.c index 2e880bf..de64bf1 100644 --- a/drivers/char/ipmi/bt-bmc.c +++ b/drivers/char/ipmi/bt-bmc.c @@ -438,8 +438,8 @@ static int bt_bmc_probe(struct platform_device *pdev) } bt_bmc->base = devm_ioremap_resource(&pdev->dev, res); - if (!bt_bmc->base) - return -ENOMEM; + if (IS_ERR(bt_bmc->base)) + return PTR_ERR(bt_bmc->base); mutex_init(&bt_bmc->mutex); init_waitqueue_head(&bt_bmc->queue); |