[PATCH 1/3] ath9k: configure the beacon only if the STA is associated

[PATCH 1/3] ath9k: configure the beacon only if the STA is associated

[Senthil Balasubramanian]

beacons configuration SHOULD be done only if the STA is associated.

Signed-off-by: Senthil Balasubramanian <senthilkumar@atheros.com>
---
 drivers/net/wireless/ath/ath9k/beacon.c |    8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/drivers/net/wireless/ath/ath9k/beacon.c b/drivers/net/wireless/ath/ath9k/beacon.c
index 422454f..d088ebf 100644
--- a/drivers/net/wireless/ath/ath9k/beacon.c
+++ b/drivers/net/wireless/ath/ath9k/beacon.c
@@ -577,6 +577,13 @@ static void ath_beacon_config_sta(struct ath_softc *sc,
 	u64 tsf;
 	int num_beacons, offset, dtim_dec_count, cfp_dec_count;

+	/* No need to configure beacon if we are not associated */
+	if (!common->curaid) {
+		ath_print(common, ATH_DBG_BEACON,
+			 "STA is not yet associated..skipping beacon config\n");
+		return;
+	}
+
 	memset(&bs, 0, sizeof(bs));
 	intval = conf->beacon_interval & ATH9K_BEACON_PERIOD;

@@ -739,7 +746,6 @@ void ath_beacon_config(struct ath_softc *sc, struct ieee80211_vif *vif)
 	enum nl80211_iftype iftype;

 	/* Setup the beacon configuration parameters */
-
 	if (vif) {
 		struct ieee80211_bss_conf *bss_conf = &vif->bss_conf;

--
1.6.6.1

Re: [PATCH 1/3] ath9k: configure the beacon only if the STA is associated

[Luis R. Rodriguez]

On Wed, Feb 3, 2010 at 9:20 AM, Senthil Balasubramanian
<senthilkumar@atheros.com> wrote:
> beacons configuration SHOULD be done only if the STA is associated.
>
> Signed-off-by: Senthil Balasubramanian <senthilkumar@atheros.com>

I believe this is Cc: stable@kernel.org

  Luis