* [PATCH] ath9k_hw: Fix ASPM L1 issue for AR9462
@ 2011-10-10 5:55 Rajkumar Manoharan
2011-10-12 18:40 ` John W. Linville
0 siblings, 1 reply; 4+ messages in thread
From: Rajkumar Manoharan @ 2011-10-10 5:55 UTC (permalink / raw)
To: linville; +Cc: linux-wireless, Rajkumar Manoharan, stable
Because of not clearing Bit 14 of AR_WA, the ASPM L1 is not
enabled when entering into sleep mode. AR9462 does not need
bit 14 to be set.
Cc: stable@kernel.org
Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
---
drivers/net/wireless/ath/ath9k/hw.c | 6 ++++--
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/ath/ath9k/hw.c b/drivers/net/wireless/ath/ath9k/hw.c
index c4ae0d1..255a538 100644
--- a/drivers/net/wireless/ath/ath9k/hw.c
+++ b/drivers/net/wireless/ath/ath9k/hw.c
@@ -542,6 +542,9 @@ static int __ath9k_hw_init(struct ath_hw *ah)
return -EIO;
}
+ if (AR_SREV_9462(ah))
+ ah->WARegVal &= ~AR_WA_D3_L1_DISABLE;
+
ath9k_hw_init_defaults(ah);
ath9k_hw_init_config(ah);
@@ -1830,8 +1833,7 @@ static void ath9k_set_power_sleep(struct ath_hw *ah, int setChip)
}
/* Clear Bit 14 of AR_WA after putting chip into Full Sleep mode. */
- if (!AR_SREV_9480(ah))
- REG_WRITE(ah, AR_WA, ah->WARegVal & ~AR_WA_D3_L1_DISABLE);
+ REG_WRITE(ah, AR_WA, ah->WARegVal & ~AR_WA_D3_L1_DISABLE);
}
/*
--
1.7.7
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] ath9k_hw: Fix ASPM L1 issue for AR9462
2011-10-10 5:55 [PATCH] ath9k_hw: Fix ASPM L1 issue for AR9462 Rajkumar Manoharan
@ 2011-10-12 18:40 ` John W. Linville
2011-10-12 18:50 ` John W. Linville
0 siblings, 1 reply; 4+ messages in thread
From: John W. Linville @ 2011-10-12 18:40 UTC (permalink / raw)
To: Rajkumar Manoharan; +Cc: linux-wireless, stable
On Mon, Oct 10, 2011 at 11:25:45AM +0530, Rajkumar Manoharan wrote:
> Because of not clearing Bit 14 of AR_WA, the ASPM L1 is not
> enabled when entering into sleep mode. AR9462 does not need
> bit 14 to be set.
>
> Cc: stable@kernel.org
> Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
CC [M] drivers/net/wireless/ath/ath9k/hw.o
drivers/net/wireless/ath/ath9k/hw.c: In function ‘__ath9k_hw_init’:
drivers/net/wireless/ath/ath9k/hw.c:544:2: error: implicit declaration of function ‘AR_SREV_9462’
Missing a patch dependency?
--
John W. Linville Someday the world will need a hero, and you
linville@tuxdriver.com might be all we have. Be ready.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ath9k_hw: Fix ASPM L1 issue for AR9462
2011-10-12 18:40 ` John W. Linville
@ 2011-10-12 18:50 ` John W. Linville
2011-10-13 3:46 ` Rajkumar Manoharan
0 siblings, 1 reply; 4+ messages in thread
From: John W. Linville @ 2011-10-12 18:50 UTC (permalink / raw)
To: Rajkumar Manoharan; +Cc: linux-wireless, stable
On Wed, Oct 12, 2011 at 02:40:44PM -0400, John W. Linville wrote:
> On Mon, Oct 10, 2011 at 11:25:45AM +0530, Rajkumar Manoharan wrote:
> > Because of not clearing Bit 14 of AR_WA, the ASPM L1 is not
> > enabled when entering into sleep mode. AR9462 does not need
> > bit 14 to be set.
> >
> > Cc: stable@kernel.org
> > Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
>
> CC [M] drivers/net/wireless/ath/ath9k/hw.o
> drivers/net/wireless/ath/ath9k/hw.c: In function ‘__ath9k_hw_init’:
> drivers/net/wireless/ath/ath9k/hw.c:544:2: error: implicit declaration of function ‘AR_SREV_9462’
>
> Missing a patch dependency?
I see that is defined by a series posted just before that -- one that
hasn't even appeared in wireless-next yet. Cc: stable@kernel.org
seems a bit inappropriate...
That series needs rework anyway -- how about you just fold this one
into that earlier series?
John
--
John W. Linville Someday the world will need a hero, and you
linville@tuxdriver.com might be all we have. Be ready.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ath9k_hw: Fix ASPM L1 issue for AR9462
2011-10-12 18:50 ` John W. Linville
@ 2011-10-13 3:46 ` Rajkumar Manoharan
0 siblings, 0 replies; 4+ messages in thread
From: Rajkumar Manoharan @ 2011-10-13 3:46 UTC (permalink / raw)
To: John W. Linville; +Cc: linux-wireless, stable
On Wed, Oct 12, 2011 at 02:50:34PM -0400, John W. Linville wrote:
> On Wed, Oct 12, 2011 at 02:40:44PM -0400, John W. Linville wrote:
> > On Mon, Oct 10, 2011 at 11:25:45AM +0530, Rajkumar Manoharan wrote:
> > > Because of not clearing Bit 14 of AR_WA, the ASPM L1 is not
> > > enabled when entering into sleep mode. AR9462 does not need
> > > bit 14 to be set.
> > >
> > > Cc: stable@kernel.org
> > > Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
> >
> > CC [M] drivers/net/wireless/ath/ath9k/hw.o
> > drivers/net/wireless/ath/ath9k/hw.c: In function ???__ath9k_hw_init???:
> > drivers/net/wireless/ath/ath9k/hw.c:544:2: error: implicit declaration of function ???AR_SREV_9462???
> >
> > Missing a patch dependency?
>
> I see that is defined by a series posted just before that -- one that
> hasn't even appeared in wireless-next yet. Cc: stable@kernel.org
> seems a bit inappropriate...
>
> That series needs rework anyway -- how about you just fold this one
> into that earlier series?
>
First of all sorry for the inconvenience. While splitting stable patch
from the patchset, I made it on top of patchset that caused the dependancy.
I will resend patch.
--
Rajkumar
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2011-10-13 3:45 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-10-10 5:55 [PATCH] ath9k_hw: Fix ASPM L1 issue for AR9462 Rajkumar Manoharan
2011-10-12 18:40 ` John W. Linville
2011-10-12 18:50 ` John W. Linville
2011-10-13 3:46 ` Rajkumar Manoharan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).