gpio: mlxbf2: fix return value check in mlxbf2_gpio_get_lock_res()

[ Upstream commit 66d8ad67aab3bc6f55e7de81565cd0d4875bd851 ]

In case of error, the function devm_ioremap() returns NULL pointer not
ERR_PTR(). The IS_ERR() test in the return value check should be
replaced with NULL test.

Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Link: https://lore.kernel.org/r/20200427110829.154785-1-weiyongjun1@huawei.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>

diff --git a/drivers/gpio/gpio-mlxbf2.c b/drivers/gpio/gpio-mlxbf2.c
index da570e63589d09896a6bfb9d7c24811604b0a5d8..cc0dd8593a4badfc66a392f68b38ed90c9b65bc6 100644 (file)
--- a/drivers/gpio/gpio-mlxbf2.c
+++ b/drivers/gpio/gpio-mlxbf2.c
@@ -110,8 +110,8 @@ static int mlxbf2_gpio_get_lock_res(struct platform_device *pdev)
        }
 
        yu_arm_gpio_lock_param.io = devm_ioremap(dev, res->start, size);
-       if (IS_ERR(yu_arm_gpio_lock_param.io))
-               ret = PTR_ERR(yu_arm_gpio_lock_param.io);
+       if (!yu_arm_gpio_lock_param.io)
+               ret = -ENOMEM;
 
 exit:
        mutex_unlock(yu_arm_gpio_lock_param.lock);