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 6a1cee55a49b68a09f8dbd7c8a5939e4b39f99d9..2c5e3dd58ceae193453405912f044b81210d3d73 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);