* [PATCH] staging: rtl8723au: core: rtw_wlan_util: fix misleading indentation
@ 2015-10-21 17:32 Luis de Bethencourt
2015-10-23 15:57 ` Jes Sorensen
0 siblings, 1 reply; 2+ messages in thread
From: Luis de Bethencourt @ 2015-10-21 17:32 UTC (permalink / raw)
To: linux-kernel
Cc: devel, gregkh, Larry.Finger, Jes.Sorensen, roberta.dobrescu, joe,
fabf, amitoj1606, linux-wireless, Luis de Bethencourt
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@osg.samsung.com>
---
drivers/staging/rtl8723au/core/rtw_wlan_util.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
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;
--
2.5.1
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] staging: rtl8723au: core: rtw_wlan_util: fix misleading indentation
2015-10-21 17:32 [PATCH] staging: rtl8723au: core: rtw_wlan_util: fix misleading indentation Luis de Bethencourt
@ 2015-10-23 15:57 ` Jes Sorensen
0 siblings, 0 replies; 2+ messages in thread
From: Jes Sorensen @ 2015-10-23 15:57 UTC (permalink / raw)
To: Luis de Bethencourt; +Cc: devel, gregkh, Larry.Finger, linux-wireless
Luis de Bethencourt <luisbg@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@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@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;
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-10-23 15:57 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-21 17:32 [PATCH] staging: rtl8723au: core: rtw_wlan_util: fix misleading indentation Luis de Bethencourt
2015-10-23 15:57 ` Jes Sorensen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).