Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1237732 > unrolled thread

Re: [PATCH v2] usb: dwc2: reset dwc2 core before dwc2_get_hwparams()

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v2] usb: dwc2: reset dwc2 core before dwc2_get_hwparams() Doug Anderson <dianders@chromium.org> - 2015-10-01 23:00 +0200
    Re: [PATCH v2] usb: dwc2: reset dwc2 core before dwc2_get_hwparams() John Youn <John.Youn@synopsys.com> - 2015-10-06 00:10 +0200

#1237732 — Re: [PATCH v2] usb: dwc2: reset dwc2 core before dwc2_get_hwparams()

John,

On Tue, Aug 18, 2015 at 5:19 PM, John Youn <John.Youn@synopsys.com> wrote:
> Hi Yunzhi,
>
> My concern is with the delays due to calling the dwc2_core_reset
> during probe. You could factor out the assertion of the core
> soft reset from the dwc2_core_reset and just use that before
> calling dwc2_get_hwparams().
>
> You had previously addressed the lengthy probe time issue here:
> http://marc.info/?l=linux-usb&m=142357721304377
>
> This reducing delays patch looks reasonable to me and I think it
> should get merged also. I'll do some testing with it later this
> week.

Note: you can also avoid the extra reset during probe with something
like <https://chromium-review.googlesource.com/#/c/303152/>.  I'm
happy to post that up if you want, though it depends on lyz's patch.

...in Chrome OS we've also just landed lyz's patch to reduce delays.
If there's any fallout I'll report on the list.

-Doug
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1239993

On 10/1/2015 1:50 PM, Doug Anderson wrote:
> John,
> 
> On Tue, Aug 18, 2015 at 5:19 PM, John Youn <John.Youn@synopsys.com> wrote:
>> Hi Yunzhi,
>>
>> My concern is with the delays due to calling the dwc2_core_reset
>> during probe. You could factor out the assertion of the core
>> soft reset from the dwc2_core_reset and just use that before
>> calling dwc2_get_hwparams().
>>
>> You had previously addressed the lengthy probe time issue here:
>> http://marc.info/?l=linux-usb&m=142357721304377
>>
>> This reducing delays patch looks reasonable to me and I think it
>> should get merged also. I'll do some testing with it later this
>> week.
> 
> Note: you can also avoid the extra reset during probe with something
> like <https://chromium-review.googlesource.com/#/c/303152/>.  I'm
> happy to post that up if you want, though it depends on lyz's patch.
> 
> ...in Chrome OS we've also just landed lyz's patch to reduce delays.
> If there's any fallout I'll report on the list.
> 
> -Doug
> 


Yes, I appreciate if you could submit that. I'd like to merge lyz's
reduce delays, lyz's bug fix for hwparams, and your fix for double
reset.

The gadget side will also need something similar which I can do if
needed. Do you guys run gadget mode?

Regards,
John

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web