Staging: most: Remove unnecessary cast on void pointer
authorShraddha Barke <shraddha.6596@gmail.com>
Wed, 14 Oct 2015 19:28:19 +0000 (00:58 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 17 Oct 2015 05:51:48 +0000 (22:51 -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
)

Signed-off-by: Shraddha Barke <shraddha.6596@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/most/hdm-dim2/dim2_hdm.c

index c7e956f5fec0e31c5abe93bf9b618474e04b9c19..38876316d8692ccfa8d62cd7aef95da2bf2d6566 100644 (file)
@@ -279,7 +279,7 @@ static int try_start_dim_transfer(struct hdm_channel *hdm_ch)
  */
 static int deliver_netinfo_thread(void *data)
 {
-       struct dim2_hdm *dev = (struct dim2_hdm *)data;
+       struct dim2_hdm *dev = data;
 
        while (!kthread_should_stop()) {
                wait_event_interruptible(dev->netinfo_waitq,
@@ -450,7 +450,7 @@ static void dim2_tasklet_fn(unsigned long data)
  */
 static irqreturn_t dim2_ahb_isr(int irq, void *_dev)
 {
-       struct dim2_hdm *dev = (struct dim2_hdm *)_dev;
+       struct dim2_hdm *dev = _dev;
        struct dim_channel *buffer[DMA_CHANNELS + 1];
        unsigned long flags;