counter: ti-ecap-capture: fix IS_ERR() vs NULL check
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 12 Oct 2022 14:51:25 +0000 (17:51 +0300)
committerWilliam Breathitt Gray <william.gray@linaro.org>
Mon, 17 Oct 2022 13:51:29 +0000 (09:51 -0400)
The devm_counter_alloc() function returns NULL on error.  It doesn't
return error pointers.

Fixes: 4e2f42aa00b6 ("counter: ti-ecap-capture: capture driver support for ECAP")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Julien Panis <jpanis@baylibre.com>
Acked-by: Vignesh Raghavendra <vigneshr@ti.com>
Link: https://lore.kernel.org/r/Y0bUbZvfDJHBG9C6@kili/
Signed-off-by: William Breathitt Gray <william.gray@linaro.org>
drivers/counter/ti-ecap-capture.c

index b8dbf02..fb1cb17 100644 (file)
@@ -480,8 +480,8 @@ static int ecap_cnt_probe(struct platform_device *pdev)
        int ret;
 
        counter_dev = devm_counter_alloc(dev, sizeof(*ecap_dev));
-       if (IS_ERR(counter_dev))
-               return PTR_ERR(counter_dev);
+       if (!counter_dev)
+               return -ENOMEM;
 
        counter_dev->name = ECAP_DRV_NAME;
        counter_dev->parent = dev;