Remove spurious void pointer casts.

Signed-off-by: Pekka Enberg <[EMAIL PROTECTED]>
---

 cifsfs.c    |    9 ++++-----
 dir.c       |    6 ++----
 file.c      |   32 +++++++++++++-------------------
 misc.c      |   17 ++++-------------
 readdir.c   |    6 +++---
 transport.c |    5 ++---
 6 files changed, 28 insertions(+), 47 deletions(-)

Index: linux/fs/cifs/cifsfs.c
===================================================================
--- linux.orig/fs/cifs/cifsfs.c 2005-01-12 20:13:39.874862088 +0200
+++ linux/fs/cifs/cifsfs.c      2005-01-12 20:13:47.453709928 +0200
@@ -221,10 +221,9 @@
 static struct inode *
 cifs_alloc_inode(struct super_block *sb)
 {
-       struct cifsInodeInfo *cifs_inode;
-       cifs_inode =
-           (struct cifsInodeInfo *) kmem_cache_alloc(cifs_inode_cachep,
-                                                     SLAB_KERNEL);
+       struct cifsInodeInfo *cifs_inode =
+               kmem_cache_alloc(cifs_inode_cachep, SLAB_KERNEL);
+
        if (!cifs_inode)
                return NULL;
        cifs_inode->cifsAttrs = 0x20;   /* default */
@@ -578,7 +577,7 @@
 static void
 cifs_init_once(void *inode, kmem_cache_t * cachep, unsigned long flags)
 {
-       struct cifsInodeInfo *cifsi = (struct cifsInodeInfo *) inode;
+       struct cifsInodeInfo *cifsi = inode;
 
        if ((flags & (SLAB_CTOR_VERIFY | SLAB_CTOR_CONSTRUCTOR)) ==
            SLAB_CTOR_CONSTRUCTOR) {
Index: linux/fs/cifs/dir.c
===================================================================
--- linux.orig/fs/cifs/dir.c    2005-01-12 20:13:40.114825608 +0200
+++ linux/fs/cifs/dir.c 2005-01-12 20:13:47.454709776 +0200
@@ -284,12 +284,10 @@
                        /* mknod case - do not leave file open */
                        CIFSSMBClose(xid, pTcon, fileHandle);
                } else if(newinode) {
-                       pCifsFile = (struct cifsFileInfo *)
-                          kmalloc(sizeof (struct cifsFileInfo), GFP_KERNEL);
+                       pCifsFile = kmalloc(sizeof (struct cifsFileInfo), 
GFP_KERNEL);
                
                        if (pCifsFile) {
-                               memset((char *)pCifsFile, 0,
-                                      sizeof (struct cifsFileInfo));
+                               memset(pCifsFile, 0, sizeof (struct 
cifsFileInfo));
                                pCifsFile->netfid = fileHandle;
                                pCifsFile->pid = current->tgid;
                                pCifsFile->pInode = newinode;
Index: linux/fs/cifs/file.c
===================================================================
--- linux.orig/fs/cifs/file.c   2005-01-12 20:13:40.119824848 +0200
+++ linux/fs/cifs/file.c        2005-01-12 20:13:47.457709320 +0200
@@ -166,7 +166,7 @@
                        kmalloc(sizeof (struct cifsFileInfo), GFP_KERNEL);
                if (file->private_data) {
                        memset(file->private_data, 0, sizeof(struct 
cifsFileInfo));
-                       pCifsFile = (struct cifsFileInfo *) file->private_data;
+                       pCifsFile = file->private_data;
                        pCifsFile->netfid = netfid;
                        pCifsFile->pid = current->tgid;
                        init_MUTEX(&pCifsFile->fh_sem);
@@ -285,7 +285,7 @@
        if(inode == NULL)
                return -EBADF;
        if (file->private_data) {
-               pCifsFile = (struct cifsFileInfo *) file->private_data;
+               pCifsFile = file->private_data;
        } else
                return -EBADF;
 
@@ -399,8 +399,7 @@
        int xid;
        struct cifs_sb_info *cifs_sb;
        struct cifsTconInfo *pTcon;
-       struct cifsFileInfo *pSMBFile =
-               (struct cifsFileInfo *) file->private_data;
+       struct cifsFileInfo *pSMBFile = file->private_data;
 
        xid = GetXid();
 
@@ -446,8 +445,7 @@
 {
        int rc = 0;
        int xid;
-       struct cifsFileInfo *pCFileStruct =
-           (struct cifsFileInfo *) file->private_data;
+       struct cifsFileInfo *pCFileStruct = file->private_data;
        char * ptmp;
 
        cFYI(1, ("Closedir inode = 0x%p with ", inode));
@@ -618,7 +616,7 @@
        if (file->private_data == NULL) {
                return -EBADF;
        } else {
-               open_file = (struct cifsFileInfo *) file->private_data;
+               open_file = file->private_data;
        }
        
        xid = GetXid();
@@ -734,7 +732,7 @@
        if (file->private_data == NULL) {
                return -EBADF;
        } else {
-               open_file = (struct cifsFileInfo *) file->private_data;
+               open_file = file->private_data;
        }
        
        xid = GetXid();
@@ -1057,7 +1055,7 @@
                FreeXid(xid);
                return -EBADF;
        }
-       open_file = (struct cifsFileInfo *)file->private_data;
+       open_file = file->private_data;
 
        if((file->f_flags & O_ACCMODE) == O_WRONLY) {
                cFYI(1,("attempting read on write only file instance"));
@@ -1136,7 +1134,7 @@
                FreeXid(xid);
                return -EBADF;
        }
-       open_file = (struct cifsFileInfo *)file->private_data;
+       open_file = file->private_data;
 
        if((file->f_flags & O_ACCMODE) == O_WRONLY) {
                cFYI(1,("attempting read on write only file instance"));
@@ -1274,7 +1272,7 @@
                FreeXid(xid);
                return -EBADF;
        }
-       open_file = (struct cifsFileInfo *)file->private_data;
+       open_file = file->private_data;
        cifs_sb = CIFS_SB(file->f_dentry->d_sb);
        pTcon = cifs_sb->tcon;
 
@@ -1712,9 +1710,7 @@
 static void reset_resume_key(struct file * dir_file, 
                                unsigned char * filename, 
                                unsigned int len,int Unicode,struct nls_table * 
nls_tab) {
-       struct cifsFileInfo *cifsFile;
-
-       cifsFile = (struct cifsFileInfo *)dir_file->private_data;
+       struct cifsFileInfo *cifsFile = dir_file->private_data;
        if(cifsFile == NULL)
                return;
        if(cifsFile->search_resume_name) {
@@ -1897,8 +1893,7 @@
                /* fallthrough */
        case 2:
                if (file->private_data != NULL) {
-                       cifsFile =
-                               (struct cifsFileInfo *) file->private_data;
+                       cifsFile = file->private_data;
                        if (cifsFile->srch_inf.endOfSearch) {
                                if(cifsFile->srch_inf.emptyDir) {
                                        cFYI(1, ("End of search, empty dir"));
@@ -1930,8 +1925,7 @@
                        if (file->private_data) {
                                memset(file->private_data, 0,
                                       sizeof (struct cifsFileInfo));
-                               cifsFile =
-                                   (struct cifsFileInfo *) file->private_data;
+                               cifsFile = file->private_data;
                                cifsFile->netfid = searchHandle;
                                cifsFile->invalidHandle = FALSE;
                                init_MUTEX(&cifsFile->fh_sem);
@@ -2104,7 +2098,7 @@
                             ("Readdir on closed srch, pos = %lld",
                              file->f_pos));
                } else {
-                       cifsFile = (struct cifsFileInfo *) file->private_data;
+                       cifsFile = file->private_data;
                        if (cifsFile->srch_inf.endOfSearch) {
                                rc = 0;
                                cFYI(1, ("End of search "));
Index: linux/fs/cifs/misc.c
===================================================================
--- linux.orig/fs/cifs/misc.c   2005-01-12 20:13:40.124824088 +0200
+++ linux/fs/cifs/misc.c        2005-01-12 20:13:47.459709016 +0200
@@ -66,11 +66,7 @@
 struct cifsSesInfo *
 sesInfoAlloc(void)
 {
-       struct cifsSesInfo *ret_buf;
-
-       ret_buf =
-           (struct cifsSesInfo *) kmalloc(sizeof (struct cifsSesInfo),
-                                          GFP_KERNEL);
+       struct cifsSesInfo *ret_buf = kmalloc(sizeof(*ret_buf), GFP_KERNEL);
        if (ret_buf) {
                memset(ret_buf, 0, sizeof (struct cifsSesInfo));
                write_lock(&GlobalSMBSeslock);
@@ -109,10 +105,7 @@
 struct cifsTconInfo *
 tconInfoAlloc(void)
 {
-       struct cifsTconInfo *ret_buf;
-       ret_buf =
-           (struct cifsTconInfo *) kmalloc(sizeof (struct cifsTconInfo),
-                                           GFP_KERNEL);
+       struct cifsTconInfo *ret_buf = kmalloc(sizeof (*ret_buf), GFP_KERNEL);
        if (ret_buf) {
                memset(ret_buf, 0, sizeof (struct cifsTconInfo));
                write_lock(&GlobalSMBSeslock);
@@ -155,8 +148,7 @@
    but it may be more efficient to always alloc same size 
    albeit slightly larger than necessary and maxbuffersize 
    defaults to this and can not be bigger */
-       ret_buf =
-           (struct smb_hdr *) mempool_alloc(cifs_req_poolp, SLAB_KERNEL | 
SLAB_NOFS);
+       ret_buf = mempool_alloc(cifs_req_poolp, SLAB_KERNEL | SLAB_NOFS);
 
        /* clear the first few header bytes */
        /* for most paths, more is cleared in header_assemble */
@@ -189,8 +181,7 @@
    but it may be more efficient to always alloc same size 
    albeit slightly larger than necessary and maxbuffersize 
    defaults to this and can not be bigger */
-       ret_buf =
-           (struct smb_hdr *) mempool_alloc(cifs_sm_req_poolp, SLAB_KERNEL | 
SLAB_NOFS);
+       ret_buf = mempool_alloc(cifs_sm_req_poolp, SLAB_KERNEL | SLAB_NOFS);
        if (ret_buf) {
        /* No need to clear memory here, cleared in header assemble */
                atomic_inc(&smBufAllocCount);
Index: linux/fs/cifs/readdir.c
===================================================================
--- linux.orig/fs/cifs/readdir.c        2005-01-12 20:13:40.126823784 +0200
+++ linux/fs/cifs/readdir.c     2005-01-12 20:13:47.460708864 +0200
@@ -65,7 +65,7 @@
        } else {
                memset(file->private_data,0,sizeof(struct cifsFileInfo));
        }
-       cifsFile = (struct cifsFileInfo *)file->private_data;
+       cifsFile = file->private_data;
        cifsFile->invalidHandle = TRUE;
        cifsFile->srch_inf.endOfSearch = FALSE;
 
@@ -221,7 +221,7 @@
        int pos_in_buf = 0;
        loff_t first_entry_in_buffer;
        loff_t index_to_find = file->f_pos;
-       struct cifsFileInfo * cifsFile = (struct cifsFileInfo 
*)file->private_data;
+       struct cifsFileInfo * cifsFile = file->private_data;
        /* check if index in the buffer */
        
        if((cifsFile == NULL) || (ppCurrentEntry == NULL) || (num_to_ret == 
NULL))
@@ -534,7 +534,7 @@
                        FreeXid(xid);
                        return rc;
                }
-               cifsFile = (struct cifsFileInfo *) file->private_data;
+               cifsFile = file->private_data;
                if (cifsFile->srch_inf.endOfSearch) {
                        if(cifsFile->srch_inf.emptyDir) {
                                cFYI(1, ("End of search, empty dir"));
Index: linux/fs/cifs/transport.c
===================================================================
--- linux.orig/fs/cifs/transport.c      2005-01-12 20:13:40.127823632 +0200
+++ linux/fs/cifs/transport.c   2005-01-12 20:13:47.462708560 +0200
@@ -48,7 +48,7 @@
                return NULL;
        }
        
-       temp = (struct mid_q_entry *) mempool_alloc(cifs_mid_poolp,SLAB_KERNEL 
| SLAB_NOFS);
+       temp = mempool_alloc(cifs_mid_poolp,SLAB_KERNEL | SLAB_NOFS);
        if (temp == NULL)
                return temp;
        else {
@@ -90,8 +90,7 @@
                cERROR(1, ("Null parms passed to AllocOplockQEntry"));
                return NULL;
        }
-       temp = (struct oplock_q_entry *) kmem_cache_alloc(cifs_oplock_cachep,
-                                                      SLAB_KERNEL);
+       temp = kmem_cache_alloc(cifs_oplock_cachep, SLAB_KERNEL);
        if (temp == NULL)
                return temp;
        else {


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to