linux-ath5k-devel mailing list

FromSubjectsort iconDate
Bob Copeland
[ath5k-devel] [PATCH 0/5] ath5k updates

I've been sitting on these patches for much too long. But anyway,
here are some updates for ath5k. 1-2 fix beacon issues, 3 is just
minor cleanup that I had sitting around forever, 4 is a reported
suspend/resume issue from Alan Jenkins, and 5 is a reported
initialization issue from Forrest Zhang.

Bob Copeland (5):
ath5k: send buffered frames after the beacon
ath5k: rework beacon configuration
ath: remove unnecessary return in ath_regd_get_band_ctl
ath5k: do not release irq across suspen...

Jul 4, 12:59 pm 2009
Bob Copeland
[ath5k-devel] [PATCH 4/5] ath5k: do not release irq across s...

Paraphrasing Rafael J. Wysocki: "drivers should not release PCI IRQs
in suspend." Doing so causes a warning during suspend/resume on some
platforms.

Cc: Rafael J. Wysocki <rjw@sisk.pl>
Reported-by: Alan Jenkins <alan-jenkins@tuffmail.co.uk>
Signed-off-by: Bob Copeland <me@bobcopeland.com>
---
drivers/net/wireless/ath/ath5k/base.c | 11 -----------
1 files changed, 0 insertions(+), 11 deletions(-)

diff --git a/drivers/net/wireless/ath/ath5k/base.c b/drivers/net/wireless/ath...

Jul 4, 12:59 pm 2009
Jiri Slaby
Re: [ath5k-devel] [PATCH 4/5] ath5k: do not release irq acro...

Hmm, I added it because I was told an irq can change over suspend (or at
least it could in the past). I think it's not true anymore, since we
would have serious problems with the code all over the tree.

Signed-off-by: Jiri Slaby <jirislaby@gmail.com>

Thanks.
_______________________________________________
ath5k-devel mailing list
ath5k-devel@lists.ath5k.org
https://lists.ath5k.org/mailman/listinfo/ath5k-devel

Jul 4, 5:24 pm 2009
Rafael J. Wysocki
Re: [ath5k-devel] [PATCH 4/5] ath5k: do not release irq acro...

Someone recently asked on a mailing list (unfortunately I can't recall
which one) if drivers should release irqs over suspend.

The answer was more-or-less that drivers are not expected to do that and it
really is not recommended. In fact, we do the whole suspend_device_irqs()
thing, because the drivers are expected not to do that.

Best,
Rafael
_______________________________________________
ath5k-devel mailing list
ath5k-devel@lists.ath5k.org
[ message continues ]

" title="https://lists.ath5k.org/mailman/listinfo/ath5k-d...">https://lists.ath5k.org/mailman/listinfo/ath5k-d...

Jul 4, 5:37 pm 2009
Bob Copeland
[ath5k-devel] [PATCH 5/5] ath5k: write PCU registers on init...

Commit d7dc100374df0c21afd8a198336ecd7999697159, "Ath5k: unify resets"
introduced a regression into 2.6.28 where the PCU registers are never
initialized, due to ath5k_reset() always passing true for change_channel.
We subsequently program a lot of these registers but several may start
in an unknown state.

Cc: stable@kernel.org
Reported-by: Forrest Zhang <forrest@hifulltech.com>
Signed-off-by: Bob Copeland <me@bobcopeland.com>
---
drivers/net/wireless/ath/ath5k/base.c | 2 +-
1 fi...

Jul 4, 12:59 pm 2009
Felix Fietkau
Re: [ath5k-devel] [PATCH 5/5] ath5k: write PCU registers on ...

Shouldn't this be chan != NULL? I'd assume that chan is NULL the first
time and change_channel thus should be false to initialize all the
registers.

- Felix
_______________________________________________
ath5k-devel mailing list
ath5k-devel@lists.ath5k.org
https://lists.ath5k.org/mailman/listinfo/ath5k-devel

Jul 4, 2:40 pm 2009
Bob Copeland
[ath5k-devel] [PATCH 1/5] ath5k: send buffered frames after ...

Enable the "Content" After Beacon queue and utilize it to send
any buffered frames for power-saving clients.

Signed-off-by: Bob Copeland <me@bobcopeland.com>
---
drivers/net/wireless/ath/ath5k/base.c | 35 ++++++++++++++++++++++++++++----
drivers/net/wireless/ath/ath5k/base.h | 9 +++----
drivers/net/wireless/ath/ath5k/qcu.c | 1 -
3 files changed, 34 insertions(+), 11 deletions(-)

diff --git a/drivers/net/wireless/ath/ath5k/base.c b/drivers/net/wireless/ath/ath5k/base.c
index c...

Jul 4, 12:59 pm 2009
Bob Copeland
[ath5k-devel] [PATCH 2/5] ath5k: rework beacon configuration

Using the enable_beacon flag allows some simplifications and fixes
some corner cases in beacon handling. This change adds a state
variable for beaconing in ath5k_beacon_config and handles both
enabling and disabling, thus eliminating the need for
ath5k_beacon_disable. We also now configure the beacon when any
of the beacon parameters change, so ath5k_beacon_reconfig is no
longer needed (its mmiowb gets moved to ath5k_beacon_config).
Finally, by locking around the whole config function, we don't
ne...

Jul 4, 12:59 pm 2009
Bob Copeland
[ath5k-devel] [PATCH 3/5] ath: remove unnecessary return in ...

'default' case already returns NO_CTL

Signed-off-by: Bob Copeland <me@bobcopeland.com>
---
drivers/net/wireless/ath/regd.c | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/ath/regd.c b/drivers/net/wireless/ath/regd.c
index eef370b..f37c832 100644
--- a/drivers/net/wireless/ath/regd.c
+++ b/drivers/net/wireless/ath/regd.c
@@ -569,7 +569,5 @@ u32 ath_regd_get_band_ctl(struct ath_regulatory *reg,
default:
return NO_CTL;
}
-
- return N...

Jul 4, 12:59 pm 2009
previous daytodaynext day
July 1, 2009July 4, 2009July 5, 2009