On Mon, Jul 10, 2017 at 11:38 PM, <mario.limoncie...@dell.com> wrote: >> -----Original Message----- >> From: Jiahau Chang [mailto:jia...@gmail.com] >> Sent: Thursday, July 6, 2017 8:58 PM >> To: Mathias Nyman <mathias.ny...@linux.intel.com> >> Cc: linux-usb@vger.kernel.org; mathias.ny...@intel.com; >> david.lai...@aculab.com; gre...@linuxfoundation.org; >> felipe.ba...@linux.intel.com; brain_...@asmedia.com.tw; Limonciello, Mario >> <mario_limoncie...@dell.com>; justin_cy_c...@asmedia.com.tw; Wang, Keith >> <keith_w...@dell.com>; yd_ts...@asmedia.com.tw; Jiahau Chang >> <lars_ch...@asmedia.com.tw>; Ian Pilcher <arequip...@gmail.com> >> Subject: Re: [PATCH v5] xhci: Bad Ethernet performance plugged in ASM1042A >> host >> >> 2017-06-28 21:42 GMT+08:00 Mathias Nyman <mathias.ny...@linux.intel.com>: >> > On 22.06.2017 07:49, Jiahau Chang wrote: >> >> >> >> When USB Ethernet is plugged in ASMEDIA ASM1042A xHCI host, bad >> >> performance was manifesting in Web browser use (like download >> >> large file such as ISO image). It is known limitation of >> >> ASM1042A that is not compatible with driver scheduling, >> >> As a workaround we can modify flow control handling of ASM1042A. >> >> The register we modify is changes the behavior >> >> >> >> Signed-off-by: Jiahau Chang <lars_ch...@asmedia.com.tw> >> >> Signed-off-by: Ian Pilcher <arequip...@gmail.com> >> >> --- >> > >> > >> > Thanks, looks good, but checkpatch complains about: >> > >> >> + usleep_range(50, 50); >> > >> > >> > having same min and max value. >> > Does usleep_range(40,60) work for you? or some other range? >> > >> It works to use usleep_range(40,60); >> Thanks for help us to upstream the patch. >> >> > I can change that myself, no need to resend. >> > > > Matthias, > > Can you still get this in for 4.13?
He is on vacation for few weeks. Perhaps in rcX. -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html