* [PATCH 0/4] rt2x00: Powersave fixes.
@ 2010-03-30 21:50 Gertjan van Wingerde
2010-03-30 21:50 ` [PATCH 1/4] rt2x00: Disable auto wakeup before waking up device Gertjan van Wingerde
` (4 more replies)
0 siblings, 5 replies; 11+ messages in thread
From: Gertjan van Wingerde @ 2010-03-30 21:50 UTC (permalink / raw)
To: John W. Linville
Cc: Ivo van Doorn, users, linux-wireless, Gertjan van Wingerde
The following 4 patches fix powersave behavior of rt2x00.
The first patch fixes the rt2500usb powersave issues found lately.
The second on enables powersave for rt2500usb again, now it is fixed.
The third and fourth are ongoing work to bring the other PCI drivers
in a better shape.
All patches are against wireless-2.6.
The patch series is also available via:
git://git.gwingerde.nl/rt2x00-2.6.git
Gertjan van Wingerde (4):
rt2x00: Disable auto wakeup before waking up device.
rt2x00: Enable powersaving by default again on rt2500usb.
rt2x00: Add wakeup interrupt handler to rt61pci.
rt2x00: Add wakeup interrupt handler to rt2800pci.
drivers/net/wireless/rt2x00/rt2400pci.c | 4 ++++
drivers/net/wireless/rt2x00/rt2500pci.c | 4 ++++
drivers/net/wireless/rt2x00/rt2500usb.c | 9 ++++-----
drivers/net/wireless/rt2x00/rt2800lib.c | 4 ++--
drivers/net/wireless/rt2x00/rt2800pci.c | 11 +++++++++++
drivers/net/wireless/rt2x00/rt61pci.c | 14 ++++++++++++++
drivers/net/wireless/rt2x00/rt73usb.c | 6 +++---
7 files changed, 42 insertions(+), 10 deletions(-)
^ permalink raw reply [flat|nested] 11+ messages in thread* [PATCH 1/4] rt2x00: Disable auto wakeup before waking up device. 2010-03-30 21:50 [PATCH 0/4] rt2x00: Powersave fixes Gertjan van Wingerde @ 2010-03-30 21:50 ` Gertjan van Wingerde 2010-03-30 21:50 ` [PATCH 2/4] rt2x00: Enable powersaving by default again on rt2500usb Gertjan van Wingerde ` (3 subsequent siblings) 4 siblings, 0 replies; 11+ messages in thread From: Gertjan van Wingerde @ 2010-03-30 21:50 UTC (permalink / raw) To: John W. Linville Cc: Ivo van Doorn, users, linux-wireless, Gertjan van Wingerde, Ondrej Zary In all drivers ensure that auto wakeup is disabled before waking up the device. This is needed to prevent connection stability issues and problems in waking up the device. Based upon a patch from Ondrej Zary <linux@rainbow-software.org> Signed-off-by: Gertjan van Wingerde <gwingerde@gmail.com> Cc: Ondrej Zary <linux@rainbow-software.org> --- drivers/net/wireless/rt2x00/rt2400pci.c | 4 ++++ drivers/net/wireless/rt2x00/rt2500pci.c | 4 ++++ drivers/net/wireless/rt2x00/rt2500usb.c | 4 ++++ drivers/net/wireless/rt2x00/rt2800lib.c | 4 ++-- drivers/net/wireless/rt2x00/rt73usb.c | 6 +++--- 5 files changed, 17 insertions(+), 5 deletions(-) diff --git a/drivers/net/wireless/rt2x00/rt2400pci.c b/drivers/net/wireless/rt2x00/rt2400pci.c index c22b040..08a4789 100644 --- a/drivers/net/wireless/rt2x00/rt2400pci.c +++ b/drivers/net/wireless/rt2x00/rt2400pci.c @@ -525,6 +525,10 @@ static void rt2400pci_config_ps(struct rt2x00_dev *rt2x00dev, rt2x00_set_field32(®, CSR20_AUTOWAKE, 1); rt2x00pci_register_write(rt2x00dev, CSR20, reg); + } else { + rt2x00pci_register_read(rt2x00dev, CSR20, ®); + rt2x00_set_field32(®, CSR20_AUTOWAKE, 0); + rt2x00pci_register_write(rt2x00dev, CSR20, reg); } rt2x00dev->ops->lib->set_device_state(rt2x00dev, state); diff --git a/drivers/net/wireless/rt2x00/rt2500pci.c b/drivers/net/wireless/rt2x00/rt2500pci.c index 52bbcf1..d084d70 100644 --- a/drivers/net/wireless/rt2x00/rt2500pci.c +++ b/drivers/net/wireless/rt2x00/rt2500pci.c @@ -573,6 +573,10 @@ static void rt2500pci_config_ps(struct rt2x00_dev *rt2x00dev, rt2x00_set_field32(®, CSR20_AUTOWAKE, 1); rt2x00pci_register_write(rt2x00dev, CSR20, reg); + } else { + rt2x00pci_register_read(rt2x00dev, CSR20, ®); + rt2x00_set_field32(®, CSR20_AUTOWAKE, 0); + rt2x00pci_register_write(rt2x00dev, CSR20, reg); } rt2x00dev->ops->lib->set_device_state(rt2x00dev, state); diff --git a/drivers/net/wireless/rt2x00/rt2500usb.c b/drivers/net/wireless/rt2x00/rt2500usb.c index dbaa781..54d2716 100644 --- a/drivers/net/wireless/rt2x00/rt2500usb.c +++ b/drivers/net/wireless/rt2x00/rt2500usb.c @@ -648,6 +648,10 @@ static void rt2500usb_config_ps(struct rt2x00_dev *rt2x00dev, rt2x00_set_field16(®, MAC_CSR18_AUTO_WAKE, 1); rt2500usb_register_write(rt2x00dev, MAC_CSR18, reg); + } else { + rt2500usb_register_read(rt2x00dev, MAC_CSR18, ®); + rt2x00_set_field16(®, MAC_CSR18_AUTO_WAKE, 0); + rt2500usb_register_write(rt2x00dev, MAC_CSR18, reg); } rt2x00dev->ops->lib->set_device_state(rt2x00dev, state); diff --git a/drivers/net/wireless/rt2x00/rt2800lib.c b/drivers/net/wireless/rt2x00/rt2800lib.c index 326fce7..68d0cfe 100644 --- a/drivers/net/wireless/rt2x00/rt2800lib.c +++ b/drivers/net/wireless/rt2x00/rt2800lib.c @@ -1014,13 +1014,13 @@ static void rt2800_config_ps(struct rt2x00_dev *rt2x00dev, rt2x00dev->ops->lib->set_device_state(rt2x00dev, state); } else { - rt2x00dev->ops->lib->set_device_state(rt2x00dev, state); - rt2800_register_read(rt2x00dev, AUTOWAKEUP_CFG, ®); rt2x00_set_field32(®, AUTOWAKEUP_CFG_AUTO_LEAD_TIME, 0); rt2x00_set_field32(®, AUTOWAKEUP_CFG_TBCN_BEFORE_WAKE, 0); rt2x00_set_field32(®, AUTOWAKEUP_CFG_AUTOWAKE, 0); rt2800_register_write(rt2x00dev, AUTOWAKEUP_CFG, reg); + + rt2x00dev->ops->lib->set_device_state(rt2x00dev, state); } } diff --git a/drivers/net/wireless/rt2x00/rt73usb.c b/drivers/net/wireless/rt2x00/rt73usb.c index 47f3e4a..7ebe14b 100644 --- a/drivers/net/wireless/rt2x00/rt73usb.c +++ b/drivers/net/wireless/rt2x00/rt73usb.c @@ -860,15 +860,15 @@ static void rt73usb_config_ps(struct rt2x00_dev *rt2x00dev, rt2x00usb_vendor_request_sw(rt2x00dev, USB_DEVICE_MODE, 0, USB_MODE_SLEEP, REGISTER_TIMEOUT); } else { - rt2x00usb_vendor_request_sw(rt2x00dev, USB_DEVICE_MODE, 0, - USB_MODE_WAKEUP, REGISTER_TIMEOUT); - rt2x00usb_register_read(rt2x00dev, MAC_CSR11, ®); rt2x00_set_field32(®, MAC_CSR11_DELAY_AFTER_TBCN, 0); rt2x00_set_field32(®, MAC_CSR11_TBCN_BEFORE_WAKEUP, 0); rt2x00_set_field32(®, MAC_CSR11_AUTOWAKE, 0); rt2x00_set_field32(®, MAC_CSR11_WAKEUP_LATENCY, 0); rt2x00usb_register_write(rt2x00dev, MAC_CSR11, reg); + + rt2x00usb_vendor_request_sw(rt2x00dev, USB_DEVICE_MODE, 0, + USB_MODE_WAKEUP, REGISTER_TIMEOUT); } } -- 1.7.0.3 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 2/4] rt2x00: Enable powersaving by default again on rt2500usb. 2010-03-30 21:50 [PATCH 0/4] rt2x00: Powersave fixes Gertjan van Wingerde 2010-03-30 21:50 ` [PATCH 1/4] rt2x00: Disable auto wakeup before waking up device Gertjan van Wingerde @ 2010-03-30 21:50 ` Gertjan van Wingerde 2010-03-31 22:29 ` Adam Baker 2010-03-30 21:50 ` [PATCH 3/4] rt2x00: Add wakeup interrupt handler to rt61pci Gertjan van Wingerde ` (2 subsequent siblings) 4 siblings, 1 reply; 11+ messages in thread From: Gertjan van Wingerde @ 2010-03-30 21:50 UTC (permalink / raw) To: John W. Linville Cc: Ivo van Doorn, users, linux-wireless, Gertjan van Wingerde Now that the powersave issues on rt2500usb have been tackled, powersave can be enabled by default again. Signed-off-by: Gertjan van Wingerde <gwingerde@gmail.com> --- drivers/net/wireless/rt2x00/rt2500usb.c | 5 ----- 1 files changed, 0 insertions(+), 5 deletions(-) diff --git a/drivers/net/wireless/rt2x00/rt2500usb.c b/drivers/net/wireless/rt2x00/rt2500usb.c index 54d2716..c1eec17 100644 --- a/drivers/net/wireless/rt2x00/rt2500usb.c +++ b/drivers/net/wireless/rt2x00/rt2500usb.c @@ -1647,11 +1647,6 @@ static int rt2500usb_probe_hw_mode(struct rt2x00_dev *rt2x00dev) unsigned int i; /* - * Disable powersaving as default. - */ - rt2x00dev->hw->wiphy->flags &= ~WIPHY_FLAG_PS_ON_BY_DEFAULT; - - /* * Initialize all hw fields. */ rt2x00dev->hw->flags = -- 1.7.0.3 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 2/4] rt2x00: Enable powersaving by default again on rt2500usb. 2010-03-30 21:50 ` [PATCH 2/4] rt2x00: Enable powersaving by default again on rt2500usb Gertjan van Wingerde @ 2010-03-31 22:29 ` Adam Baker 2010-03-31 22:34 ` Gertjan van Wingerde 0 siblings, 1 reply; 11+ messages in thread From: Adam Baker @ 2010-03-31 22:29 UTC (permalink / raw) To: linux-wireless Gertjan van Wingerde <gwingerde@...> writes: > > Now that the powersave issues on rt2500usb have been tackled, powersave > can be enabled by default again. > > Signed-off-by: Gertjan van Wingerde <gwingerde@...> > --- > drivers/net/wireless/rt2x00/rt2500usb.c | 5 ----- > 1 files changed, 0 insertions(+), 5 deletions(-) > > diff --git a/drivers/net/wireless/rt2x00/rt2500usb.c b/drivers/net/wireless/rt2x00/rt2500usb.c > index 54d2716..c1eec17 100644 > --- a/drivers/net/wireless/rt2x00/rt2500usb.c > +++ b/drivers/net/wireless/rt2x00/rt2500usb.c > @@ -1647,11 +1647,6 @@ static int rt2500usb_probe_hw_mode(struct rt2x00_dev *rt2x00dev) > unsigned int i; > > /* > - * Disable powersaving as default. > - */ > - rt2x00dev->hw->wiphy->flags &= ~WIPHY_FLAG_PS_ON_BY_DEFAULT; > This looks like it deletes one line too few leaving a nested open comment block in the code. Adam ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/4] rt2x00: Enable powersaving by default again on rt2500usb. 2010-03-31 22:29 ` Adam Baker @ 2010-03-31 22:34 ` Gertjan van Wingerde 0 siblings, 0 replies; 11+ messages in thread From: Gertjan van Wingerde @ 2010-03-31 22:34 UTC (permalink / raw) To: Adam Baker; +Cc: linux-wireless On 04/01/10 00:29, Adam Baker wrote: > Gertjan van Wingerde <gwingerde@...> writes: > >> >> Now that the powersave issues on rt2500usb have been tackled, powersave >> can be enabled by default again. >> >> Signed-off-by: Gertjan van Wingerde <gwingerde@...> >> --- >> drivers/net/wireless/rt2x00/rt2500usb.c | 5 ----- >> 1 files changed, 0 insertions(+), 5 deletions(-) >> >> diff --git a/drivers/net/wireless/rt2x00/rt2500usb.c > b/drivers/net/wireless/rt2x00/rt2500usb.c >> index 54d2716..c1eec17 100644 >> --- a/drivers/net/wireless/rt2x00/rt2500usb.c >> +++ b/drivers/net/wireless/rt2x00/rt2500usb.c >> @@ -1647,11 +1647,6 @@ static int rt2500usb_probe_hw_mode(struct rt2x00_dev > *rt2x00dev) >> unsigned int i; >> >> /* >> - * Disable powersaving as default. >> - */ >> - rt2x00dev->hw->wiphy->flags &= ~WIPHY_FLAG_PS_ON_BY_DEFAULT; >> > This looks like it deletes one line too few leaving a nested open comment > block in the code. > Nope, it doesn't. This is just the peculiar way diff creates patches within git. If you take a look at the full patch, then you'll see that the patch removes all lines including the '/*' of the next comment. So the end-result is still good. --- Gertjan. ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 3/4] rt2x00: Add wakeup interrupt handler to rt61pci. 2010-03-30 21:50 [PATCH 0/4] rt2x00: Powersave fixes Gertjan van Wingerde 2010-03-30 21:50 ` [PATCH 1/4] rt2x00: Disable auto wakeup before waking up device Gertjan van Wingerde 2010-03-30 21:50 ` [PATCH 2/4] rt2x00: Enable powersaving by default again on rt2500usb Gertjan van Wingerde @ 2010-03-30 21:50 ` Gertjan van Wingerde 2010-03-30 21:50 ` [PATCH 4/4] rt2x00: Add wakeup interrupt handler to rt2800pci Gertjan van Wingerde 2010-04-06 19:34 ` [PATCH 0/4] rt2x00: Powersave fixes John W. Linville 4 siblings, 0 replies; 11+ messages in thread From: Gertjan van Wingerde @ 2010-03-30 21:50 UTC (permalink / raw) To: John W. Linville Cc: Ivo van Doorn, users, linux-wireless, Gertjan van Wingerde This is needed to wake up the device automatically for receiving beacons, and is required for proper powersave handling. Signed-off-by: Gertjan van Wingerde <gwingerde@gmail.com> --- drivers/net/wireless/rt2x00/rt61pci.c | 14 ++++++++++++++ 1 files changed, 14 insertions(+), 0 deletions(-) diff --git a/drivers/net/wireless/rt2x00/rt61pci.c b/drivers/net/wireless/rt2x00/rt61pci.c index e2da928..ac69dbe 100644 --- a/drivers/net/wireless/rt2x00/rt61pci.c +++ b/drivers/net/wireless/rt2x00/rt61pci.c @@ -2117,6 +2117,14 @@ static void rt61pci_txdone(struct rt2x00_dev *rt2x00dev) } } +static void rt61pci_wakeup(struct rt2x00_dev *rt2x00dev) +{ + struct ieee80211_conf conf = { .flags = 0 }; + struct rt2x00lib_conf libconf = { .conf = &conf }; + + rt61pci_config(rt2x00dev, &libconf, IEEE80211_CONF_CHANGE_PS); +} + static irqreturn_t rt61pci_interrupt(int irq, void *dev_instance) { struct rt2x00_dev *rt2x00dev = dev_instance; @@ -2164,6 +2172,12 @@ static irqreturn_t rt61pci_interrupt(int irq, void *dev_instance) rt2x00pci_register_write(rt2x00dev, M2H_CMD_DONE_CSR, 0xffffffff); + /* + * 4 - MCU Autowakeup interrupt. + */ + if (rt2x00_get_field32(reg_mcu, MCU_INT_SOURCE_CSR_TWAKEUP)) + rt61pci_wakeup(rt2x00dev); + return IRQ_HANDLED; } -- 1.7.0.3 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 4/4] rt2x00: Add wakeup interrupt handler to rt2800pci. 2010-03-30 21:50 [PATCH 0/4] rt2x00: Powersave fixes Gertjan van Wingerde ` (2 preceding siblings ...) 2010-03-30 21:50 ` [PATCH 3/4] rt2x00: Add wakeup interrupt handler to rt61pci Gertjan van Wingerde @ 2010-03-30 21:50 ` Gertjan van Wingerde 2010-04-06 19:34 ` [PATCH 0/4] rt2x00: Powersave fixes John W. Linville 4 siblings, 0 replies; 11+ messages in thread From: Gertjan van Wingerde @ 2010-03-30 21:50 UTC (permalink / raw) To: John W. Linville Cc: Ivo van Doorn, users, linux-wireless, Gertjan van Wingerde This is needed to wake up the device automatically for receiving beacons, and is required for proper powersave handling. Signed-off-by: Gertjan van Wingerde <gwingerde@gmail.com> --- drivers/net/wireless/rt2x00/rt2800pci.c | 11 +++++++++++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/drivers/net/wireless/rt2x00/rt2800pci.c b/drivers/net/wireless/rt2x00/rt2800pci.c index 91cce2d..357873c 100644 --- a/drivers/net/wireless/rt2x00/rt2800pci.c +++ b/drivers/net/wireless/rt2x00/rt2800pci.c @@ -1006,6 +1006,14 @@ static void rt2800pci_txdone(struct rt2x00_dev *rt2x00dev) } } +static void rt2800pci_wakeup(struct rt2x00_dev *rt2x00dev) +{ + struct ieee80211_conf conf = { .flags = 0 }; + struct rt2x00lib_conf libconf = { .conf = &conf }; + + rt2800_config(rt2x00dev, &libconf, IEEE80211_CONF_CHANGE_PS); +} + static irqreturn_t rt2800pci_interrupt(int irq, void *dev_instance) { struct rt2x00_dev *rt2x00dev = dev_instance; @@ -1030,6 +1038,9 @@ static irqreturn_t rt2800pci_interrupt(int irq, void *dev_instance) if (rt2x00_get_field32(reg, INT_SOURCE_CSR_TX_FIFO_STATUS)) rt2800pci_txdone(rt2x00dev); + if (rt2x00_get_field32(reg, INT_SOURCE_CSR_AUTO_WAKEUP)) + rt2800pci_wakeup(rt2x00dev); + return IRQ_HANDLED; } -- 1.7.0.3 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 0/4] rt2x00: Powersave fixes. 2010-03-30 21:50 [PATCH 0/4] rt2x00: Powersave fixes Gertjan van Wingerde ` (3 preceding siblings ...) 2010-03-30 21:50 ` [PATCH 4/4] rt2x00: Add wakeup interrupt handler to rt2800pci Gertjan van Wingerde @ 2010-04-06 19:34 ` John W. Linville 2010-04-06 19:52 ` Gertjan van Wingerde 4 siblings, 1 reply; 11+ messages in thread From: John W. Linville @ 2010-04-06 19:34 UTC (permalink / raw) To: Gertjan van Wingerde; +Cc: Ivo van Doorn, users, linux-wireless On Tue, Mar 30, 2010 at 11:50:22PM +0200, Gertjan van Wingerde wrote: > The following 4 patches fix powersave behavior of rt2x00. > The first patch fixes the rt2500usb powersave issues found lately. > The second on enables powersave for rt2500usb again, now it is fixed. > The third and fourth are ongoing work to bring the other PCI drivers > in a better shape. > > All patches are against wireless-2.6. > > The patch series is also available via: > > git://git.gwingerde.nl/rt2x00-2.6.git > > Gertjan van Wingerde (4): > rt2x00: Disable auto wakeup before waking up device. > rt2x00: Enable powersaving by default again on rt2500usb. > rt2x00: Add wakeup interrupt handler to rt61pci. > rt2x00: Add wakeup interrupt handler to rt2800pci. I don't have any objection to these patches, but they don't seem appropriate for 2.6.34 to me. As such, basing them on wireless-next-2.6 seems more appropriate. Any objection to me applying them for 2.6.35? 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] 11+ messages in thread
* Re: [PATCH 0/4] rt2x00: Powersave fixes. 2010-04-06 19:34 ` [PATCH 0/4] rt2x00: Powersave fixes John W. Linville @ 2010-04-06 19:52 ` Gertjan van Wingerde 2010-04-06 21:18 ` John W. Linville 2010-04-06 21:26 ` Ivo van Doorn 0 siblings, 2 replies; 11+ messages in thread From: Gertjan van Wingerde @ 2010-04-06 19:52 UTC (permalink / raw) To: John W. Linville; +Cc: Ivo van Doorn, users, linux-wireless On 04/06/10 21:34, John W. Linville wrote: > On Tue, Mar 30, 2010 at 11:50:22PM +0200, Gertjan van Wingerde wrote: >> The following 4 patches fix powersave behavior of rt2x00. >> The first patch fixes the rt2500usb powersave issues found lately. >> The second on enables powersave for rt2500usb again, now it is fixed. >> The third and fourth are ongoing work to bring the other PCI drivers >> in a better shape. >> >> All patches are against wireless-2.6. >> >> The patch series is also available via: >> >> git://git.gwingerde.nl/rt2x00-2.6.git >> >> Gertjan van Wingerde (4): >> rt2x00: Disable auto wakeup before waking up device. >> rt2x00: Enable powersaving by default again on rt2500usb. >> rt2x00: Add wakeup interrupt handler to rt61pci. >> rt2x00: Add wakeup interrupt handler to rt2800pci. > > I don't have any objection to these patches, but they don't > seem appropriate for 2.6.34 to me. As such, basing them on > wireless-next-2.6 seems more appropriate. > > Any objection to me applying them for 2.6.35? > You're the boss ;-) No, no objections from my side. We already disabled powersaving in 2.6.34, so we should be covered for that release. Note that patch 2 then won't apply to wireless-next-2.6, as the patch that disables powersaving is only in wireless-2.6. How do you want to handle applying patch 2 from the series; how do we synchronize? --- Gertjan. ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 0/4] rt2x00: Powersave fixes. 2010-04-06 19:52 ` Gertjan van Wingerde @ 2010-04-06 21:18 ` John W. Linville 2010-04-06 21:26 ` Ivo van Doorn 1 sibling, 0 replies; 11+ messages in thread From: John W. Linville @ 2010-04-06 21:18 UTC (permalink / raw) To: Gertjan van Wingerde; +Cc: Ivo van Doorn, users, linux-wireless On Tue, Apr 06, 2010 at 09:52:22PM +0200, Gertjan van Wingerde wrote: > On 04/06/10 21:34, John W. Linville wrote: > > Any objection to me applying them for 2.6.35? > > > > You're the boss ;-) More like "coordinator" I think... :-) > No, no objections from my side. We already disabled powersaving in 2.6.34, > so we should be covered for that release. > > Note that patch 2 then won't apply to wireless-next-2.6, as the patch > that disables powersaving is only in wireless-2.6. > > How do you want to handle applying patch 2 from the series; how do we > synchronize? I'm hitting this type of problem with a number of pending patches now. I'll need to pull wireless-2.6 into wireless-next-2.6 soon. 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] 11+ messages in thread
* Re: [PATCH 0/4] rt2x00: Powersave fixes. 2010-04-06 19:52 ` Gertjan van Wingerde 2010-04-06 21:18 ` John W. Linville @ 2010-04-06 21:26 ` Ivo van Doorn 1 sibling, 0 replies; 11+ messages in thread From: Ivo van Doorn @ 2010-04-06 21:26 UTC (permalink / raw) To: John W. Linville; +Cc: Gertjan van Wingerde, users, linux-wireless On Tuesday 06 April 2010, Gertjan van Wingerde wrote: > On 04/06/10 21:34, John W. Linville wrote: > > On Tue, Mar 30, 2010 at 11:50:22PM +0200, Gertjan van Wingerde wrote: > >> The following 4 patches fix powersave behavior of rt2x00. > >> The first patch fixes the rt2500usb powersave issues found lately. > >> The second on enables powersave for rt2500usb again, now it is fixed. > >> The third and fourth are ongoing work to bring the other PCI drivers > >> in a better shape. > >> > >> All patches are against wireless-2.6. > >> > >> The patch series is also available via: > >> > >> git://git.gwingerde.nl/rt2x00-2.6.git > >> > >> Gertjan van Wingerde (4): > >> rt2x00: Disable auto wakeup before waking up device. > >> rt2x00: Enable powersaving by default again on rt2500usb. > >> rt2x00: Add wakeup interrupt handler to rt61pci. > >> rt2x00: Add wakeup interrupt handler to rt2800pci. > > > > I don't have any objection to these patches, but they don't > > seem appropriate for 2.6.34 to me. As such, basing them on > > wireless-next-2.6 seems more appropriate. > > > > Any objection to me applying them for 2.6.35? > > > > You're the boss ;-) > > No, no objections from my side. We already disabled powersaving in 2.6.34, > so we should be covered for that release. > > Note that patch 2 then won't apply to wireless-next-2.6, as the patch > that disables powersaving is only in wireless-2.6. > > How do you want to handle applying patch 2 from the series; how do we > synchronize? John, not sure how important you consider it, but please note that the first 2 patches would fix: https://bugzilla.kernel.org/show_bug.cgi?id=15699 Ivo ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2010-04-06 21:30 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2010-03-30 21:50 [PATCH 0/4] rt2x00: Powersave fixes Gertjan van Wingerde 2010-03-30 21:50 ` [PATCH 1/4] rt2x00: Disable auto wakeup before waking up device Gertjan van Wingerde 2010-03-30 21:50 ` [PATCH 2/4] rt2x00: Enable powersaving by default again on rt2500usb Gertjan van Wingerde 2010-03-31 22:29 ` Adam Baker 2010-03-31 22:34 ` Gertjan van Wingerde 2010-03-30 21:50 ` [PATCH 3/4] rt2x00: Add wakeup interrupt handler to rt61pci Gertjan van Wingerde 2010-03-30 21:50 ` [PATCH 4/4] rt2x00: Add wakeup interrupt handler to rt2800pci Gertjan van Wingerde 2010-04-06 19:34 ` [PATCH 0/4] rt2x00: Powersave fixes John W. Linville 2010-04-06 19:52 ` Gertjan van Wingerde 2010-04-06 21:18 ` John W. Linville 2010-04-06 21:26 ` Ivo van Doorn
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).