Staging: rtl8723bs: fix spaces in ieee80211.h
authorMarco Cesati <marcocesati@gmail.com>
Mon, 15 Mar 2021 17:06:05 +0000 (18:06 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 16 Mar 2021 11:23:17 +0000 (12:23 +0100)
This commit fixes the following checkpatch.pl errors:

    ERROR:POINTER_LOCATION: "foo*bar" should be "foo *bar"
    #1001: FILE: ./include/ieee80211.h:1001:
    +u8 *rtw_get_ie(u8*pbuf, signed int index, signed int *len, signed int limit);

    ERROR:POINTER_LOCATION: "(foo*)" should be "(foo *)"
    #1029: FILE: ./include/ieee80211.h:1029:
    + for (ie = (void*)buf; (((u8 *)ie) - ((u8 *)buf) + 1) < buf_len; ie = (void*)(((u8 *)ie) + *(((u8 *)ie)+1) + 2))

    ERROR:POINTER_LOCATION: "(foo*)" should be "(foo *)"
    #1029: FILE: ./include/ieee80211.h:1029:
    + for (ie = (void*)buf; (((u8 *)ie) - ((u8 *)buf) + 1) < buf_len; ie = (void*)(((u8 *)ie) + *(((u8 *)ie)+1) + 2))

    ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
    #1049: FILE: ./include/ieee80211.h:1049:
    +u16 rtw_mcs_rate(u8 rf_type, u8 bw_40MHz, u8 short_GI, unsigned char * MCS_rate);

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Marco Cesati <marcocesati@gmail.com>
Link: https://lore.kernel.org/r/20210315170618.2566-45-marcocesati@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/include/ieee80211.h

index bda5712..1c671b7 100644 (file)
@@ -998,7 +998,7 @@ enum secondary_ch_offset {
        SCB = 3,  /* secondary channel below */
 };
 
-u8 *rtw_get_ie(u8*pbuf, signed int index, signed int *len, signed int limit);
+u8 *rtw_get_ie(u8 *pbuf, signed int index, signed int *len, signed int limit);
 u8 *rtw_get_ie_ex(u8 *in_ie, uint in_len, u8 eid, u8 *oui, u8 oui_len, u8 *ie, uint *ielen);
 int rtw_ies_remove_ie(u8 *ies, uint *ies_len, uint offset, u8 eid, u8 *oui, u8 oui_len);
 
@@ -1026,7 +1026,8 @@ u8 *rtw_get_wps_attr_content(u8 *wps_ie, uint wps_ielen, u16 target_attr_id, u8
  * @buf_len:
  */
 #define for_each_ie(ie, buf, buf_len) \
-       for (ie = (void*)buf; (((u8 *)ie) - ((u8 *)buf) + 1) < buf_len; ie = (void*)(((u8 *)ie) + *(((u8 *)ie)+1) + 2))
+       for (ie = (void *)buf; (((u8 *)ie) - ((u8 *)buf) + 1) < buf_len; \
+               ie = (void *)(((u8 *)ie) + *(((u8 *)ie) + 1) + 2))
 
 uint   rtw_get_rateset_len(u8 *rateset);
 
@@ -1046,7 +1047,7 @@ void rtw_get_bcn_info(struct wlan_network *pnetwork);
 
 void rtw_macaddr_cfg(struct device *dev, u8 *mac_addr);
 
-u16 rtw_mcs_rate(u8 rf_type, u8 bw_40MHz, u8 short_GI, unsigned char * MCS_rate);
+u16 rtw_mcs_rate(u8 rf_type, u8 bw_40MHz, u8 short_GI, unsigned char *MCS_rate);
 
 int rtw_action_frame_parse(const u8 *frame, u32 frame_len, u8 *category, u8 *action);
 const char *action_public_str(u8 action);