-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Update Tinyusb #2379
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
Update Tinyusb #2379
Conversation
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.
nrf build break since it requires nrfx update to version 2.x to support new mcu nrf52833 and nrf5340. @dhalbert is there any issue upgrading nrfx to v2, I can help to do that as well.
@hathach There is probably not an issue upgrading to nrfx 2.x, but I already made changes to 1.x which will need to be merged (make |
Got it, I will help to submit PR to update our nrfx folk to v2 & cpython. It has an API changes to require even single module e.g NRF_USBD must be passed to driver function as well. |
@hierophect I made a PR to your branch to fix warnings https://github.com/hierophect/circuitpython/pull/1 I am upgrading nrfx for circuitpython to v2.0 (will submit a separate PR), once this is done, the travis build should pass, and we will also be able to add support for nrf52833 and nrf5340. |
once this PR get merged #2395 we could fetch from that, and hopefully the travis build could be fixed. |
stm32 explicitily enable/disable vbus sense, fix warning
@hathach thanks for the PR, I didn't have the explicit disable which might have caused problems down the line. |
@hierophect I submit another PR to sync with master tips, though since the master is very actively developed, there is quite a bit of chances. But I think it is safe to merge, you may want to double check and/or review the changes yourself https://github.com/hierophect/circuitpython/pull/2 . Hopefully once it is merged this PR is ready as well. |
merge with current master of circuitpython
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.
except for maybe rename BOARD_NO_VBUS
to BOARD_NO_VBUS_SENSE
, I think this PR is ready to merge. @hierophect do you want to make any other changes for the PR ?
Merged to fix issue #2421 we could rename things later on if needed. |
Works on feather_stm32f405 connected to a Linux system! Thanks! |
@hathach thanks for picking up the slack on this while I was traveling, sorry I wasn't able to update promptly. I'll get a PR in for the text fix today. |
@hathach I'm getting git bugs with tinyusb when I use Happens even after a fresh clone. Something to look into? |
@hierophect no problems at all, I am not in hurry If not for fixing the urgent bug with feather stm32f405. Regarding the submodule, I am not so sure why even a fresh clone didn't work hmmmm. Please try to follow Scott suggestion hathach/tinyusb#242 to see if that works. Scott is git master :) |
This PR updates to the latest version of TinyUSB to access manual VBUS enabling for STM32 microcontrollers. Currently, it compiles on all ports but NRF, which has some renamed structures.
I'd appreciate any help in testing across ports and boards since I don't currently have most of them on hand! Edits of any kind are welcome.