drm/tve200: Check for IS_ERR instead of NULL in probe
authorDan Carpenter <dan.carpenter@oracle.com>
Mon, 25 Sep 2017 10:25:20 +0000 (13:25 +0300)
committerLinus Walleij <linus.walleij@linaro.org>
Sat, 30 Sep 2017 23:29:54 +0000 (01:29 +0200)
devm_ioremap_resource() returns error pointer, it never returns NULL on
error.

Fixes: 179c02fe90a4 ("drm/tve200: Add new driver for TVE200")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20170925102520.a7spymwqqbsczzz2@mwanda
drivers/gpu/drm/tve200/tve200_drv.c

index eae38b669f0a3a0eb101fd0ebca221a2395918db..6939f7455a2d1b8b2e367b86df366ab5f152eb2a 100644 (file)
@@ -225,7 +225,7 @@ static int tve200_probe(struct platform_device *pdev)
 
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        priv->regs = devm_ioremap_resource(dev, res);
-       if (!priv->regs) {
+       if (IS_ERR(priv->regs)) {
                dev_err(dev, "%s failed mmio\n", __func__);
                ret = -EINVAL;
                goto clk_disable;