diff options
author | Bartosz Markowski <bartosz.markowski@tieto.com> | 2012-02-29 15:14:27 +0100 |
---|---|---|
committer | Philippe Langlais <philippe.langlais@stericsson.com> | 2012-05-22 11:06:38 +0200 |
commit | f7d39df3167cb443e5e3ed319266abac4d2ed845 (patch) | |
tree | 841c7a1dc401ac1aee15e02b70679ebe084724ac | |
parent | d23d0b764a2081af18cd9be4b35626fdd310582c (diff) |
cw1200: Do not send BSS params if we are not associated
Connection parameters must be set only if we are
associated. This is firmware requirement. Otherwise
WLAN device connot operate correctly.
ST-Ericsson ID: 361264
Change-Id: I13b34a641b6cfd58d2e599475669700973e6bfba
Signed-off-by: Bartosz Markowski <bartosz.markowski@tieto.com>
Reviewed-on: http://gerrit.lud.stericsson.com/gerrit/36721
Reviewed-by: QABUILD
-rwxr-xr-x | drivers/staging/cw1200/ap.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/staging/cw1200/ap.c b/drivers/staging/cw1200/ap.c index f09d9031d0f..f328cff389e 100755 --- a/drivers/staging/cw1200/ap.c +++ b/drivers/staging/cw1200/ap.c @@ -566,8 +566,13 @@ void cw1200_bss_info_changed(struct ieee80211_hw *dev, priv->cqm_beacon_loss_count ? priv->cqm_beacon_loss_count : priv->cqm_link_loss_count; - WARN_ON(wsm_set_bss_params(priv, &priv->bss_params)); - priv->setbssparams_done = true; + /* Make sure we are associated before sending + * set_bss_params to firmware */ + if (priv->bss_params.aid) { + WARN_ON(wsm_set_bss_params(priv, + &priv->bss_params)); + priv->setbssparams_done = true; + } } #endif /* CONFIG_CW1200_USE_STE_EXTENSIONS */ } |