* [Outreachy kernel][PATCH] staging: rtlwifi: Unneeded function halbtc_get_wifi_rssi() @ 2018-10-22 10:27 Bhanusree Pola 2018-10-22 14:02 ` Vaishali Thakkar 0 siblings, 1 reply; 4+ messages in thread From: Bhanusree Pola @ 2018-10-22 10:27 UTC (permalink / raw) To: outreachy-kernel; +Cc: Greg KH In the function halbtc_get_wifi_rssi: if and else are functionally identical. Unneeded return variable undec_sm_pwdb. Function not needed and so assigned value rtlpriv->dm.undec_sm_pwdb where the function halbtc_get_wifi_rssi() is called. Signed-off-by: Bhanusree Pola <bhanusreemahesh@gmail.com> --- drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c b/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c index 24e19ff..b519d18 100644 --- a/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c +++ b/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c @@ -476,17 +476,6 @@ static u32 halbtc_get_wifi_link_status(struct btc_coexist *btcoexist) return ret_val; } -static s32 halbtc_get_wifi_rssi(struct rtl_priv *rtlpriv) -{ - int undec_sm_pwdb = 0; - - if (rtlpriv->mac80211.link_state >= MAC80211_LINKED) - undec_sm_pwdb = rtlpriv->dm.undec_sm_pwdb; - else /* associated entry pwdb */ - undec_sm_pwdb = rtlpriv->dm.undec_sm_pwdb; - return undec_sm_pwdb; -} - static bool halbtc_get(void *void_btcoexist, u8 get_type, void *out_buf) { struct btc_coexist *btcoexist = (struct btc_coexist *)void_btcoexist; @@ -585,7 +574,7 @@ static bool halbtc_get(void *void_btcoexist, u8 get_type, void *out_buf) *bool_tmp = false; break; case BTC_GET_S4_WIFI_RSSI: - *s32_tmp = halbtc_get_wifi_rssi(rtlpriv); + *s32_tmp = rtlpriv->dm.undec_sm_pwdb; break; case BTC_GET_S4_HS_RSSI: *s32_tmp = 0; -- 2.7.4 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Outreachy kernel][PATCH] staging: rtlwifi: Unneeded function halbtc_get_wifi_rssi() 2018-10-22 10:27 [Outreachy kernel][PATCH] staging: rtlwifi: Unneeded function halbtc_get_wifi_rssi() Bhanusree Pola @ 2018-10-22 14:02 ` Vaishali Thakkar 2018-10-22 14:31 ` Bhanusree Mahesh 0 siblings, 1 reply; 4+ messages in thread From: Vaishali Thakkar @ 2018-10-22 14:02 UTC (permalink / raw) To: bhanusreemahesh; +Cc: outreachy-kernel, Greg KH On Mon, Oct 22, 2018 at 3:59 PM Bhanusree Pola <bhanusreemahesh@gmail.com> wrote: > > In the function halbtc_get_wifi_rssi: > > if and else are functionally identical. > Unneeded return variable undec_sm_pwdb. > Function not needed and so assigned value rtlpriv->dm.undec_sm_pwdb where > the function halbtc_get_wifi_rssi() is called. Hi Bhanusree, The subject line is not clear here. It should be something like 'Remove unneeded function ... '. Always write what patch is doing rather than just mentioning keywords. Similarly for the commit log, I feel that it can improved a bit with have proper sentences and structure. Probably something like: 'In function x, <insert_reason_one> and <insert_reason_two>. So, function is not needed. And then <insert_what_patch_is_doing>. > Signed-off-by: Bhanusree Pola <bhanusreemahesh@gmail.com> > --- > drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c | 13 +------------ > 1 file changed, 1 insertion(+), 12 deletions(-) > > diff --git a/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c b/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c > index 24e19ff..b519d18 100644 > --- a/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c > +++ b/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c > @@ -476,17 +476,6 @@ static u32 halbtc_get_wifi_link_status(struct btc_coexist *btcoexist) > return ret_val; > } > > -static s32 halbtc_get_wifi_rssi(struct rtl_priv *rtlpriv) > -{ > - int undec_sm_pwdb = 0; > - > - if (rtlpriv->mac80211.link_state >= MAC80211_LINKED) > - undec_sm_pwdb = rtlpriv->dm.undec_sm_pwdb; > - else /* associated entry pwdb */ > - undec_sm_pwdb = rtlpriv->dm.undec_sm_pwdb; > - return undec_sm_pwdb; > -} > - > static bool halbtc_get(void *void_btcoexist, u8 get_type, void *out_buf) > { > struct btc_coexist *btcoexist = (struct btc_coexist *)void_btcoexist; > @@ -585,7 +574,7 @@ static bool halbtc_get(void *void_btcoexist, u8 get_type, void *out_buf) > *bool_tmp = false; > break; > case BTC_GET_S4_WIFI_RSSI: > - *s32_tmp = halbtc_get_wifi_rssi(rtlpriv); > + *s32_tmp = rtlpriv->dm.undec_sm_pwdb; > break; > case BTC_GET_S4_HS_RSSI: > *s32_tmp = 0; > -- > 2.7.4 > > -- > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. > To post to this group, send email to outreachy-kernel@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20181022102752.GA12423%40aakashd-VirtualBox. > For more options, visit https://groups.google.com/d/optout. ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Outreachy kernel][PATCH] staging: rtlwifi: Unneeded function halbtc_get_wifi_rssi() 2018-10-22 14:02 ` Vaishali Thakkar @ 2018-10-22 14:31 ` Bhanusree Mahesh 2018-10-22 14:32 ` Julia Lawall 0 siblings, 1 reply; 4+ messages in thread From: Bhanusree Mahesh @ 2018-10-22 14:31 UTC (permalink / raw) To: Vaishali Thakkar; +Cc: outreachy-kernel, Greg Kroah-Hartman [-- Attachment #1: Type: text/plain, Size: 3178 bytes --] Ok, will do that. As I was asked to be imperative I did not elaborate the commit log. Will change it and send. Bhanusree On Mon, 22 Oct 2018 19:32 Vaishali Thakkar, <vthakkar@vaishalithakkar.in> wrote: > On Mon, Oct 22, 2018 at 3:59 PM Bhanusree Pola > <bhanusreemahesh@gmail.com> wrote: > > > > In the function halbtc_get_wifi_rssi: > > > > if and else are functionally identical. > > Unneeded return variable undec_sm_pwdb. > > Function not needed and so assigned value rtlpriv->dm.undec_sm_pwdb where > > the function halbtc_get_wifi_rssi() is called. > > Hi Bhanusree, > > The subject line is not clear here. It should be something like > 'Remove unneeded function ... '. Always write what patch is > doing rather than just mentioning keywords. > > Similarly for the commit log, I feel that it can improved a bit with > have proper sentences and structure. Probably something like: > > 'In function x, <insert_reason_one> and <insert_reason_two>. > So, function is not needed. And then <insert_what_patch_is_doing>. > > > Signed-off-by: Bhanusree Pola <bhanusreemahesh@gmail.com> > > --- > > drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c | 13 +------------ > > 1 file changed, 1 insertion(+), 12 deletions(-) > > > > diff --git a/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c > b/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c > > index 24e19ff..b519d18 100644 > > --- a/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c > > +++ b/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c > > @@ -476,17 +476,6 @@ static u32 halbtc_get_wifi_link_status(struct > btc_coexist *btcoexist) > > return ret_val; > > } > > > > -static s32 halbtc_get_wifi_rssi(struct rtl_priv *rtlpriv) > > -{ > > - int undec_sm_pwdb = 0; > > - > > - if (rtlpriv->mac80211.link_state >= MAC80211_LINKED) > > - undec_sm_pwdb = rtlpriv->dm.undec_sm_pwdb; > > - else /* associated entry pwdb */ > > - undec_sm_pwdb = rtlpriv->dm.undec_sm_pwdb; > > - return undec_sm_pwdb; > > -} > > - > > static bool halbtc_get(void *void_btcoexist, u8 get_type, void *out_buf) > > { > > struct btc_coexist *btcoexist = (struct btc_coexist > *)void_btcoexist; > > @@ -585,7 +574,7 @@ static bool halbtc_get(void *void_btcoexist, u8 > get_type, void *out_buf) > > *bool_tmp = false; > > break; > > case BTC_GET_S4_WIFI_RSSI: > > - *s32_tmp = halbtc_get_wifi_rssi(rtlpriv); > > + *s32_tmp = rtlpriv->dm.undec_sm_pwdb; > > break; > > case BTC_GET_S4_HS_RSSI: > > *s32_tmp = 0; > > -- > > 2.7.4 > > > > -- > > You received this message because you are subscribed to the Google > Groups "outreachy-kernel" group. > > To unsubscribe from this group and stop receiving emails from it, send > an email to outreachy-kernel+unsubscribe@googlegroups.com. > > To post to this group, send email to outreachy-kernel@googlegroups.com. > > To view this discussion on the web visit > https://groups.google.com/d/msgid/outreachy-kernel/20181022102752.GA12423%40aakashd-VirtualBox > . > > For more options, visit https://groups.google.com/d/optout. > [-- Attachment #2: Type: text/html, Size: 4645 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Outreachy kernel][PATCH] staging: rtlwifi: Unneeded function halbtc_get_wifi_rssi() 2018-10-22 14:31 ` Bhanusree Mahesh @ 2018-10-22 14:32 ` Julia Lawall 0 siblings, 0 replies; 4+ messages in thread From: Julia Lawall @ 2018-10-22 14:32 UTC (permalink / raw) To: Bhanusree Mahesh; +Cc: Vaishali Thakkar, outreachy-kernel, Greg Kroah-Hartman [-- Attachment #1: Type: text/plain, Size: 4411 bytes --] On Mon, 22 Oct 2018, Bhanusree Mahesh wrote: > Ok, will do that. As I was asked to be imperative I did not elaborate the > commit log. Will change it and send. Remember not to top post as well :) julia > Bhanusree > > > > On Mon, 22 Oct 2018 19:32 Vaishali Thakkar, <vthakkar@vaishalithakkar.in> > wrote: > On Mon, Oct 22, 2018 at 3:59 PM Bhanusree Pola > <bhanusreemahesh@gmail.com> wrote: > > > > In the function halbtc_get_wifi_rssi: > > > > if and else are functionally identical. > > Unneeded return variable undec_sm_pwdb. > > Function not needed and so assigned value > rtlpriv->dm.undec_sm_pwdb where > > the function halbtc_get_wifi_rssi() is called. > > Hi Bhanusree, > > The subject line is not clear here. It should be something like > 'Remove unneeded function ... '. Always write what patch is > doing rather than just mentioning keywords. > > Similarly for the commit log, I feel that it can improved a bit > with > have proper sentences and structure. Probably something like: > > 'In function x, <insert_reason_one> and <insert_reason_two>. > So, function is not needed. And then > <insert_what_patch_is_doing>. > > > Signed-off-by: Bhanusree Pola <bhanusreemahesh@gmail.com> > > --- > > drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c | 13 > +------------ > > 1 file changed, 1 insertion(+), 12 deletions(-) > > > > diff --git a/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c > b/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c > > index 24e19ff..b519d18 100644 > > --- a/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c > > +++ b/drivers/staging/rtlwifi/btcoexist/halbtcoutsrc.c > > @@ -476,17 +476,6 @@ static u32 > halbtc_get_wifi_link_status(struct btc_coexist *btcoexist) > > return ret_val; > > } > > > > -static s32 halbtc_get_wifi_rssi(struct rtl_priv *rtlpriv) > > -{ > > - int undec_sm_pwdb = 0; > > - > > - if (rtlpriv->mac80211.link_state >= MAC80211_LINKED) > > - undec_sm_pwdb = rtlpriv->dm.undec_sm_pwdb; > > - else /* associated entry pwdb */ > > - undec_sm_pwdb = rtlpriv->dm.undec_sm_pwdb; > > - return undec_sm_pwdb; > > -} > > - > > static bool halbtc_get(void *void_btcoexist, u8 get_type, > void *out_buf) > > { > > struct btc_coexist *btcoexist = (struct btc_coexist > *)void_btcoexist; > > @@ -585,7 +574,7 @@ static bool halbtc_get(void > *void_btcoexist, u8 get_type, void *out_buf) > > *bool_tmp = false; > > break; > > case BTC_GET_S4_WIFI_RSSI: > > - *s32_tmp = halbtc_get_wifi_rssi(rtlpriv); > > + *s32_tmp = rtlpriv->dm.undec_sm_pwdb; > > break; > > case BTC_GET_S4_HS_RSSI: > > *s32_tmp = 0; > > -- > > 2.7.4 > > > > -- > > You received this message because you are subscribed to the > Google Groups "outreachy-kernel" group. > > To unsubscribe from this group and stop receiving emails from > it, send an email to > outreachy-kernel+unsubscribe@googlegroups.com. > > To post to this group, send email to > outreachy-kernel@googlegroups.com. > > To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/20181022102752.GA12423%4 > 0aakashd-VirtualBox. > > For more options, visit https://groups.google.com/d/optout. > > -- > You received this message because you are subscribed to the Google Groups > "outreachy-kernel" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to outreachy-kernel+unsubscribe@googlegroups.com. > To post to this group, send email to outreachy-kernel@googlegroups.com. > To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/CAB0Mk-NsLvS5fn8Hi58G1Y4 > ykVRquWqp1ybU%3DYw6A1fvtSLZYg%40mail.gmail.com. > For more options, visit https://groups.google.com/d/optout. > > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-10-22 14:32 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-10-22 10:27 [Outreachy kernel][PATCH] staging: rtlwifi: Unneeded function halbtc_get_wifi_rssi() Bhanusree Pola 2018-10-22 14:02 ` Vaishali Thakkar 2018-10-22 14:31 ` Bhanusree Mahesh 2018-10-22 14:32 ` Julia Lawall
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.