OSDN Git Service

staging: ccree: remove unnecessary cast on kmalloc
authorGustavo A. R. Silva <garsilva@embeddedor.com>
Sun, 9 Jul 2017 05:43:37 +0000 (00:43 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 16 Jul 2017 06:40:57 +0000 (08:40 +0200)
The assignment operator implicitly converts a void pointer to the type of the
pointer it is assigned to.

This issue was detected using Coccinelle and the following semantic patch:

@@
expression * e;
expression arg1, arg2;
type T;
@@

- e=(T*)
+ e=
kmalloc(arg1, arg2);

Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Acked-by: Gilad Ben-Yossef <gilad@benyossef.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ccree/ssi_buffer_mgr.c

index b35871e..18a8694 100644 (file)
@@ -1725,8 +1725,7 @@ int ssi_buffer_mgr_init(struct ssi_drvdata *drvdata)
        struct buff_mgr_handle *buff_mgr_handle;
        struct device *dev = &drvdata->plat_dev->dev;
 
-       buff_mgr_handle = (struct buff_mgr_handle *)
-               kmalloc(sizeof(struct buff_mgr_handle), GFP_KERNEL);
+       buff_mgr_handle = kmalloc(sizeof(struct buff_mgr_handle), GFP_KERNEL);
        if (!buff_mgr_handle)
                return -ENOMEM;