-
Notifications
You must be signed in to change notification settings - Fork 1.2k
dwc2/host: enable disconnect interrupt + handle it #3067
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
dwc2/host: enable disconnect interrupt + handle it #3067
Conversation
Signed-off-by: Maxime Vincent <[email protected]>
Hi, I ended up with the same thing in #2960 I also removed |
Aha, nice! It doesn't matter to me, whichever you can get merged first :) |
Yes please, just double checked ST's HAL code.
@hathach is busy atm, since dwc2 is widely used it's better wait for Espressif @roma-jam for a double check. Out of topic: Have you experienced any bouncing issue doing quickly connection/disconnection like in #2960 ? |
… to prevent double removal Signed-off-by: Maxime Vincent <[email protected]>
ebb3b6e
to
6607b76
Compare
|
Can we merge this? Is there any more action required from my side? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's reliable in my end, @hathach do you need more test ?
perfect, thank you |
Describe the PR
On boards using the Synopsys DWC2 USB IP core, there were no disconnection events when a USB device was unplugged.
This PR enabled the
GINTSTS_DISCINT
flag, to get an interrupt on disconnection + handles it by callinghcd_event_device_remove()
.Additional context
Tested on an STM32F7, but behavior should be identical on any device using the
dwc2