2012-11-16 12:51:51

by Mohammed Shafi Shajakhan

[permalink] [raw]
Subject: [PATCH 1/8] ath6kl: Remove erroneous flag clearing

From: Mohammed Shafi Shajakhan <[email protected]>

WLAN_ENABLED is vif specific, not part of
the driver's struct ath6kl. Proper clearing
of this flag is already taken care in
ath6kl_cleanup_vif.

Cc: wei-jen jlin <[email protected]>
Signed-off-by: Mohammed Shafi Shajakhan <[email protected]>
---
drivers/net/wireless/ath/ath6kl/init.c | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/ath/ath6kl/init.c b/drivers/net/wireless/ath/ath6kl/init.c
index 424676e..4fddf12 100644
--- a/drivers/net/wireless/ath/ath6kl/init.c
+++ b/drivers/net/wireless/ath/ath6kl/init.c
@@ -1798,8 +1798,6 @@ void ath6kl_stop_txrx(struct ath6kl *ar)
"attempting to reset target on instance destroy\n");
ath6kl_reset_device(ar, ar->target_type, true, true);

- clear_bit(WLAN_ENABLED, &ar->flag);
-
up(&ar->sem);
}
EXPORT_SYMBOL(ath6kl_stop_txrx);
--
1.7.0.4



2012-11-27 19:45:51

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH 1/8] ath6kl: Remove erroneous flag clearing

On 11/16/2012 02:51 PM, Mohammed Shafi Shajakhan wrote:
> From: Mohammed Shafi Shajakhan <[email protected]>
>
> WLAN_ENABLED is vif specific, not part of
> the driver's struct ath6kl. Proper clearing
> of this flag is already taken care in
> ath6kl_cleanup_vif.
>
> Cc: wei-jen jlin <[email protected]>
> Signed-off-by: Mohammed Shafi Shajakhan <[email protected]>

Thanks, all eight patches applied.

Kalle