<table cellspacing="0" cellpadding="0" border="0" ><tr><td valign="top" style="font: inherit;">I dont know, sorry,<br><br>--- On <b>Thu, 5/17/12, Sarah Sharp <i><sarah.a.sharp@linux.intel.com></i></b> wrote:<br><blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px;"><br>From: Sarah Sharp <sarah.a.sharp@linux.intel.com><br>Subject: Re: [RFC 13/13] USB: Disable hub-initiated LPM for comms devices.<br>To: "Tilman Schmidt" <tilman@imap.cc><br>Cc: gigaset307x-common@lists.sourceforge.net, libertas-dev@lists.infradead.org, "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>, linux-usb@vger.kernel.org, linux-wireless@vger.kernel.org, users@rt2x00.serialmonkey.com, linux-bluetooth@vger.kernel.org, ath9k-devel@lists.ath9k.org, "Alan Stern" <stern@rowland.harvard.edu>, "Hansjoerg Lipp" <hjlipp@web.de>, netdev@vger.kernel.org<br>Date: Thursday, May 17, 2012, 1:31 PM<br><br><div
 class="plainMail">On Thu, May 17, 2012 at 07:07:32PM +0200, Tilman Schmidt wrote:<br>> Am 16.05.2012 23:55, schrieb Sarah Sharp:<br>> > Set the disable_hub_initiated_lpm flag for for all USB communications<br>> > drivers.  I know there aren't currently any USB 3.0 devices that<br>> > implement these class specifications, but we should be ready if they do.<br>> <br>> I follow the argument for class drivers. But this patch also<br>> modifies drivers for specific existing USB 2.0 only devices<br>> which are unlikely to ever grow USB 3.0 support, such as the<br>> Gigaset ISDN driver:<br>> <br>> >  drivers/isdn/gigaset/bas-gigaset.c            |    1 +<br>> >  drivers/isdn/gigaset/usb-gigaset.c            |    1 +<br><br>Is there a particular reason why you think that driver is unlikely to<br>ever get USB 3.0
 support?  I pretty much grabbed any USB driver that<br>looked like a communications driver without looking too closely at the<br>code.<br><br>> What is the interest of setting the disable_hub_initiated_lpm<br>> flag for these?<br><br>It's partially to lay the foundation for anyone who wants to make a USB<br>3.0 communications driver in the future.  They're likely to start from<br>some USB 2.0 class driver, and copy a lot of code.  If they notice that<br>flag is set in all the USB communications class drivers, they're likely<br>to set it as well.<br><br>I'm not quite sure where the best place to provide documentation on the<br>flag is.  I've added the kernel doc comments to the structure, but maybe<br>it needs to be documented somewhere in Documentation/usb/?<br><br>Sarah Sharp<br><br>_______________________________________________<br>libertas-dev mailing list<br><a ymailto="mailto:libertas-dev@lists.infradead.org"
 href="/mc/compose?to=libertas-dev@lists.infradead.org">libertas-dev@lists.infradead.org</a><br><a href="http://lists.infradead.org/mailman/listinfo/libertas-dev" target="_blank">http://lists.infradead.org/mailman/listinfo/libertas-dev</a><br></div></blockquote></td></tr></table>