* [PATCH 0/3] staging: rtl8723bs: clean up driver code in rtw_ioctl_set
@ 2022-10-20 2:10 Emily Peri
2022-10-20 2:10 ` [PATCH 1/3] staging: rtl8723bs: fix white space warnings Emily Peri
` (2 more replies)
0 siblings, 3 replies; 10+ messages in thread
From: Emily Peri @ 2022-10-20 2:10 UTC (permalink / raw)
To: gregkh, outreachy, linux-staging, linux-kernel; +Cc: Emily Peri
Created patchset to fix all checkpatch warnings in rtw_ioctl_set
Emily Peri (3):
staging: rtl8723bs: fix white space warnings
staging: rtl8723bs: align block comment stars
staging: rtl8723bs: remove unnecessary parenthesis
drivers/staging/rtl8723bs/core/rtw_ioctl_set.c | 17 +++++++++--------
1 file changed, 9 insertions(+), 8 deletions(-)
--
2.34.1
^ permalink raw reply [flat|nested] 10+ messages in thread* [PATCH 1/3] staging: rtl8723bs: fix white space warnings 2022-10-20 2:10 [PATCH 0/3] staging: rtl8723bs: clean up driver code in rtw_ioctl_set Emily Peri @ 2022-10-20 2:10 ` Emily Peri 2022-10-20 4:04 ` Praveen Kumar 2022-10-20 4:57 ` Greg KH 2022-10-20 2:10 ` [PATCH 2/3] staging: rtl8723bs: align block comment stars Emily Peri 2022-10-20 2:10 ` [PATCH 3/3] staging: rtl8723bs: remove unnecessary parenthesis Emily Peri 2 siblings, 2 replies; 10+ messages in thread From: Emily Peri @ 2022-10-20 2:10 UTC (permalink / raw) To: gregkh, outreachy, linux-staging, linux-kernel; +Cc: Emily Peri Fix the following checkpatch warnings in rtw_ioctl_set: 1) Add missing blankline after declaration 2) Replace spaces used for indent with tab 3) Remove space before tab Signed-off-by: Emily Peri <eperi1024@gmail.com> --- drivers/staging/rtl8723bs/core/rtw_ioctl_set.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c index 8c11daff2d59..354e61a8f2bd 100644 --- a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c +++ b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c @@ -78,6 +78,7 @@ u8 rtw_do_join(struct adapter *padapter) goto exit; } else { int select_ret; + spin_unlock_bh(&(pmlmepriv->scanned_queue.lock)); select_ret = rtw_select_and_join_from_scanned_queue(pmlmepriv); if (select_ret == _SUCCESS) { @@ -311,7 +312,7 @@ u8 rtw_set_802_11_infrastructure_mode(struct adapter *padapter, if ((*pold_state == Ndis802_11Infrastructure) || (*pold_state == Ndis802_11IBSS)) { if (check_fwstate(pmlmepriv, _FW_LINKED) == true) rtw_indicate_disconnect(padapter); /* will clr Linked_state; before this function, we must have checked whether issue dis-assoc_cmd or not */ - } + } *pold_state = networktype; @@ -367,7 +368,7 @@ u8 rtw_set_802_11_disassociate(struct adapter *padapter) u8 rtw_set_802_11_bssid_list_scan(struct adapter *padapter, struct ndis_802_11_ssid *pssid, int ssid_max_num) { - struct mlme_priv *pmlmepriv = &padapter->mlmepriv; + struct mlme_priv *pmlmepriv = &padapter->mlmepriv; u8 res = true; if (!padapter) { -- 2.34.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] staging: rtl8723bs: fix white space warnings 2022-10-20 2:10 ` [PATCH 1/3] staging: rtl8723bs: fix white space warnings Emily Peri @ 2022-10-20 4:04 ` Praveen Kumar 2022-10-20 5:05 ` Emily Peri 2022-10-20 4:57 ` Greg KH 1 sibling, 1 reply; 10+ messages in thread From: Praveen Kumar @ 2022-10-20 4:04 UTC (permalink / raw) To: Emily Peri, gregkh, outreachy, linux-staging, linux-kernel On 20-10-2022 07:40, Emily Peri wrote: > Fix the following checkpatch warnings in rtw_ioctl_set: > 1) Add missing blankline after declaration > 2) Replace spaces used for indent with tab > 3) Remove space before tab > > Signed-off-by: Emily Peri <eperi1024@gmail.com> > --- > drivers/staging/rtl8723bs/core/rtw_ioctl_set.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c > index 8c11daff2d59..354e61a8f2bd 100644 > --- a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c > +++ b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c > @@ -78,6 +78,7 @@ u8 rtw_do_join(struct adapter *padapter) > goto exit; > } else { > int select_ret; > + > spin_unlock_bh(&(pmlmepriv->scanned_queue.lock)); > select_ret = rtw_select_and_join_from_scanned_queue(pmlmepriv); > if (select_ret == _SUCCESS) { > @@ -311,7 +312,7 @@ u8 rtw_set_802_11_infrastructure_mode(struct adapter *padapter, > if ((*pold_state == Ndis802_11Infrastructure) || (*pold_state == Ndis802_11IBSS)) { > if (check_fwstate(pmlmepriv, _FW_LINKED) == true) > rtw_indicate_disconnect(padapter); /* will clr Linked_state; before this function, we must have checked whether issue dis-assoc_cmd or not */ > - } > + } I think indentation is wrong here, it should be only 1 tab instead of 2 tabs ? > > *pold_state = networktype; > > @@ -367,7 +368,7 @@ u8 rtw_set_802_11_disassociate(struct adapter *padapter) > > u8 rtw_set_802_11_bssid_list_scan(struct adapter *padapter, struct ndis_802_11_ssid *pssid, int ssid_max_num) > { > - struct mlme_priv *pmlmepriv = &padapter->mlmepriv; > + struct mlme_priv *pmlmepriv = &padapter->mlmepriv; > u8 res = true; > > if (!padapter) { Regards, ~Praveen. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] staging: rtl8723bs: fix white space warnings 2022-10-20 4:04 ` Praveen Kumar @ 2022-10-20 5:05 ` Emily Peri 2022-10-20 5:59 ` Michael Straube 0 siblings, 1 reply; 10+ messages in thread From: Emily Peri @ 2022-10-20 5:05 UTC (permalink / raw) To: Praveen Kumar; +Cc: gregkh, outreachy, linux-staging, linux-kernel On Thu, Oct 20, 2022 at 09:34:07AM +0530, Praveen Kumar wrote: > On 20-10-2022 07:40, Emily Peri wrote: > > Fix the following checkpatch warnings in rtw_ioctl_set: > > 1) Add missing blankline after declaration > > 2) Replace spaces used for indent with tab > > 3) Remove space before tab > > > > Signed-off-by: Emily Peri <eperi1024@gmail.com> > > --- > > drivers/staging/rtl8723bs/core/rtw_ioctl_set.c | 5 +++-- > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c > > index 8c11daff2d59..354e61a8f2bd 100644 > > --- a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c > > +++ b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c > > @@ -78,6 +78,7 @@ u8 rtw_do_join(struct adapter *padapter) > > goto exit; > > } else { > > int select_ret; > > + > > spin_unlock_bh(&(pmlmepriv->scanned_queue.lock)); > > select_ret = rtw_select_and_join_from_scanned_queue(pmlmepriv); > > if (select_ret == _SUCCESS) { > > @@ -311,7 +312,7 @@ u8 rtw_set_802_11_infrastructure_mode(struct adapter *padapter, > > if ((*pold_state == Ndis802_11Infrastructure) || (*pold_state == Ndis802_11IBSS)) { > > if (check_fwstate(pmlmepriv, _FW_LINKED) == true) > > rtw_indicate_disconnect(padapter); /* will clr Linked_state; before this function, we must have checked whether issue dis-assoc_cmd or not */ > > - } > > + } > > I think indentation is wrong here, it should be only 1 tab instead of 2 tabs ? Thanks Praveen for the feedback! But I think this if-statement is nested inside another one beginning at line 295. Can you confirm? > > > > *pold_state = networktype; > > > > @@ -367,7 +368,7 @@ u8 rtw_set_802_11_disassociate(struct adapter *padapter) > > > > u8 rtw_set_802_11_bssid_list_scan(struct adapter *padapter, struct ndis_802_11_ssid *pssid, int ssid_max_num) > > { > > - struct mlme_priv *pmlmepriv = &padapter->mlmepriv; > > + struct mlme_priv *pmlmepriv = &padapter->mlmepriv; > > u8 res = true; > > > > if (!padapter) { > > Regards, > > ~Praveen. Now that I think about-- in this last edit I removed a space to appease checkpatch, but maybe I actually should have removed the tab? Thanks! Emily ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] staging: rtl8723bs: fix white space warnings 2022-10-20 5:05 ` Emily Peri @ 2022-10-20 5:59 ` Michael Straube 0 siblings, 0 replies; 10+ messages in thread From: Michael Straube @ 2022-10-20 5:59 UTC (permalink / raw) To: Emily Peri, Praveen Kumar; +Cc: gregkh, outreachy, linux-staging, linux-kernel On 10/20/22 07:05, Emily Peri wrote: >>> >>> *pold_state = networktype; >>> >>> @@ -367,7 +368,7 @@ u8 rtw_set_802_11_disassociate(struct adapter *padapter) >>> >>> u8 rtw_set_802_11_bssid_list_scan(struct adapter *padapter, struct ndis_802_11_ssid *pssid, int ssid_max_num) >>> { >>> - struct mlme_priv *pmlmepriv = &padapter->mlmepriv; >>> + struct mlme_priv *pmlmepriv = &padapter->mlmepriv; >>> u8 res = true; >>> >>> if (!padapter) { >> >> Regards, >> >> ~Praveen. > > Now that I think about-- in this last edit I removed a space to appease > checkpatch, but maybe I actually should have removed the tab? Hi Emily, you are right, just use spaces. struct mlme_priv *pmlmepriv = &padapter->mlmepriv; thanks, Michael ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] staging: rtl8723bs: fix white space warnings 2022-10-20 2:10 ` [PATCH 1/3] staging: rtl8723bs: fix white space warnings Emily Peri 2022-10-20 4:04 ` Praveen Kumar @ 2022-10-20 4:57 ` Greg KH 2022-10-20 5:28 ` Emily Peri 1 sibling, 1 reply; 10+ messages in thread From: Greg KH @ 2022-10-20 4:57 UTC (permalink / raw) To: Emily Peri; +Cc: outreachy, linux-staging, linux-kernel On Wed, Oct 19, 2022 at 07:10:51PM -0700, Emily Peri wrote: > Fix the following checkpatch warnings in rtw_ioctl_set: > 1) Add missing blankline after declaration > 2) Replace spaces used for indent with tab > 3) Remove space before tab When you have to list the different things you do, you should really break that up into individual patches. This should be 3. thanks, greg k-h ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] staging: rtl8723bs: fix white space warnings 2022-10-20 4:57 ` Greg KH @ 2022-10-20 5:28 ` Emily Peri 2022-10-20 7:07 ` Greg KH 0 siblings, 1 reply; 10+ messages in thread From: Emily Peri @ 2022-10-20 5:28 UTC (permalink / raw) To: Greg KH; +Cc: outreachy, linux-staging, linux-kernel On Thu, Oct 20, 2022 at 06:57:03AM +0200, Greg KH wrote: > On Wed, Oct 19, 2022 at 07:10:51PM -0700, Emily Peri wrote: > > Fix the following checkpatch warnings in rtw_ioctl_set: > > 1) Add missing blankline after declaration > > 2) Replace spaces used for indent with tab > > 3) Remove space before tab > > When you have to list the different things you do, you should really > break that up into individual patches. This should be 3. > > thanks, > > greg k-h Thanks greg for the feedback! I got the idea to put them all in the same patch from the PatchPhilosophy guide, which suggested combining a bunch of white space corrections into one commit. But, it sounds like I misinterpreted what the guide was saying. Maybe if there are identical warnings, such as "trailing white space" in the driver code, could those corrections go together in a single patch? ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] staging: rtl8723bs: fix white space warnings 2022-10-20 5:28 ` Emily Peri @ 2022-10-20 7:07 ` Greg KH 0 siblings, 0 replies; 10+ messages in thread From: Greg KH @ 2022-10-20 7:07 UTC (permalink / raw) To: Emily Peri; +Cc: outreachy, linux-staging, linux-kernel On Wed, Oct 19, 2022 at 10:28:42PM -0700, Emily Peri wrote: > On Thu, Oct 20, 2022 at 06:57:03AM +0200, Greg KH wrote: > > On Wed, Oct 19, 2022 at 07:10:51PM -0700, Emily Peri wrote: > > > Fix the following checkpatch warnings in rtw_ioctl_set: > > > 1) Add missing blankline after declaration > > > 2) Replace spaces used for indent with tab > > > 3) Remove space before tab > > > > When you have to list the different things you do, you should really > > break that up into individual patches. This should be 3. > > > > thanks, > > > > greg k-h > > Thanks greg for the feedback! I got the idea to put them all in the same > patch from the PatchPhilosophy guide, which suggested combining a bunch > of white space corrections into one commit. But, it sounds like I > misinterpreted what the guide was saying. Maybe if there are identical > warnings, such as "trailing white space" in the driver code, could those > corrections go together in a single patch? > Yes, you are correct, you could do a "fix all trailing whitespace in this file" in one patch. thanks, greg k-h ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 2/3] staging: rtl8723bs: align block comment stars 2022-10-20 2:10 [PATCH 0/3] staging: rtl8723bs: clean up driver code in rtw_ioctl_set Emily Peri 2022-10-20 2:10 ` [PATCH 1/3] staging: rtl8723bs: fix white space warnings Emily Peri @ 2022-10-20 2:10 ` Emily Peri 2022-10-20 2:10 ` [PATCH 3/3] staging: rtl8723bs: remove unnecessary parenthesis Emily Peri 2 siblings, 0 replies; 10+ messages in thread From: Emily Peri @ 2022-10-20 2:10 UTC (permalink / raw) To: gregkh, outreachy, linux-staging, linux-kernel; +Cc: Emily Peri Align '*' on each line of block comment in rtw_ioctl_set. Issue found by checkpatch. Signed-off-by: Emily Peri <eperi1024@gmail.com> --- drivers/staging/rtl8723bs/core/rtw_ioctl_set.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c index 354e61a8f2bd..ac957035bf1a 100644 --- a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c +++ b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c @@ -463,11 +463,11 @@ u8 rtw_set_802_11_add_wep(struct adapter *padapter, struct ndis_802_11_wep *wep) } /* -* rtw_get_cur_max_rate - -* @adapter: pointer to struct adapter structure -* -* Return 0 or 100Kbps -*/ + * rtw_get_cur_max_rate - + * @adapter: pointer to struct adapter structure + * + * Return 0 or 100Kbps + */ u16 rtw_get_cur_max_rate(struct adapter *adapter) { int i = 0; -- 2.34.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 3/3] staging: rtl8723bs: remove unnecessary parenthesis 2022-10-20 2:10 [PATCH 0/3] staging: rtl8723bs: clean up driver code in rtw_ioctl_set Emily Peri 2022-10-20 2:10 ` [PATCH 1/3] staging: rtl8723bs: fix white space warnings Emily Peri 2022-10-20 2:10 ` [PATCH 2/3] staging: rtl8723bs: align block comment stars Emily Peri @ 2022-10-20 2:10 ` Emily Peri 2 siblings, 0 replies; 10+ messages in thread From: Emily Peri @ 2022-10-20 2:10 UTC (permalink / raw) To: gregkh, outreachy, linux-staging, linux-kernel; +Cc: Emily Peri Remove extra parenthesis in conditional statement in rtw_ioctl_set. Issue found by checkpatch. Signed-off-by: Emily Peri <eperi1024@gmail.com> --- drivers/staging/rtl8723bs/core/rtw_ioctl_set.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c index ac957035bf1a..32194dabd587 100644 --- a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c +++ b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c @@ -160,7 +160,7 @@ u8 rtw_set_802_11_ssid(struct adapter *padapter, struct ndis_802_11_ssid *ssid) if (check_fwstate(pmlmepriv, _FW_LINKED|WIFI_ADHOC_MASTER_STATE) == true) { if ((pmlmepriv->assoc_ssid.ssid_length == ssid->ssid_length) && (!memcmp(&pmlmepriv->assoc_ssid.ssid, ssid->ssid, ssid->ssid_length))) { - if ((check_fwstate(pmlmepriv, WIFI_STATION_STATE) == false)) { + if (check_fwstate(pmlmepriv, WIFI_STATION_STATE) == false) { if (rtw_is_same_ibss(padapter, pnetwork) == false) { /* if in WIFI_ADHOC_MASTER_STATE | WIFI_ADHOC_STATE, create bss or rejoin again */ rtw_disassoc_cmd(padapter, 0, true); -- 2.34.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-10-20 7:07 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-10-20 2:10 [PATCH 0/3] staging: rtl8723bs: clean up driver code in rtw_ioctl_set Emily Peri 2022-10-20 2:10 ` [PATCH 1/3] staging: rtl8723bs: fix white space warnings Emily Peri 2022-10-20 4:04 ` Praveen Kumar 2022-10-20 5:05 ` Emily Peri 2022-10-20 5:59 ` Michael Straube 2022-10-20 4:57 ` Greg KH 2022-10-20 5:28 ` Emily Peri 2022-10-20 7:07 ` Greg KH 2022-10-20 2:10 ` [PATCH 2/3] staging: rtl8723bs: align block comment stars Emily Peri 2022-10-20 2:10 ` [PATCH 3/3] staging: rtl8723bs: remove unnecessary parenthesis Emily Peri
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.