staging: tidspbridge: dont cast void* from kmalloc()

PATCH New Thomas Meyer 1 participant 1 message Aug 13, 2011 Staging

lore

Share & tools

lore

Thomas Meyer

author

2011-08-13 14:11 UTC #0 lore raw

PATCH 1/1 staging: tidspbridge: dont cast void* from kmalloc()

From: Thomas Meyer <[email protected]>

Casting (void *) value returned by kmalloc is useless as mentioned in Documentation/CodingStyle, Chap 14.

The semantic patch that makes this output is available in scripts/coccinelle/api/alloc/drop_kmalloc_cast.cocci.

More information about semantic patching is available at
http://coccinelle.lip6.fr/

Signed-off-by Thomas Meyer <[email protected]>

---

drivers/staging/tidspbridge/gen/gh.c 2011-08-01 19:58:07.230403689 +0200 renamed from drivers/staging/tidspbridge/gen/gh.c 2011-04-14 10:34:38.600880635 +0200 +1 -1

diff -u -p a/drivers/staging/tidspbridge/gen/gh.c b/drivers/staging/tidspbridge/gen/gh.c
@@ -55,7 +55,7 @@ struct gh_t_hash_tab *gh_create(u16 max_
 	hash_tab->match = match;
 	hash_tab->delete = delete == NULL ? noop : delete;
 
-	hash_tab->buckets = (struct element **)
+	hash_tab->buckets =
 	    kzalloc(sizeof(struct element *) * max_bucket, GFP_KERNEL);
 	if (hash_tab->buckets == NULL) {
 		gh_delete(hash_tab);