Commit 824f830f authored by Jérôme Pouiller's avatar Jérôme Pouiller Committed by Greg Kroah-Hartman
Browse files

staging: wfx: drop useless wfx_fwd_probe_req() 



wfx_fwd_probe_req() is a function of two lines called from only one
place. In order to uniformize all filtering functions, drop
wfx_fwd_probe_req().

Signed-off-by: default avatarJérôme Pouiller <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20200415161147.69738-12-Jerome.Pouiller@silabs.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent ed4a707d

drivers/staging/wfx/sta.c

+5 −9
Original line number Diff line number Diff line
@@ -113,13 +113,6 @@ void wfx_cqm_bssloss_sm(struct wfx_vif *wvif, int init, int good, int bad) 
	mutex_unlock(&wvif->bss_loss_lock);

}



int wfx_fwd_probe_req(struct wfx_vif *wvif, bool enable)

{

	wvif->fwd_probe_req = enable;

	return hif_set_rx_filter(wvif, wvif->filter_bssid,

				 wvif->fwd_probe_req);

}



void wfx_update_filtering(struct wfx_vif *wvif)

{

	int i;
@@ -249,9 +242,12 @@ void wfx_configure_filter(struct ieee80211_hw *hw, 
		}



		if (*total_flags & FIF_PROBE_REQ)

			wfx_fwd_probe_req(wvif, true);

			wvif->fwd_probe_req = true;

		else

			wfx_fwd_probe_req(wvif, false);

			wvif->fwd_probe_req = false;

		hif_set_rx_filter(wvif, wvif->filter_bssid,

				  wvif->fwd_probe_req);



		mutex_unlock(&wvif->scan_lock);

	}

	mutex_unlock(&wdev->conf_mutex);

drivers/staging/wfx/sta.h

+0 −1
Original line number Diff line number Diff line
@@ -88,7 +88,6 @@ void wfx_suspend_resume_mc(struct wfx_vif *wvif, enum sta_notify_cmd cmd); 


// Other Helpers

void wfx_cqm_bssloss_sm(struct wfx_vif *wvif, int init, int good, int bad);

int wfx_fwd_probe_req(struct wfx_vif *wvif, bool enable);

u32 wfx_rate_mask_to_hw(struct wfx_dev *wdev, u32 rates);



#endif /* WFX_STA_H */