wireless: Remove casts to same type

Adding casts of objects to the same type is unnecessary
and confusing for a human reader.

For example, this cast:

        int y;
        int *p = (int *)&y;

I used the coccinelle script below to find and remove these
unnecessary casts.  I manually removed the conversions this
script produces of casts with __force, __iomem and __user.

@@
type T;
T *p;
@@

-       (T *)p
+       p

Neatened the mwifiex_deauthenticate_infra function which
was doing odd things with array pointers and not using
is_zero_ether_addr.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 10 insertions, 14 deletions

diff --git a/drivers/net/wireless/mwifiex/scan.c b/drivers/net/wireless/mwifiex/scan.c
index 74f045715723..8fa763fa629a 100644
--- a/drivers/net/wireless/mwifiex/scan.c
+++ b/drivers/net/wireless/mwifiex/scan.c
@@ -1014,14 +1014,12 @@ mwifiex_ret_802_11_scan_get_tlv_ptrs(struct mwifiex_adapter *adapter,
                         case TLV_TYPE_TSFTIMESTAMP:
                                 dev_dbg(adapter->dev, "info: SCAN_RESP: TSF "
                                         "timestamp TLV, len = %d\n", tlv_len);
-                                *tlv_data = (struct mwifiex_ie_types_data *)
-                                        current_tlv;
+                                *tlv_data = current_tlv;
                                 break;
                         case TLV_TYPE_CHANNELBANDLIST:
                                 dev_dbg(adapter->dev, "info: SCAN_RESP: channel"
                                         " band list TLV, len = %d\n", tlv_len);
-                                *tlv_data = (struct mwifiex_ie_types_data *)
-                                        current_tlv;
+                                *tlv_data = current_tlv;
                                 break;
                         default:
                                 dev_err(adapter->dev,
@@ -1226,15 +1224,15 @@ int mwifiex_update_bss_desc_with_ie(struct mwifiex_adapter *adapter,
                                         bss_entry->beacon_buf);
                         break;
                 case WLAN_EID_BSS_COEX_2040:
-                        bss_entry->bcn_bss_co_2040 = (u8 *) (current_ptr +
-                                        sizeof(struct ieee_types_header));
+                        bss_entry->bcn_bss_co_2040 = current_ptr +
+                                sizeof(struct ieee_types_header);
                         bss_entry->bss_co_2040_offset = (u16) (current_ptr +
                                         sizeof(struct ieee_types_header) -
                                                 bss_entry->beacon_buf);
                         break;
                 case WLAN_EID_EXT_CAPABILITY:
-                        bss_entry->bcn_ext_cap = (u8 *) (current_ptr +
-                                        sizeof(struct ieee_types_header));
+                        bss_entry->bcn_ext_cap = current_ptr +
+                                sizeof(struct ieee_types_header);
                         bss_entry->ext_cap_offset = (u16) (current_ptr +
                                         sizeof(struct ieee_types_header) -
                                         bss_entry->beacon_buf);
@@ -1683,8 +1681,7 @@ int mwifiex_ret_802_11_scan(struct mwifiex_private *priv,
                                 goto done;
                         }
                         if (element_id == WLAN_EID_DS_PARAMS) {
-                                channel = *(u8 *) (current_ptr +
-                                        sizeof(struct ieee_types_header));
+                                channel = *(current_ptr + sizeof(struct ieee_types_header));
                                 break;
                         }
 
@@ -2010,12 +2007,11 @@ mwifiex_save_curr_bcn(struct mwifiex_private *priv)
 
         if (curr_bss->bcn_bss_co_2040)
                 curr_bss->bcn_bss_co_2040 =
-                        (u8 *) (curr_bss->beacon_buf +
-                                        curr_bss->bss_co_2040_offset);
+                        (curr_bss->beacon_buf + curr_bss->bss_co_2040_offset);
 
         if (curr_bss->bcn_ext_cap)
-                curr_bss->bcn_ext_cap = (u8 *) (curr_bss->beacon_buf +
-                                curr_bss->ext_cap_offset);
+                curr_bss->bcn_ext_cap = curr_bss->beacon_buf +
+                        curr_bss->ext_cap_offset;
 }
 
 /*