Staging: media: imx: mipi-csi2: Remove unnecessary error messages
authorBhanusree Pola <bhanusreemahesh@gmail.com>
Sat, 23 Mar 2019 07:51:29 +0000 (13:21 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 25 Mar 2019 20:44:53 +0000 (05:44 +0900)
The function devm_ioremap already prints error message  when there
is no enough memory.
Remove print statements that give error messages
Issue found using coccinelle
The following semantic patch is used to solve this:

<smpl>
@@
expression x;
constant char[] C;
identifier f;
@@

x = (\(kmalloc\|devm_kzalloc\|kmalloc_array\|devm_ioremap\|
usb_alloc_urb\|alloc_netdev\|dev_alloc_skb\)(...));

if(x==NULL)
{
...
(
-f(C,...);
|
-f(...,C);
)
...
}
</smpl>

Signed-off-by: Bhanusree Pola <bhanusreemahesh@gmail.com>
Reviewed-by: Fabio Estevam <festevam@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/media/imx/imx6-mipi-csi2.c

index 6a1cee5..2c5e3dd 100644 (file)
@@ -628,10 +628,8 @@ static int csi2_probe(struct platform_device *pdev)
        }
 
        csi2->base = devm_ioremap(&pdev->dev, res->start, PAGE_SIZE);
-       if (!csi2->base) {
-               v4l2_err(&csi2->sd, "failed to map CSI-2 registers\n");
+       if (!csi2->base)
                return -ENOMEM;
-       }
 
        mutex_init(&csi2->lock);