2009-08-21 20:53:37

by Joerg Albert

Subject: [PATCH 2/2 v2] ar9170: remove unnecessary call to ar9170_set_beacon_timers

Signed-off-by: Joerg Albert <[email protected]>

---
drivers/net/wireless/ath/ar9170/main.c | 6 ------
1 files changed, 0 insertions(+), 6 deletions(-)

diff --git a/drivers/net/wireless/ath/ar9170/main.c b/drivers/net/wireless/ath/ar9170/main.c
index 911e502..5b73194 100644
--- a/drivers/net/wireless/ath/ar9170/main.c
+++ b/drivers/net/wireless/ath/ar9170/main.c
@@ -2031,12 +2031,6 @@ static int ar9170_op_config(struct ieee80211_hw *hw, u32 changed)
goto out;
}

- if (changed & BSS_CHANGED_BEACON_INT) {
- err = ar9170_set_beacon_timers(ar);
- if (err)
- goto out;
- }
-
if (changed & IEEE80211_CONF_CHANGE_CHANNEL) {

/* adjust slot time for 5 GHz */
--
1.6.0.4

2009-08-21 21:24:42

by Christian Lamparter

Subject: Re: [PATCH 2/2 v2] ar9170: remove unnecessary call to ar9170_set_beacon_timers

On Friday 21 August 2009 22:53:37 Joerg Albert wrote:
>
> Signed-off-by: Joerg Albert <[email protected]>
Acked-by: Christian Lamparter <[email protected]>

> ---
> drivers/net/wireless/ath/ar9170/main.c | 6 ------
> 1 files changed, 0 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ar9170/main.c b/drivers/net/wireless/ath/ar9170/main.c
> index 911e502..5b73194 100644
> --- a/drivers/net/wireless/ath/ar9170/main.c
> +++ b/drivers/net/wireless/ath/ar9170/main.c
> @@ -2031,12 +2031,6 @@ static int ar9170_op_config(struct ieee80211_hw *hw, u32 changed)
> goto out;
> }
>
> - if (changed & BSS_CHANGED_BEACON_INT) {
> - err = ar9170_set_beacon_timers(ar);
> - if (err)
> - goto out;
> - }
> -
> if (changed & IEEE80211_CONF_CHANGE_CHANNEL) {
>
> /* adjust slot time for 5 GHz */
>