staging:rtl8188eu:core Add spaces around &,+,-
PATCH 1/1 staging:rtl8188eu:core Add spaces around &,+,-
This Patch is to solve coding style issues, Major changes to
add Space around &,+,-. Major pending is to fix CamelCase
Signed-off-by Ashish Kalra <[email protected]>
--- drivers/staging/rtl8188eu/core/rtw_ap.c | 65 +++++++++++++++------------------ 1 file changed, 29 insertions(+), 36 deletions(-)
drivers/staging/rtl8188eu/core/rtw_ap.c +29 -36
@@ -44,7 +44,7 @@ void free_mlme_ap_info(struct adapter *padapter)struct sta_priv *pstapriv = &padapter->stapriv;struct mlme_priv *pmlmepriv = &padapter->mlmepriv;struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv; -struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info); +struct mlme_ext_info *pmlmeinfo = &pmlmeext->mlmext_info; pmlmepriv->update_bcn = false; pmlmeext->bstart_bss = false;
@@ -76,7 +76,7 @@ static void update_BCNTIM(struct adapter *padapter)/* update TIM IE */ p =rtw_get_ie (pie + _FIXED_IE_LENGTH_, _TIM_IE_, &tim_ielen, - pnetwork_mlmeext->IELength - _FIXED_IE_LENGTH_); + pnetwork_mlmeext->IELength - _FIXED_IE_LENGTH_);if (p && tim_ielen >0 ) { tim_ielen +=2 ; premainder_ie = p + tim_ielen;
@@ -97,7 +97,7 @@ static void update_BCNTIM(struct adapter *padapter) &tmp_len, (pnetwork_mlmeext->IELength - _BEACON_IE_OFFSET_));if (p) - offset += tmp_len+2 ; + offset += tmp_len +2 ;/* DS Parameter Set IE, len = 3 */ offset +=3 ;
@@ -119,7 +119,7 @@ static void update_BCNTIM(struct adapter *padapter) } *dst_ie++ = _TIM_IE_; -if ((pstapriv->tim_bitmap&0xff00 ) && (pstapriv->tim_bitmap&0x00fc )) +if ((pstapriv->tim_bitmap &0xff00 ) && (pstapriv->tim_bitmap &0x00fc )) tim_ielen =5 ;else tim_ielen =4 ;
@@ -129,7 +129,7 @@ static void update_BCNTIM(struct adapter *padapter) *dst_ie++ =0 ;/* DTIM count */ *dst_ie++ =1 ;/* DTIM period */ -if (pstapriv->tim_bitmap&BIT (0 ))/* for bc/mc frames */ +if (pstapriv->tim_bitmap &BIT (0 ))/* for bc/mc frames */ *dst_ie++ =BIT (0 );/* bitmap ctrl */ else *dst_ie++ =0 ;
@@ -314,7 +314,6 @@ void expire_timeout_chk(struct adapter *padapter)spin_lock_bh (&pstapriv->auth_list_lock); } } - }spin_unlock_bh (&pstapriv->auth_list_lock);
@@ -635,9 +634,7 @@ void update_sta_info_apmode(struct adapter *padapter, struct sta_info *psta)else psta->ieee8021x_blocked = false; -/* update sta's cap */ -/* ERP */ VCS_update (padapter, psta);/* HT related cap */
@@ -694,9 +691,9 @@ static void update_hw_ht_param(struct adapter *padapter)/* handle A-MPDU parameter field */ /* -ampdu_params_info [1:0]:Max AMPDU Len => 0:8k , 1:16k, 2:32k, 3:64k -ampdu_params_info [4:2]:Min MPDU Start Spacing -*/ +* ampdu_params_info [1:0]:Max AMPDU Len => 0:8k , 1:16k, 2:32k, 3:64k +* ampdu_params_info [4:2]:Min MPDU Start Spacing +*/ max_AMPDU_len = pmlmeinfo->HT_caps.ampdu_params_info &0x03 ; min_MPDU_spacing = (pmlmeinfo->HT_caps.ampdu_params_info &0x1c ) >>2 ;
@@ -734,7 +731,6 @@ static void start_bss_network(struct adapter *padapter, u8 *pbuf) cur_bwmode =HT_CHANNEL_WIDTH_20 ; cur_ch_offset =HAL_PRIME_CHNL_OFFSET_DONT_CARE ; -/* check if there is wps ie, * if there is wpsie in beacon, the hostapd will update * beacon twice when stating hostapd, and at first time the
@@ -884,7 +880,6 @@ int rtw_check_beacon_data(struct adapter *padapter, u8 *pbuf, int len)if (check_fwstate (pmlmepriv,WIFI_AP_STATE ) != true)return _FAIL; -if (len <0 || len >MAX_IE_SZ )return _FAIL;
@@ -894,7 +889,6 @@ int rtw_check_beacon_data(struct adapter *padapter, u8 *pbuf, int len)memcpy (ie, pbuf, pbss_network->IELength); -if (pbss_network->InfrastructureMode != Ndis802_11APMode)return _FAIL;
@@ -1043,14 +1037,14 @@ int rtw_check_beacon_data(struct adapter *padapter, u8 *pbuf, int len) (psecuritypriv->wpa2_pairwise_cipher & WPA_CIPHER_CCMP)) pht_cap->ampdu_params_info |= (IEEE80211_HT_CAP_AMPDU_DENSITY & (0x07 << 2)); else - pht_cap->ampdu_params_info |= (IEEE80211_HT_CAP_AMPDU_DENSITY&0x00); + pht_cap->ampdu_params_info |= (IEEE80211_HT_CAP_AMPDU_DENSITY & 0x00);/* set Max Rx AMPDU size to 64K */ pht_cap->ampdu_params_info |= (IEEE80211_HT_CAP_AMPDU_FACTOR &0x03 ); pht_cap->mcs.rx_mask[0 ] =0xff ; pht_cap->mcs.rx_mask[1 ] =0x0 ; -memcpy (&pmlmepriv->htpriv.ht_cap, p+2 , ie_len); +memcpy (&pmlmepriv->htpriv.ht_cap, p +2 , ie_len); }/* parsing HT_INFO_IE */
@@ -1250,9 +1244,9 @@ static void update_bcn_erpinfo_ie(struct adapter *padapter)struct ndis_802_11_var_ie *pIE = (struct ndis_802_11_var_ie *)p;if (pmlmepriv->num_sta_non_erp ==1 ) - pIE->data[0 ] |=RTW_ERP_INFO_NON_ERP_PRESENT |RTW_ERP_INFO_USE_PROTECTION ; + pIE->data[0 ] |=RTW_ERP_INFO_NON_ERP_PRESENT |RTW_ERP_INFO_USE_PROTECTION ;else - pIE->data[0 ] &= ~(RTW_ERP_INFO_NON_ERP_PRESENT |RTW_ERP_INFO_USE_PROTECTION ); + pIE->data[0 ] &= ~(RTW_ERP_INFO_NON_ERP_PRESENT |RTW_ERP_INFO_USE_PROTECTION );if (pmlmepriv->num_sta_no_short_preamble >0 ) pIE->data[0 ] |=RTW_ERP_INFO_BARKER_PREAMBLE_MODE ;
@@ -1281,12 +1275,12 @@ static void update_bcn_wps_ie(struct adapter *padapter)if (!pwps_ie_src)return ; - pwps_ie =rtw_get_wps_ie (ie+_FIXED_IE_LENGTH_, ielen-_FIXED_IE_LENGTH_,NULL , &wps_ielen); + pwps_ie =rtw_get_wps_ie (ie + _FIXED_IE_LENGTH_, ielen - _FIXED_IE_LENGTH_,NULL , &wps_ielen);if (!pwps_ie || wps_ielen ==0 )return ; - wps_offset = (uint)(pwps_ie-ie); + wps_offset = (uint)(pwps_ie - ie); premainder_ie = pwps_ie + wps_ielen;
@@ -1300,14 +1294,14 @@ static void update_bcn_wps_ie(struct adapter *padapter) wps_ielen = (uint )pwps_ie_src[1 ];/* to get ie data len */ if ((wps_offset+wps_ielen+2 +remainder_ielen) <=MAX_IE_SZ ) { -memcpy (pwps_ie, pwps_ie_src, wps_ielen+2 ); - pwps_ie += (wps_ielen+2 ); +memcpy (pwps_ie, pwps_ie_src, wps_ielen +2 ); + pwps_ie += (wps_ielen +2 );if (pbackup_remainder_ie)memcpy (pwps_ie, pbackup_remainder_ie, remainder_ielen);/* update IELength */ - pnetwork->IELength = wps_offset + (wps_ielen+2 ) + remainder_ielen; + pnetwork->IELength = wps_offset + (wps_ielen +2 ) + remainder_ielen; }kfree (pbackup_remainder_ie);
@@ -1362,16 +1356,16 @@ void update_beacon(struct adapter *padapter, u8 ie_id, u8 *oui, u8 tx) }/* -op_mode -Set to 0 (HT pure) under the following conditions -- all STAs in the BSS are 20/40 MHz HT in 20/40 MHz BSS or -- all STAs in the BSS are 20 MHz HT in 20 MHz BSS -Set to 1 (HT non-member protection) if there may be non-HT STAs -in both the primary and the secondary channel -Set to 2 if only HT STAs are associated in BSS, -however and at least one 20 MHz HT STA is associated -Set to 3 (HT mixed mode) when one or more non-HT STAs are associated -(currently non-GF HT station is considered as non-HT STA also) +*op_mode +*Set to 0 (HT pure) under the following conditions +* - all STAs in the BSS are 20/40 MHz HT in 20/40 MHz BSS or +* - all STAs in the BSS are 20 MHz HT in 20 MHz BSS +*Set to 1 (HT non-member protection) if there may be non-HT STAs +* in both the primary and the secondary channel +*Set to 2 if only HT STAs are associated in BSS, +* however and at least one 20 MHz HT STA is associated +*Set to 3 (HT mixed mode) when one or more non-HT STAs are associated +* (currently non-GF HT station is considered as non-HT STA also) */ static int rtw_ht_operation_update (struct adapter *padapter) {
@@ -1690,7 +1684,6 @@ u8 ap_free_sta(struct adapter *padapter, struct sta_info *psta,/* clear cam entry / key */ rtw_clearstakey_cmd (padapter, (u8 *)psta, (u8)(psta->mac_id +3 ), true); -spin_lock_bh (&psta->lock); psta->state &= ~_FW_LINKED;spin_unlock_bh (&psta->lock);
@@ -1754,7 +1747,7 @@ void sta_info_update(struct adapter *padapter, struct sta_info *psta)struct mlme_priv *pmlmepriv = &padapter->mlmepriv;/* update wmm cap. */ -if (WLAN_STA_WME &flags) +if (WLAN_STA_WME & flags) psta->qos_option =1 ;else psta->qos_option =0 ;
@@ -1763,7 +1756,7 @@ void sta_info_update(struct adapter *padapter, struct sta_info *psta) psta->qos_option =0 ;/* update 802.11n ht cap. */ -if (WLAN_STA_HT &flags) { +if (WLAN_STA_HT & flags) { psta->htpriv.ht_option = true; psta->qos_option =1 ; }else {
Signature · 1 line
1.9.1
Greg KH
maintainerQuoted 2 lines · Ashish Kalra
This Patch is to solve coding style issues, Major changes to
add Space around &,+,-. Major pending is to fix CamelCase
Please only do one type of thing per patch. And "fix all issues" is not one thing.
thanks,
greg k-h