* [PATCH 1/2] staging: rtl8723bs: remove dead code, rtw_port_switch_chk()
@ 2018-12-10 9:52 Michael Straube
2018-12-10 9:52 ` [PATCH 2/2] staging: rtl8723bs: fix typo in the TODO file Michael Straube
2018-12-10 11:33 ` [PATCH 1/2] staging: rtl8723bs: remove dead code, rtw_port_switch_chk() Hans de Goede
0 siblings, 2 replies; 3+ messages in thread
From: Michael Straube @ 2018-12-10 9:52 UTC (permalink / raw)
To: gregkh
Cc: hadess, hdegoede, Larry.Finger, devel, linux-kernel, Michael Straube
The function rtw_port_switch_chk() returns always false.
Remove the function and related dead code.
Signed-off-by: Michael Straube <straube.linux@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 34 -------------------
.../staging/rtl8723bs/include/rtw_mlme_ext.h | 1 -
2 files changed, 35 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
index 8445d516c93d..35003d8d9ba4 100644
--- a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
+++ b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
@@ -5334,13 +5334,6 @@ void report_add_sta_event(struct adapter *padapter, unsigned char *MacAddr, int
return;
}
-
-bool rtw_port_switch_chk(struct adapter *adapter)
-{
- bool switch_needed = false;
- return switch_needed;
-}
-
/****************************************************************************
Following are the event callback functions
@@ -5414,7 +5407,6 @@ static void rtw_mlmeext_disconnect(struct adapter *padapter)
struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv;
struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);
struct wlan_bssid_ex *pnetwork = (struct wlan_bssid_ex *)(&(pmlmeinfo->network));
- u8 state_backup = (pmlmeinfo->state&0x03);
/* set_opmode_cmd(padapter, infra_client_with_mlme); */
@@ -5440,17 +5432,6 @@ static void rtw_mlmeext_disconnect(struct adapter *padapter)
pmlmeinfo->state = WIFI_FW_NULL_STATE;
- if (state_backup == WIFI_FW_STATION_STATE) {
- if (rtw_port_switch_chk(padapter)) {
- rtw_hal_set_hwreg(padapter, HW_VAR_PORT_SWITCH, NULL);
- {
- struct adapter *port0_iface = dvobj_get_port0_adapter(adapter_to_dvobj(padapter));
- if (port0_iface)
- rtw_lps_ctrl_wk_cmd(port0_iface, LPS_CTRL_CONNECT, 0);
- }
- }
- }
-
/* switch to the 20M Hz mode after disconnect */
pmlmeext->cur_bwmode = CHANNEL_WIDTH_20;
pmlmeext->cur_ch_offset = HAL_PRIME_CHNL_OFFSET_DONT_CARE;
@@ -5530,9 +5511,6 @@ void mlmeext_joinbss_event_callback(struct adapter *padapter, int join_res)
rtw_hal_macid_wakeup(padapter, psta->mac_id);
}
- if (rtw_port_switch_chk(padapter))
- rtw_hal_set_hwreg(padapter, HW_VAR_PORT_SWITCH, NULL);
-
join_type = 2;
rtw_hal_set_hwreg(padapter, HW_VAR_MLME_JOIN, (u8 *)(&join_type));
@@ -6087,18 +6065,6 @@ u8 setopmode_hdl(struct adapter *padapter, u8 *pbuf)
rtw_auto_ap_start_beacon(padapter);
#endif
- if (rtw_port_switch_chk(padapter)) {
- rtw_hal_set_hwreg(padapter, HW_VAR_PORT_SWITCH, NULL);
-
- if (psetop->mode == Ndis802_11APMode)
- adapter_to_pwrctl(padapter)->fw_psmode_iface_id = 0xff; /* ap mode won't dowload rsvd pages */
- else if (psetop->mode == Ndis802_11Infrastructure) {
- struct adapter *port0_iface = dvobj_get_port0_adapter(adapter_to_dvobj(padapter));
- if (port0_iface)
- rtw_lps_ctrl_wk_cmd(port0_iface, LPS_CTRL_CONNECT, 0);
- }
- }
-
if (psetop->mode == Ndis802_11APMode) {
/* Do this after port switch to */
/* prevent from downloading rsvd page to wrong port */
diff --git a/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h b/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
index 4c3141882143..f6eabad4bbe0 100644
--- a/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
+++ b/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
@@ -648,7 +648,6 @@ void report_survey_event(struct adapter *padapter, union recv_frame *precv_frame
void report_surveydone_event(struct adapter *padapter);
void report_del_sta_event(struct adapter *padapter, unsigned char* MacAddr, unsigned short reason);
void report_add_sta_event(struct adapter *padapter, unsigned char* MacAddr, int cam_idx);
-bool rtw_port_switch_chk(struct adapter *adapter);
void report_wmm_edca_update(struct adapter *padapter);
void beacon_timing_control(struct adapter *padapter);
--
2.19.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 2/2] staging: rtl8723bs: fix typo in the TODO file
2018-12-10 9:52 [PATCH 1/2] staging: rtl8723bs: remove dead code, rtw_port_switch_chk() Michael Straube
@ 2018-12-10 9:52 ` Michael Straube
2018-12-10 11:33 ` [PATCH 1/2] staging: rtl8723bs: remove dead code, rtw_port_switch_chk() Hans de Goede
1 sibling, 0 replies; 3+ messages in thread
From: Michael Straube @ 2018-12-10 9:52 UTC (permalink / raw)
To: gregkh
Cc: hadess, hdegoede, Larry.Finger, devel, linux-kernel, Michael Straube
HGz -> GHz
Signed-off-by: Michael Straube <straube.linux@gmail.com>
---
drivers/staging/rtl8723bs/TODO | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8723bs/TODO b/drivers/staging/rtl8723bs/TODO
index 80dbdaca3a8f..58e02f944b6d 100644
--- a/drivers/staging/rtl8723bs/TODO
+++ b/drivers/staging/rtl8723bs/TODO
@@ -1,6 +1,6 @@
TODO:
- find and remove code blocks guarded by never set CONFIG_FOO defines
-- find and remove remaining code valid only for 5 HGz. Most of the obvious
+- find and remove remaining code valid only for 5 GHz. Most of the obvious
ones have been removed, but things like channel > 14 still exist.
- find and remove any code for other chips that is left over
- convert any remaining unusual variable types
--
2.19.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/2] staging: rtl8723bs: remove dead code, rtw_port_switch_chk()
2018-12-10 9:52 [PATCH 1/2] staging: rtl8723bs: remove dead code, rtw_port_switch_chk() Michael Straube
2018-12-10 9:52 ` [PATCH 2/2] staging: rtl8723bs: fix typo in the TODO file Michael Straube
@ 2018-12-10 11:33 ` Hans de Goede
1 sibling, 0 replies; 3+ messages in thread
From: Hans de Goede @ 2018-12-10 11:33 UTC (permalink / raw)
To: Michael Straube, gregkh; +Cc: hadess, Larry.Finger, devel, linux-kernel
Hi,
Thank you for the patches.
On 10-12-18 10:52, Michael Straube wrote:
> The function rtw_port_switch_chk() returns always false.
> Remove the function and related dead code.
>
> Signed-off-by: Michael Straube <straube.linux@gmail.com>
Both patches look good to me:
Acked-by: Hans de Goede <hdegoede@redhat.com>
Regards,
Hans
> ---
> drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 34 -------------------
> .../staging/rtl8723bs/include/rtw_mlme_ext.h | 1 -
> 2 files changed, 35 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
> index 8445d516c93d..35003d8d9ba4 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
> @@ -5334,13 +5334,6 @@ void report_add_sta_event(struct adapter *padapter, unsigned char *MacAddr, int
> return;
> }
>
> -
> -bool rtw_port_switch_chk(struct adapter *adapter)
> -{
> - bool switch_needed = false;
> - return switch_needed;
> -}
> -
> /****************************************************************************
>
> Following are the event callback functions
> @@ -5414,7 +5407,6 @@ static void rtw_mlmeext_disconnect(struct adapter *padapter)
> struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv;
> struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);
> struct wlan_bssid_ex *pnetwork = (struct wlan_bssid_ex *)(&(pmlmeinfo->network));
> - u8 state_backup = (pmlmeinfo->state&0x03);
>
> /* set_opmode_cmd(padapter, infra_client_with_mlme); */
>
> @@ -5440,17 +5432,6 @@ static void rtw_mlmeext_disconnect(struct adapter *padapter)
>
> pmlmeinfo->state = WIFI_FW_NULL_STATE;
>
> - if (state_backup == WIFI_FW_STATION_STATE) {
> - if (rtw_port_switch_chk(padapter)) {
> - rtw_hal_set_hwreg(padapter, HW_VAR_PORT_SWITCH, NULL);
> - {
> - struct adapter *port0_iface = dvobj_get_port0_adapter(adapter_to_dvobj(padapter));
> - if (port0_iface)
> - rtw_lps_ctrl_wk_cmd(port0_iface, LPS_CTRL_CONNECT, 0);
> - }
> - }
> - }
> -
> /* switch to the 20M Hz mode after disconnect */
> pmlmeext->cur_bwmode = CHANNEL_WIDTH_20;
> pmlmeext->cur_ch_offset = HAL_PRIME_CHNL_OFFSET_DONT_CARE;
> @@ -5530,9 +5511,6 @@ void mlmeext_joinbss_event_callback(struct adapter *padapter, int join_res)
> rtw_hal_macid_wakeup(padapter, psta->mac_id);
> }
>
> - if (rtw_port_switch_chk(padapter))
> - rtw_hal_set_hwreg(padapter, HW_VAR_PORT_SWITCH, NULL);
> -
> join_type = 2;
> rtw_hal_set_hwreg(padapter, HW_VAR_MLME_JOIN, (u8 *)(&join_type));
>
> @@ -6087,18 +6065,6 @@ u8 setopmode_hdl(struct adapter *padapter, u8 *pbuf)
> rtw_auto_ap_start_beacon(padapter);
> #endif
>
> - if (rtw_port_switch_chk(padapter)) {
> - rtw_hal_set_hwreg(padapter, HW_VAR_PORT_SWITCH, NULL);
> -
> - if (psetop->mode == Ndis802_11APMode)
> - adapter_to_pwrctl(padapter)->fw_psmode_iface_id = 0xff; /* ap mode won't dowload rsvd pages */
> - else if (psetop->mode == Ndis802_11Infrastructure) {
> - struct adapter *port0_iface = dvobj_get_port0_adapter(adapter_to_dvobj(padapter));
> - if (port0_iface)
> - rtw_lps_ctrl_wk_cmd(port0_iface, LPS_CTRL_CONNECT, 0);
> - }
> - }
> -
> if (psetop->mode == Ndis802_11APMode) {
> /* Do this after port switch to */
> /* prevent from downloading rsvd page to wrong port */
> diff --git a/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h b/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
> index 4c3141882143..f6eabad4bbe0 100644
> --- a/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
> +++ b/drivers/staging/rtl8723bs/include/rtw_mlme_ext.h
> @@ -648,7 +648,6 @@ void report_survey_event(struct adapter *padapter, union recv_frame *precv_frame
> void report_surveydone_event(struct adapter *padapter);
> void report_del_sta_event(struct adapter *padapter, unsigned char* MacAddr, unsigned short reason);
> void report_add_sta_event(struct adapter *padapter, unsigned char* MacAddr, int cam_idx);
> -bool rtw_port_switch_chk(struct adapter *adapter);
> void report_wmm_edca_update(struct adapter *padapter);
>
> void beacon_timing_control(struct adapter *padapter);
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-12-10 11:33 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-10 9:52 [PATCH 1/2] staging: rtl8723bs: remove dead code, rtw_port_switch_chk() Michael Straube
2018-12-10 9:52 ` [PATCH 2/2] staging: rtl8723bs: fix typo in the TODO file Michael Straube
2018-12-10 11:33 ` [PATCH 1/2] staging: rtl8723bs: remove dead code, rtw_port_switch_chk() Hans de Goede
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).