staging: ozwpan: Remove useless cast on void pointer
authorTapasweni Pathak <tapaswenipathak@gmail.com>
Thu, 30 Oct 2014 11:32:57 +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/ozwpan/ozhcd.c
drivers/staging/ozwpan/ozusbsvc1.c

index 3d03e9255831c0d77cb9b57ba6d5abbe3d038d7f..27217d75f155bfa2a6960cd80c7a1d1f669c4f40 100644 (file)
@@ -730,7 +730,7 @@ void oz_hcd_pd_reset(void *hpd, void *hport)
 {
        /* Cleanup the current configuration and report reset to the core.
         */
-       struct oz_port *port = (struct oz_port *)hport;
+       struct oz_port *port = hport;
        struct oz_hcd *ozhcd = port->ozhcd;
 
        oz_dbg(ON, "PD Reset\n");
@@ -749,7 +749,7 @@ void oz_hcd_pd_reset(void *hpd, void *hport)
 void oz_hcd_get_desc_cnf(void *hport, u8 req_id, int status, const u8 *desc,
                        int length, int offset, int total_size)
 {
-       struct oz_port *port = (struct oz_port *)hport;
+       struct oz_port *port = hport;
        struct urb *urb;
        int err = 0;
 
@@ -889,7 +889,7 @@ static void oz_hcd_complete_set_interface(struct oz_port *port, struct urb *urb,
 void oz_hcd_control_cnf(void *hport, u8 req_id, u8 rcode, const u8 *data,
        int data_len)
 {
-       struct oz_port *port = (struct oz_port *)hport;
+       struct oz_port *port = hport;
        struct urb *urb;
        struct usb_ctrlrequest *setup;
        struct usb_hcd *hcd = port->ozhcd->hcd;
@@ -1036,7 +1036,7 @@ static inline int oz_usb_get_frame_number(void)
 int oz_hcd_heartbeat(void *hport)
 {
        int rc = 0;
-       struct oz_port *port = (struct oz_port *)hport;
+       struct oz_port *port = hport;
        struct oz_hcd *ozhcd = port->ozhcd;
        struct oz_urb_link *urbl, *n;
        LIST_HEAD(xfr_list);
index be7ee01c50aba30694227052a9658fa96aeb1fbf..d434d8c6fff67c04b58d6cac5c76a6832bae5bc3 100644 (file)
@@ -56,7 +56,7 @@ static int oz_usb_submit_elt(struct oz_elt_buf *eb, struct oz_elt_info *ei,
 int oz_usb_get_desc_req(void *hpd, u8 req_id, u8 req_type, u8 desc_type,
        u8 index, __le16 windex, int offset, int len)
 {
-       struct oz_usb_ctx *usb_ctx = (struct oz_usb_ctx *)hpd;
+       struct oz_usb_ctx *usb_ctx = hpd;
        struct oz_pd *pd = usb_ctx->pd;
        struct oz_elt *elt;
        struct oz_get_desc_req *body;
@@ -92,7 +92,7 @@ int oz_usb_get_desc_req(void *hpd, u8 req_id, u8 req_type, u8 desc_type,
  */
 static int oz_usb_set_config_req(void *hpd, u8 req_id, u8 index)
 {
-       struct oz_usb_ctx *usb_ctx = (struct oz_usb_ctx *)hpd;
+       struct oz_usb_ctx *usb_ctx = hpd;
        struct oz_pd *pd = usb_ctx->pd;
        struct oz_elt *elt;
        struct oz_elt_buf *eb = &pd->elt_buff;
@@ -115,7 +115,7 @@ static int oz_usb_set_config_req(void *hpd, u8 req_id, u8 index)
  */
 static int oz_usb_set_interface_req(void *hpd, u8 req_id, u8 index, u8 alt)
 {
-       struct oz_usb_ctx *usb_ctx = (struct oz_usb_ctx *)hpd;
+       struct oz_usb_ctx *usb_ctx = hpd;
        struct oz_pd *pd = usb_ctx->pd;
        struct oz_elt *elt;
        struct oz_elt_buf *eb = &pd->elt_buff;
@@ -140,7 +140,7 @@ static int oz_usb_set_interface_req(void *hpd, u8 req_id, u8 index, u8 alt)
 static int oz_usb_set_clear_feature_req(void *hpd, u8 req_id, u8 type,
                        u8 recipient, u8 index, __le16 feature)
 {
-       struct oz_usb_ctx *usb_ctx = (struct oz_usb_ctx *)hpd;
+       struct oz_usb_ctx *usb_ctx = hpd;
        struct oz_pd *pd = usb_ctx->pd;
        struct oz_elt *elt;
        struct oz_elt_buf *eb = &pd->elt_buff;
@@ -166,7 +166,7 @@ static int oz_usb_set_clear_feature_req(void *hpd, u8 req_id, u8 type,
 static int oz_usb_vendor_class_req(void *hpd, u8 req_id, u8 req_type,
        u8 request, __le16 value, __le16 index, const u8 *data, int data_len)
 {
-       struct oz_usb_ctx *usb_ctx = (struct oz_usb_ctx *)hpd;
+       struct oz_usb_ctx *usb_ctx = hpd;
        struct oz_pd *pd = usb_ctx->pd;
        struct oz_elt *elt;
        struct oz_elt_buf *eb = &pd->elt_buff;
@@ -244,7 +244,7 @@ int oz_usb_control_req(void *hpd, u8 req_id, struct usb_ctrlrequest *setup,
  */
 int oz_usb_send_isoc(void *hpd, u8 ep_num, struct urb *urb)
 {
-       struct oz_usb_ctx *usb_ctx = (struct oz_usb_ctx *)hpd;
+       struct oz_usb_ctx *usb_ctx = hpd;
        struct oz_pd *pd = usb_ctx->pd;
        struct oz_elt_buf *eb;
        int i;