On 12/13/09 16:18, Ivo van Doorn wrote: > On Sunday 13 December 2009, Gertjan van Wingerde wrote: >> On 12/13/09 11:15, Ivo van Doorn wrote: >>> On Sunday 13 December 2009, Gertjan van Wingerde wrote: >>>> We've had many reports of rt61pci failures with powersaving enabled. >>>> Therefore, as a stop-gap measure, disable powersaving of the rt61pci >>>> until we have found a proper solution. >>> >>> This disables powersaving completely, can't we set the default powersaving >>> to disabled? That way for people for who it does work, it can still be enabled. >>> >> >> Well, when I was looking into this I came up with a number of issues in the code >> which makes me believe that it simply isn't working at all. >> One example is that the rt61pci devices rely on handling the wakeup interrupt >> by the driver to wake up the devices. As far as I can tell there is no handling >> of the wakeup interrupt in the rt61pci driver. >> That's why I decided to disable it all together, as I don't think it works >> properly for anyone (although some people may not really notice it). > > Ok, but then please disable it for _all_ rt2x00 drivers, or at least all PCI rt2x00 drivers > since they would have the same wakeup interrupt requirement. > I'll see if this will be the case for all PCI rt2x00 devices and adapt for the ones that need it. I didn't look into the other devices yet, and ISTR that at least rt2500pci doesn't have this requirement. --- Gertjan. -- 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