> -----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? Thanks,
