diff options
author | Jérôme Pouiller <jerome.pouiller@silabs.com> | 2020-05-15 10:33:22 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-05-15 16:00:54 +0200 |
commit | a1ae4d07fd94da1b3f7597f1594af2e35f1065cc (patch) | |
tree | 42cb9ad125682e69352e11dfb3cbbf1e84a842fc /drivers/staging/wfx/sta.c | |
parent | ac0e50a5a3cb6b6c41c4e568144f780a89bd0d54 (diff) |
staging: wfx: fix error reporting in wfx_start_ap()
If AP did not start, the error was not reported to mac80211.
Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20200515083325.378539-17-Jerome.Pouiller@silabs.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/wfx/sta.c')
-rw-r--r-- | drivers/staging/wfx/sta.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/staging/wfx/sta.c b/drivers/staging/wfx/sta.c index 57304ed42e79..f448957c1a92 100644 --- a/drivers/staging/wfx/sta.c +++ b/drivers/staging/wfx/sta.c @@ -476,14 +476,17 @@ int wfx_start_ap(struct ieee80211_hw *hw, struct ieee80211_vif *vif) { struct wfx_vif *wvif = (struct wfx_vif *)vif->drv_priv; struct wfx_dev *wdev = wvif->wdev; + int ret; wvif = NULL; while ((wvif = wvif_iterate(wdev, wvif)) != NULL) wfx_update_pm(wvif); wvif = (struct wfx_vif *)vif->drv_priv; wfx_upload_ap_templates(wvif); - hif_start(wvif, &vif->bss_conf, wvif->channel); - return 0; + ret = hif_start(wvif, &vif->bss_conf, wvif->channel); + if (ret > 0) + return -EIO; + return ret; } void wfx_stop_ap(struct ieee80211_hw *hw, struct ieee80211_vif *vif) |