Re: [PATCH] staging: rtl8723bs: Fix indentation in conditional statements

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Oct 25, 2022 at 07:03:17AM +0200, Julia Lawall wrote:
> 
> 
> On Mon, 24 Oct 2022, Emily Peri wrote:
> 
> > Remove/add tabs in block statements in rtl8723bs/core to fix checkpatch
> > warnings for suspect code indent for conditionals.
> 
> It's a bit picky, but one case is a loop.
> 
> julia

Thanks Julia, I did notice that too. I hoped by writing 'block
statements' in the description I could cover that case. Checkpatch
actually gave the same "WARNING: suspect code indent for conditional
statements" for both the loop and the conditionals, so that was what I
was referring to in the descripton. I agree I could have made that more
clear!

> > Signed-off-by: Emily Peri <eperi1024@xxxxxxxxx>
> > ---
> >  drivers/staging/rtl8723bs/core/rtw_mlme.c     | 30 +++++++++----------
> >  drivers/staging/rtl8723bs/core/rtw_mlme_ext.c |  2 +-
> >  drivers/staging/rtl8723bs/core/rtw_recv.c     |  6 ++--
> >  drivers/staging/rtl8723bs/core/rtw_sta_mgt.c  |  2 +-
> >  4 files changed, 19 insertions(+), 21 deletions(-)
> >
> > diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rtl8723bs/core/rtw_mlme.c
> > index 6498fd17e1d3..c6fd6cf741ef 100644
> > --- a/drivers/staging/rtl8723bs/core/rtw_mlme.c
> > +++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c
> > @@ -389,7 +389,7 @@ int is_same_network(struct wlan_bssid_ex *src, struct wlan_bssid_ex *dst, u8 fea
> >  	__le16 tmps, tmpd;
> >
> >  	if (rtw_bug_check(dst, src, &s_cap, &d_cap) == false)
> > -			return false;
> > +		return false;
> >
> >  	memcpy((u8 *)&tmps, rtw_get_capability_from_ie(src->ies), 2);
> >  	memcpy((u8 *)&tmpd, rtw_get_capability_from_ie(dst->ies), 2);
> > @@ -669,7 +669,7 @@ int rtw_is_desired_network(struct adapter *adapter, struct wlan_network *pnetwor
> >  		uint ie_len = 0;
> >
> >  		if ((desired_encmode == Ndis802_11EncryptionDisabled) && (privacy != 0))
> > -	    bselected = false;
> > +			bselected = false;
> >
> >  		if (psecuritypriv->ndisauthtype == Ndis802_11AuthModeWPA2PSK) {
> >  			p = rtw_get_ie(pnetwork->network.ies + _BEACON_IE_OFFSET_, WLAN_EID_RSN, &ie_len, (pnetwork->network.ie_length - _BEACON_IE_OFFSET_));
> > @@ -795,7 +795,7 @@ void rtw_surveydone_event_callback(struct adapter	*adapter, u8 *pbuf)
> >  			pmlmepriv->to_join = false;
> >  			s_ret = rtw_select_and_join_from_scanned_queue(pmlmepriv);
> >  			if (s_ret == _SUCCESS) {
> > -			     _set_timer(&pmlmepriv->assoc_timer, MAX_JOIN_TIMEOUT);
> > +				_set_timer(&pmlmepriv->assoc_timer, MAX_JOIN_TIMEOUT);
> >  			} else if (s_ret == 2) {/* there is no need to wait for join */
> >  				_clr_fwstate_(pmlmepriv, _FW_UNDER_LINKING);
> >  				rtw_indicate_connect(adapter);
> > @@ -2010,8 +2010,8 @@ int rtw_restruct_wmm_ie(struct adapter *adapter, u8 *in_ie, u8 *out_ie, uint in_
> >
> >  		if (in_ie[i] == 0xDD && in_ie[i+2] == 0x00 && in_ie[i+3] == 0x50  && in_ie[i+4] == 0xF2 && in_ie[i+5] == 0x02 && i+5 < in_len) { /* WMM element ID and OUI */
> >  			for (j = i; j < i + 9; j++) {
> > -					out_ie[ielength] = in_ie[j];
> > -					ielength++;
> > +				out_ie[ielength] = in_ie[j];
> > +				ielength++;
> >  			}
> >  			out_ie[initial_out_len + 1] = 0x07;
> >  			out_ie[initial_out_len + 6] = 0x00;
> > @@ -2064,15 +2064,13 @@ static int rtw_append_pmkid(struct adapter *Adapter, int iEntry, u8 *ie, uint ie
> >
> >  	if (ie[13] <= 20) {
> >  		/*  The RSN IE didn't include the PMK ID, append the PMK information */
> > -			ie[ie_len] = 1;
> > -			ie_len++;
> > -			ie[ie_len] = 0;	/* PMKID count = 0x0100 */
> > -			ie_len++;
> > -			memcpy(&ie[ie_len], &psecuritypriv->PMKIDList[iEntry].PMKID, 16);
> > -
> > -			ie_len += 16;
> > -			ie[13] += 18;/* PMKID length = 2+16 */
> > -
> > +		ie[ie_len] = 1;
> > +		ie_len++;
> > +		ie[ie_len] = 0;	/* PMKID count = 0x0100 */
> > +		ie_len++;
> > +		memcpy(&ie[ie_len], &psecuritypriv->PMKIDList[iEntry].PMKID, 16);
> > +		ie_len += 16;
> > +		ie[13] += 18;/* PMKID length = 2+16 */
> >  	}
> >  	return ie_len;
> >  }
> > @@ -2091,9 +2089,9 @@ signed int rtw_restruct_sec_ie(struct adapter *adapter, u8 *in_ie, u8 *out_ie, u
> >  	memcpy(out_ie, in_ie, 12);
> >  	ielength = 12;
> >  	if ((ndisauthmode == Ndis802_11AuthModeWPA) || (ndisauthmode == Ndis802_11AuthModeWPAPSK))
> > -			authmode = WLAN_EID_VENDOR_SPECIFIC;
> > +		authmode = WLAN_EID_VENDOR_SPECIFIC;
> >  	if ((ndisauthmode == Ndis802_11AuthModeWPA2) || (ndisauthmode == Ndis802_11AuthModeWPA2PSK))
> > -			authmode = WLAN_EID_RSN;
> > +		authmode = WLAN_EID_RSN;
> >
> >  	if (check_fwstate(pmlmepriv, WIFI_UNDER_WPS)) {
> >  		memcpy(out_ie+ielength, psecuritypriv->wps_ie, psecuritypriv->wps_ie_len);
> > diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
> > index 8e74b4f47b94..1148c9829890 100644
> > --- a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
> > +++ b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
> > @@ -5447,7 +5447,7 @@ u8 disconnect_hdl(struct adapter *padapter, unsigned char *pbuf)
> >  	u8 val8;
> >
> >  	if (is_client_associated_to_ap(padapter))
> > -			issue_deauth_ex(padapter, pnetwork->mac_address, WLAN_REASON_DEAUTH_LEAVING, param->deauth_timeout_ms/100, 100);
> > +		issue_deauth_ex(padapter, pnetwork->mac_address, WLAN_REASON_DEAUTH_LEAVING, param->deauth_timeout_ms/100, 100);
> >
> >  	if (((pmlmeinfo->state&0x03) == WIFI_FW_ADHOC_STATE) || ((pmlmeinfo->state&0x03) == WIFI_FW_AP_STATE)) {
> >  		/* Stop BCN */
> > diff --git a/drivers/staging/rtl8723bs/core/rtw_recv.c b/drivers/staging/rtl8723bs/core/rtw_recv.c
> > index 2825375bff94..7c7b6495965f 100644
> > --- a/drivers/staging/rtl8723bs/core/rtw_recv.c
> > +++ b/drivers/staging/rtl8723bs/core/rtw_recv.c
> > @@ -161,7 +161,7 @@ int rtw_free_recvframe(union recv_frame *precvframe, struct __queue *pfree_recv_
> >
> >  	if (padapter) {
> >  		if (pfree_recv_queue == &precvpriv->free_recv_queue)
> > -				precvpriv->free_recvframe_cnt++;
> > +			precvpriv->free_recvframe_cnt++;
> >  	}
> >  	spin_unlock_bh(&pfree_recv_queue->lock);
> >  	return _SUCCESS;
> > @@ -691,8 +691,8 @@ static signed int sta2sta_data_frame(struct adapter *adapter, union recv_frame *
> >  		if (bmcast) {
> >  			/*  For AP mode, if DA == MCAST, then BSSID should be also MCAST */
> >  			if (!IS_MCAST(pattrib->bssid)) {
> > -					ret = _FAIL;
> > -					goto exit;
> > +				ret = _FAIL;
> > +				goto exit;
> >  			}
> >  		} else { /*  not mc-frame */
> >  			/*  For AP mode, if DA is non-MCAST, then it must be BSSID, and bssid == BSSID */
> > diff --git a/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c b/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c
> > index beb11d89db18..c7de81f21bec 100644
> > --- a/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c
> > +++ b/drivers/staging/rtl8723bs/core/rtw_sta_mgt.c
> > @@ -551,7 +551,7 @@ u8 rtw_access_ctrl(struct adapter *padapter, u8 *mac_addr)
> >  	else if (pacl_list->mode == 2)/* deny unless in accept list */
> >  		res = match;
> >  	else
> > -		 res = true;
> > +		res = true;
> >
> >  	return res;
> >  }
> > --
> > 2.34.1
> >
> >
> >




[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux