Re: [PATCH 1/2] usb: xhci: add relaxed timing quirk bit

2017-11-23 Thread Adam Wallis
On 11/23/2017 5:59 AM, Mathias Nyman wrote: > On 23.11.2017 01:32, Adam Wallis wrote: >> On 11/22/2017 10:24 AM, Mathias Nyman wrote: >> [..] >>> >>> We know have at least two hosts/platforms that need custom interrupt >>> moderation >>> values >>> >>> How about adding a u32 device property for xh

Re: [PATCH 1/2] usb: xhci: add relaxed timing quirk bit

2017-11-23 Thread Mathias Nyman
On 23.11.2017 01:32, Adam Wallis wrote: On 11/22/2017 10:24 AM, Mathias Nyman wrote: [..] We know have at least two hosts/platforms that need custom interrupt moderation values How about adding a u32 device property for xhci with the interrupt moderation interval in nanoseconds?  And also add

Re: [PATCH 1/2] usb: xhci: add relaxed timing quirk bit

2017-11-22 Thread Adam Wallis
On 11/22/2017 10:24 AM, Mathias Nyman wrote: [..] > > We know have at least two hosts/platforms that need custom interrupt > moderation > values > > How about adding a u32 device property for xhci with the interrupt moderation > interval in > nanoseconds?  And also add a u32 imod_interval variab

Re: [PATCH 1/2] usb: xhci: add relaxed timing quirk bit

2017-11-22 Thread Adam Wallis
On 11/22/2017 10:24 AM, Mathias Nyman wrote: > On 22.11.2017 02:07, Adam Wallis wrote: >> On 11/21/2017 3:06 PM, Rob Herring wrote: >> >> [..] >> I like where you are going with this. Are you saying that I could read for a device property read from firmware (for DTB or ACPI) like DW

Re: [PATCH 1/2] usb: xhci: add relaxed timing quirk bit

2017-11-22 Thread Mathias Nyman
On 22.11.2017 02:07, Adam Wallis wrote: On 11/21/2017 3:06 PM, Rob Herring wrote: [..] I like where you are going with this. Are you saying that I could read for a device property read from firmware (for DTB or ACPI) like DWC3 does for "snps,hird-threshold"? Is that for the same thing? If so

Re: [PATCH 1/2] usb: xhci: add relaxed timing quirk bit

2017-11-21 Thread Adam Wallis
On 11/21/2017 3:06 PM, Rob Herring wrote: [..] >> I like where you are going with this. Are you saying that I could read for a >> device property read from firmware (for DTB or ACPI) like DWC3 does for >> "snps,hird-threshold"? > > Is that for the same thing? If so, drop the vendor prefix and us

Re: [PATCH 1/2] usb: xhci: add relaxed timing quirk bit

2017-11-21 Thread Rob Herring
On Tue, Nov 21, 2017 at 1:49 PM, Adam Wallis wrote: > On 11/21/2017 2:11 PM, Rob Herring wrote: >> On Tue, Nov 21, 2017 at 12:18:09PM -0500, Adam Wallis wrote: >>> Certain systems may run with CPUs at a very slow frequency. This >>> patch adds a quirk bit that can be used to relax certain timings,

Re: [PATCH 1/2] usb: xhci: add relaxed timing quirk bit

2017-11-21 Thread Adam Wallis
On 11/21/2017 2:11 PM, Rob Herring wrote: > On Tue, Nov 21, 2017 at 12:18:09PM -0500, Adam Wallis wrote: >> Certain systems may run with CPUs at a very slow frequency. This >> patch adds a quirk bit that can be used to relax certain timings, etc. >> >> This quirk might be needed for other fields in

Re: [PATCH 1/2] usb: xhci: add relaxed timing quirk bit

2017-11-21 Thread Rob Herring
On Tue, Nov 21, 2017 at 12:18:09PM -0500, Adam Wallis wrote: > Certain systems may run with CPUs at a very slow frequency. This > patch adds a quirk bit that can be used to relax certain timings, etc. > > This quirk might be needed for other fields in the future, but > initially, it will be used o