Staging: gdm724x: Remove unnecessary print statements
authorBhanusree Pola <bhanusreemahesh@gmail.com>
Fri, 22 Mar 2019 12:40:10 +0000 (18:10 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 22 Mar 2019 14:16:28 +0000 (15:16 +0100)
Remove print statements that provide information about error messages
when memory allocation is failed.
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>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/gdm724x/gdm_lte.c

index 3c2aab7a921e89157f2929adf8be1241835d558b..db11498f6fc7d99342c22eb5cc41504da6a34842 100644 (file)
@@ -871,7 +871,6 @@ int register_lte_device(struct phy_dev *phy_dev,
                net = alloc_netdev(sizeof(struct nic), pdn_dev_name,
                                   NET_NAME_UNKNOWN, ether_setup);
                if (!net) {
-                       pr_err("alloc_netdev failed\n");
                        ret = -ENOMEM;
                        goto err;
                }