staging: media: lirc: Remove useless cast on void pointer
authorTapasweni Pathak <tapaswenipathak@gmail.com>
Thu, 30 Oct 2014 11:32:48 +0000 (17:02 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 30 Oct 2014 20:05:48 +0000 (13:05 -0700)
void pointers do not need to be cast to other pointer types.

The semantic patch used to find this:

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

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

Build tested it.

Signed-off-by: Tapasweni Pathak <tapaswenipathak@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/media/lirc/lirc_igorplugusb.c
drivers/staging/media/lirc/lirc_imon.c
drivers/staging/media/lirc/lirc_sasem.c

index 431d1e8..fd5d3d2 100644 (file)
@@ -282,7 +282,7 @@ static void send_fragment(struct igorplug *ir, struct lirc_buffer *buf,
 static int igorplugusb_remote_poll(void *data, struct lirc_buffer *buf)
 {
        int ret;
-       struct igorplug *ir = (struct igorplug *)data;
+       struct igorplug *ir = data;
 
        if (!ir || !ir->usbdev)  /* Has the device been removed? */
                return -ENODEV;
index e3b1fa9..f98418c 100644 (file)
@@ -495,7 +495,7 @@ static int ir_open(void *data)
        /* prevent races with disconnect */
        mutex_lock(&driver_lock);
 
-       context = (struct imon_context *)data;
+       context = data;
 
        /* initial IR protocol decode variables */
        context->rx.count = 0;
@@ -516,7 +516,7 @@ static void ir_close(void *data)
 {
        struct imon_context *context;
 
-       context = (struct imon_context *)data;
+       context = data;
        if (!context) {
                pr_err("%s: no context for device\n", __func__);
                return;
index 05969fb..123ddf6 100644 (file)
@@ -488,7 +488,7 @@ static int ir_open(void *data)
        /* prevent races with disconnect */
        mutex_lock(&disconnect_lock);
 
-       context = (struct sasem_context *) data;
+       context = data;
 
        mutex_lock(&context->ctx_lock);
 
@@ -530,7 +530,7 @@ static void ir_close(void *data)
 {
        struct sasem_context *context;
 
-       context = (struct sasem_context *)data;
+       context = data;
        if (!context) {
                pr_err("%s: no context for device\n", __func__);
                return;