On Thu, Aug 02, 2007 at 06:15:05PM -0700, Greg KH wrote: > Well, if you do this, then you can pretty much delete the whole quirk > table we have, right?
At the moment, yes. > And personally, I want to do better than Windows XP when it comes to > power management. This patch is only going to suspend a very tiny > subset of devices, including a whole bunch of ones that do not even have > drivers in Linux, causing our power footprint to be bigger than needed. I agree. I'd much rather see us suspending devices whenever possible - it's just that I have concerns over the scalability of the blacklist, given the number of devices that seem to have issues. > Also, we have udev rules for SANE that disables their autosuspend > settings, which handles the majority of the devices we have seen with > problems. Several printers seem to have the issue as well, and the blacklist seems to contain some odd miscellaneous devices like the Blackberry. The main concern I have is that kernel developers just don't tend to be the sort of people that use webcams, printers or scanners, so we're relying on normal users to go to the effort of reporting that their device has stopped working. > So I really don't want to accept this patch. But, what problems are you > seeing with our current suspend logic that you feel we need to be this > harsh? It's definitely a brute force approach, but it's one that means that we get the low hanging fruit (ie, pretty much anything that's likely to be plugged into a laptop) while massively reducing the probability of breaking anyone's system. Saving some power is a nice win, but breaking someone's printer is a pretty big loss. -- Matthew Garrett | [EMAIL PROTECTED] ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ linux-usb-devel@lists.sourceforge.net To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
