Here's my second series on cleanups to fixate driver core changes in the right place. In this series I stuff power save helpers which use the driver for locking. Next thing I'm looking at is the usage of sc->bssidmask and sc->curbssid even on hw.c on ath9k -- we should not access the sc on hw code, anyway, while reviewing that I noticed ath5k/ar9170 could benefit from a few changes already in place on ath9k. It seems the last thing we'll need to address is read/write ops. I've considered defining read/write ops on the ath common structure, or at the very least on ath_hw, upon initialization then the driver core would set this. ath9k would use ath9k_iowrite32() and ath9k_ioread32() would ath9k_htc would define its own. REG_READ and REG_WRITE could then also point to the common->ops->read / write or if we don't want to take that approach the ath_hw->rw->read / write. Other suggestions are welcomed. A minor thing I did notice is we use ieee80211_hw on ath9k hw.c for configuration -- this is fine but I realized virtual wiphys are not using their own hw conf, but the primary wiphy's. Luis R. Rodriguez (7): ath9k: move ps helpers onto core driver when reseting tsf ath9k: move ath9k_ps_wakeup() and ath9k_ps_restore() to main.c ath9k: avoid usage of ath9k_hw_setpower() on hw.c ath9k: move ath9k_hw_setpower() to main.c ath9k: rename driver core and hw power save helpers ath: move ath_bcast_mac to common header atheros: use get_unaligned_le*() for bssid mask setting drivers/net/wireless/ath/ar9170/mac.c | 10 +++--- drivers/net/wireless/ath/ath.h | 3 ++ drivers/net/wireless/ath/ath5k/ath5k.h | 7 ---- drivers/net/wireless/ath/ath5k/attach.c | 2 +- drivers/net/wireless/ath/ath5k/base.c | 2 +- drivers/net/wireless/ath/ath5k/pcu.c | 38 +++++++++++-------- drivers/net/wireless/ath/ath5k/reset.c | 6 ++- drivers/net/wireless/ath/ath9k/ath9k.h | 2 - drivers/net/wireless/ath/ath9k/beacon.c | 17 ++++++++- drivers/net/wireless/ath/ath9k/hw.c | 50 +------------------------- drivers/net/wireless/ath/ath9k/hw.h | 5 ++- drivers/net/wireless/ath/ath9k/main.c | 59 ++++++++++++++++++++++++++++--- 12 files changed, 109 insertions(+), 92 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html