drivers/s390/net: Drop memory allocation cast
authorJulia Lawall <julia@diku.dk>
Sun, 16 May 2010 21:15:13 +0000 (21:15 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 18 May 2010 05:42:54 +0000 (22:42 -0700)
Drop cast on the result of kmalloc and similar functions.

The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@@
type T;
@@

- (T *)
  (\(kmalloc\|kzalloc\|kcalloc\|kmem_cache_alloc\|kmem_cache_zalloc\|
   kmem_cache_alloc_node\|kmalloc_node\|kzalloc_node\)(...))
// </smpl>

Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Frank Blaschka <frank.blaschka@de.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/s390/net/ctcm_main.c
drivers/s390/net/ctcm_mpc.c
drivers/s390/net/lcs.c
drivers/s390/net/qeth_core_main.c

index e35713dd05040d6ef75e26f18833bf447469db27..4ecafbf91211b72775315a95e20a322e0e11e834 100644 (file)
@@ -1364,8 +1364,7 @@ static int add_channel(struct ccw_device *cdev, enum ctcm_channel_types type,
 
        ch->protocol = priv->protocol;
        if (IS_MPC(priv)) {
-               ch->discontact_th = (struct th_header *)
-                               kzalloc(TH_HEADER_LENGTH, gfp_type());
+               ch->discontact_th = kzalloc(TH_HEADER_LENGTH, gfp_type());
                if (ch->discontact_th == NULL)
                                        goto nomem_return;
 
@@ -1379,8 +1378,7 @@ static int add_channel(struct ccw_device *cdev, enum ctcm_channel_types type,
        } else
                ccw_num = 8;
 
-       ch->ccw = (struct ccw1 *)
-               kzalloc(ccw_num * sizeof(struct ccw1), GFP_KERNEL | GFP_DMA);
+       ch->ccw = kzalloc(ccw_num * sizeof(struct ccw1), GFP_KERNEL | GFP_DMA);
        if (ch->ccw == NULL)
                                        goto nomem_return;
 
index 5978b390153f82210a67650852643518c9fd0dae..87c24d2936d6e34d1de09da7ebe27aff148d8f61 100644 (file)
@@ -669,8 +669,7 @@ static void ctcmpc_send_sweep_resp(struct channel *rch)
                                goto done;
        }
 
-       header = (struct th_sweep *)
-                       kmalloc(sizeof(struct th_sweep), gfp_type());
+       header = kmalloc(sizeof(struct th_sweep), gfp_type());
 
        if (!header) {
                dev_kfree_skb_any(sweep_skb);
@@ -1191,8 +1190,7 @@ static void ctcmpc_unpack_skb(struct channel *ch, struct sk_buff *pskb)
                        skb_pull(pskb, new_len); /* point to next PDU */
                }
        } else {
-               mpcginfo = (struct mpcg_info *)
-                               kmalloc(sizeof(struct mpcg_info), gfp_type());
+               mpcginfo = kmalloc(sizeof(struct mpcg_info), gfp_type());
                if (mpcginfo == NULL)
                                        goto done;
 
index 9b19ea13b4d8593302620a60da62bf8c653abdef..0f19d540b655be900be7e946fd0838c77d95bb03 100644 (file)
@@ -1238,8 +1238,7 @@ lcs_set_mc_addresses(struct lcs_card *card, struct in_device *in4_dev)
                ipm = lcs_check_addr_entry(card, im4, buf);
                if (ipm != NULL)
                        continue;       /* Address already in list. */
-               ipm = (struct lcs_ipm_list *)
-                       kzalloc(sizeof(struct lcs_ipm_list), GFP_ATOMIC);
+               ipm = kzalloc(sizeof(struct lcs_ipm_list), GFP_ATOMIC);
                if (ipm == NULL) {
                        pr_info("Not enough memory to add"
                                " new multicast entry!\n");
index fd1f48c771fce241705aa0fcbaacdaa62405ddc6..e862c2d8e5c04da49e69f2c6344eee65217c1fc8 100644 (file)
@@ -606,7 +606,7 @@ static int qeth_setup_channel(struct qeth_channel *channel)
 
        QETH_DBF_TEXT(SETUP, 2, "setupch");
        for (cnt = 0; cnt < QETH_CMD_BUFFER_NO; cnt++) {
-               channel->iob[cnt].data = (char *)
+               channel->iob[cnt].data =
                        kmalloc(QETH_BUFSIZE, GFP_DMA|GFP_KERNEL);
                if (channel->iob[cnt].data == NULL)
                        break;