[PATCH] staging: rtl8723au: core: rtw_wlan_util: fix misleading indentation

Jes Sorensen Jes.Sorensen at redhat.com
Fri Oct 23 15:57:15 UTC 2015


Luis de Bethencourt <luisbg at osg.samsung.com> writes:
> For loop is outside of the else branch of the above conditional statement.
> Fixing misleading indentation.
>
> Fix a smatch warning:
> drivers/staging/rtl8723au/core/rtw_wlan_util.c:528
> WMMOnAssocRsp23a() warn: curly braces intended?
>
> Signed-off-by: Luis de Bethencourt <luisbg at osg.samsung.com>
> ---
>  drivers/staging/rtl8723au/core/rtw_wlan_util.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Acked-by: Jes Sorensen <Jes.Sorensen at redhat.com>

>
> diff --git a/drivers/staging/rtl8723au/core/rtw_wlan_util.c b/drivers/staging/rtl8723au/core/rtw_wlan_util.c
> index 5e87360..cc2b84b 100644
> --- a/drivers/staging/rtl8723au/core/rtw_wlan_util.c
> +++ b/drivers/staging/rtl8723au/core/rtw_wlan_util.c
> @@ -525,7 +525,7 @@ void WMMOnAssocRsp23a(struct rtw_adapter *padapter)
>  	else
>  		aSifsTime = 16;
>  
> -		for (i = 0; i < 4; i++) {
> +	for (i = 0; i < 4; i++) {
>  		ACI = (pmlmeinfo->WMM_param.ac_param[i].ACI_AIFSN >> 5) & 0x03;
>  		ACM = (pmlmeinfo->WMM_param.ac_param[i].ACI_AIFSN >> 4) & 0x01;


More information about the devel mailing list