Staging: lustre: obdclass: Remove unnecessary cast on void pointer
authorShraddha Barke <shraddha.6596@gmail.com>
Wed, 14 Oct 2015 19:28:21 +0000 (00:58 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 17 Oct 2015 05:26:35 +0000 (22:26 -0700)
void pointers do not need to be cast to other pointer types.

Semantic patch used to find this:

@r@
expression x;
void* e;
type T;
identifier f;
@@

(
  *((T *)e)
|
  ((T *)x)[...]
|
  ((T *)x)->f
|
- (T *)
  e
)

Signed-off-by: Shraddha Barke <shraddha.6596@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lustre/obdclass/lprocfs_status.c
drivers/staging/lustre/lustre/obdclass/obd_config.c
drivers/staging/lustre/lustre/obdclass/obd_mount.c

index 7cc4b554af1db3cd0d93b45cadc9ba86d0b51e71..3228feed4747390629e4ae532469e6063c0c80fc 100644 (file)
@@ -612,7 +612,7 @@ int lprocfs_rd_import(struct seq_file *m, void *data)
 {
        struct lprocfs_counter          ret;
        struct lprocfs_counter_header   *header;
-       struct obd_device               *obd    = (struct obd_device *)data;
+       struct obd_device               *obd    = data;
        struct obd_import               *imp;
        struct obd_import_conn          *conn;
        int                             j;
@@ -757,7 +757,7 @@ EXPORT_SYMBOL(lprocfs_rd_import);
 
 int lprocfs_rd_state(struct seq_file *m, void *data)
 {
-       struct obd_device *obd = (struct obd_device *)data;
+       struct obd_device *obd = data;
        struct obd_import *imp;
        int j, k;
 
@@ -797,7 +797,7 @@ EXPORT_SYMBOL(lprocfs_at_hist_helper);
 /* See also ptlrpc_lprocfs_rd_timeouts */
 int lprocfs_rd_timeouts(struct seq_file *m, void *data)
 {
-       struct obd_device *obd = (struct obd_device *)data;
+       struct obd_device *obd = data;
        struct obd_import *imp;
        unsigned int cur, worst;
        time64_t now, worstt;
index 64e2d1b368be6bb7b15fb3b1774e62b2c71a71ef..38e5e13c7fded2236593db600a06806ed1bc2725 100644 (file)
@@ -88,7 +88,7 @@ static int class_match_param(char *buf, char *key, char **valp)
 
 static int parse_nid(char *buf, void *value, int quiet)
 {
-       lnet_nid_t *nid = (lnet_nid_t *)value;
+       lnet_nid_t *nid = value;
 
        *nid = libcfs_str2nid(buf);
        if (*nid != LNET_NID_ANY)
@@ -101,7 +101,7 @@ static int parse_nid(char *buf, void *value, int quiet)
 
 static int parse_net(char *buf, void *value)
 {
-       __u32 *net = (__u32 *)value;
+       __u32 *net = value;
 
        *net = libcfs_str2net(buf);
        CDEBUG(D_INFO, "Net %s\n", libcfs_net2str(*net));
index 0d324cc2824369b3b5ea13e1d91ec9ae260d1bf7..149c838135835161d0475a0fe1c64915203263d8 100644 (file)
@@ -1100,7 +1100,7 @@ static int lustre_fill_super(struct super_block *sb, void *data, int silent)
        obd_zombie_barrier();
 
        /* Figure out the lmd from the mount options */
-       if (lmd_parse((char *)(lmd2->lmd2_data), lmd)) {
+       if (lmd_parse((lmd2->lmd2_data), lmd)) {
                lustre_put_lsi(sb);
                rc = -EINVAL;
                goto out;