On 10/28/2016 3:09 AM, Felipe Balbi wrote: > > Hi, > > John Youn <John.Youn@xxxxxxxxxxxx> writes: >> On 10/27/2016 3:47 AM, Felipe Balbi wrote: >>> >>> Hi, >>> >>> John Youn <johnyoun@xxxxxxxxxxxx> writes: >>>> Add interrupt moderation interval binding for dwc3. >>>> >>>> Signed-off-by: John Youn <johnyoun@xxxxxxxxxxxx> >>>> --- >>>> Documentation/devicetree/bindings/usb/dwc3.txt | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/Documentation/devicetree/bindings/usb/dwc3.txt b/Documentation/devicetree/bindings/usb/dwc3.txt >>>> index e3e6983..17de9fc 100644 >>>> --- a/Documentation/devicetree/bindings/usb/dwc3.txt >>>> +++ b/Documentation/devicetree/bindings/usb/dwc3.txt >>>> @@ -53,6 +53,7 @@ Optional properties: >>>> - snps,quirk-frame-length-adjustment: Value for GFLADJ_30MHZ field of GFLADJ >>>> register for post-silicon frame length adjustment when the >>>> fladj_30mhz_sdbnd signal is invalid or incorrect. >>>> + - snps,imod_interval: the interrupt moderation interval. >>> >>> on top of all other comments, what's the unit here? nanoseconds? clock cycles? >>> >> >> Number of 250 ns intervals. I'll update the description to clarify. > > it's probably better to add it in nanoseconds itself, then let driver > compute register value with DIV_ROUND_UP() > I'm fine with it either way, but I think "increments of 250 ns" is slightly cleaner in that it reflects the exact settings that are possible and documented, and also fits neatly into a u16. I assume you'd also like the internal variable dwc3->imod_interval changed as well? Regards, John -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html