All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] ath9k: wake up chip before initializing PAPRD table
@ 2011-05-24  9:26 Mohammed Shafi Shajakhan
  2011-05-24  9:40 ` Mohammed Shafi
  0 siblings, 1 reply; 2+ messages in thread
From: Mohammed Shafi Shajakhan @ 2011-05-24  9:26 UTC (permalink / raw)
  To: linville, Senthilkumar.Balasubramanian
  Cc: linux-wireless, lrodriguez, Mohammed Shafi Shajakhan

From: Mohammed Shafi Shajakhan <mshajakhan@atheros.com>

In PAPRD table initialization path we do some register read, so
make sure the chip is awake during that. Currently PAPRD is disabled.

Signed-off-by: Mohammed Shafi Shajakhan <mshajakhan@atheros.com>
---
 drivers/net/wireless/ath/ath9k/main.c |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c
index a198ee3..4a71f4e 100644
--- a/drivers/net/wireless/ath/ath9k/main.c
+++ b/drivers/net/wireless/ath/ath9k/main.c
@@ -394,8 +394,10 @@ void ath_paprd_calibrate(struct work_struct *work)
 	if (!caldata)
 		return;
 
+	ath9k_ps_wakeup(sc);
+
 	if (ar9003_paprd_init_table(ah) < 0)
-		return;
+		goto fail_paprd;
 
 	skb = alloc_skb(len, GFP_KERNEL);
 	if (!skb)
@@ -411,7 +413,6 @@ void ath_paprd_calibrate(struct work_struct *work)
 	memcpy(hdr->addr2, hw->wiphy->perm_addr, ETH_ALEN);
 	memcpy(hdr->addr3, hw->wiphy->perm_addr, ETH_ALEN);
 
-	ath9k_ps_wakeup(sc);
 	for (chain = 0; chain < AR9300_MAX_CHAINS; chain++) {
 		if (!(common->tx_chainmask & BIT(chain)))
 			continue;
-- 
1.7.0.4


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH v2] ath9k: wake up chip before initializing PAPRD table
  2011-05-24  9:26 [PATCH v2] ath9k: wake up chip before initializing PAPRD table Mohammed Shafi Shajakhan
@ 2011-05-24  9:40 ` Mohammed Shafi
  0 siblings, 0 replies; 2+ messages in thread
From: Mohammed Shafi @ 2011-05-24  9:40 UTC (permalink / raw)
  To: linux-wireless
  Cc: linville, Senthilkumar Balasubramanian, Vivek Natarajan, Luis Rodriguez

On Tuesday 24 May 2011 02:56 PM, Mohammed Shajakhan wrote:
> From: Mohammed Shafi Shajakhan<mshajakhan@atheros.com>
>
> In PAPRD table initialization path we do some register read, so
> make sure the chip is awake during that. Currently PAPRD is disabled.
>
> Signed-off-by: Mohammed Shafi Shajakhan<mshajakhan@atheros.com>
> ---
>   drivers/net/wireless/ath/ath9k/main.c |    5 +++--
>   1 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c
> index a198ee3..4a71f4e 100644
> --- a/drivers/net/wireless/ath/ath9k/main.c
> +++ b/drivers/net/wireless/ath/ath9k/main.c
> @@ -394,8 +394,10 @@ void ath_paprd_calibrate(struct work_struct *work)
>   	if (!caldata)
>   		return;
>
> +	ath9k_ps_wakeup(sc);
> +
>   	if (ar9003_paprd_init_table(ah)<  0)
> -		return;
> +		goto fail_paprd;
>
>   	skb = alloc_skb(len, GFP_KERNEL);
>   	if (!skb)

need to handle this case of skb_allocation failure :(, will send a v3. 
thanks to Vivek.

> @@ -411,7 +413,6 @@ void ath_paprd_calibrate(struct work_struct *work)
>   	memcpy(hdr->addr2, hw->wiphy->perm_addr, ETH_ALEN);
>   	memcpy(hdr->addr3, hw->wiphy->perm_addr, ETH_ALEN);
>
> -	ath9k_ps_wakeup(sc);
>   	for (chain = 0; chain<  AR9300_MAX_CHAINS; chain++) {
>   		if (!(common->tx_chainmask&  BIT(chain)))
>   			continue;

-- 
thanks,
shafi

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-05-24  9:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-24  9:26 [PATCH v2] ath9k: wake up chip before initializing PAPRD table Mohammed Shafi Shajakhan
2011-05-24  9:40 ` Mohammed Shafi

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.