Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Feature/dfu #552

Merged
merged 25 commits into from
Nov 17, 2022
Merged

Feature/dfu #552

merged 25 commits into from
Nov 17, 2022

Conversation

xmos-jmccarthy
Copy link
Contributor

@xmos-jmccarthy xmos-jmccarthy commented Nov 9, 2022 •
edited
Loading

Add DFU example over USB.

Uses a new DFU image driver, which utilized libquadflash at boot to create an image table for the app to use.

Closes #551

@xmos-jmccarthy
Update tusb DFU demo
170a3ca
@xmos-jmccarthy
Add dfu app skeleton, fix strange xscope bug
4157b4b 
Link order appears to cause the dfu app to break when acquiring the
xscope lock for no discernable reason.  Save commit for later
investigation.

To reproduce in the dfu bsp_config add rtos::drivers::usb before
rtos::sw_services::usb in the target_link_libraries call.
@xmos-jmccarthy
Fix align bug between tusb and xud
b4dd1b1
@xmos-jmccarthy
Update rtos submodule
e29c1dd
@xmos-jmccarthy
Add dfu example over usb
8ed15c1
@xmos-jmccarthy xmos-jmccarthy changed the base branch from main to develop November 9, 2022 22:34
@xmos-jmccarthy
Copy link
Contributor Author

Marking do not merge.
TODO:
Update readme
Resolve rtos submodule conflict.
Add cmake command to create upgrade image
Add cmake command to flash with factory image

@xmos-jmccarthy xmos-jmccarthy mentioned this pull request Nov 10, 2022
Closed
2 tasks
keithm-xmos
keithm-xmos approved these changes Nov 11, 2022
View reviewed changes
Copy link
Contributor

@keithm-xmos keithm-xmos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rubber stamp

@keithm-xmos
Copy link
Contributor

I noticed that the RTOS core examples do not build.

https://github.com/xmos/xcore_sdk/actions/runs/3440356711/jobs/5738743733

I wonder if this is because the Docker builder uses XTC 15.1.0. We need to update to a newer version but that may be a couple weeks away.

@xmos-jmccarthy
Copy link
Contributor Author

Do not merge yet. Adding DFU support for Tools 15.1.4.

@xmos-jmccarthy
Copy link
Contributor Author

Moved QSPI spec out of DFU driver to allow users to have more flexibility. This enables support for Tools 15.1.3 and 15.1.4 at the cost of a little extra explicit initialization code.

Tested on Linux with 15.1.3, 15.1.4, and 15.2.0.

@xmos-jmccarthy
Copy link
Contributor Author

#553
All OS verified.

Readme updated for potential Windows FAQ.

@xmos-jmccarthy xmos-jmccarthy merged commit 68dd860 into develop Nov 17, 2022
@keithm-xmos keithm-xmos deleted the feature/dfu branch November 21, 2022 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@keithm-xmos keithm-xmos keithm-xmos approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xmos-jmccarthy @keithm-xmos