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.

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

Implement USB DFU #10

Closed
keithm-xmos opened this issue May 10, 2021 · 3 comments · Fixed by #130
Closed

Implement USB DFU #10

keithm-xmos opened this issue May 10, 2021 · 3 comments · Fixed by #130
Assignees
Labels
size:S small type:enhancement New feature or request

Comments

@keithm-xmos
Copy link
Contributor

keithm-xmos commented May 10, 2021

Implement DFU over USB in the STLP voice application.

@keithm-xmos keithm-xmos added the type:enhancement New feature or request label May 10, 2021
@keithm-xmos keithm-xmos added the status:help wanted Extra attention is needed label Jan 1, 2022
ed-xmos referenced this issue in xmos/fwk_voice Feb 9, 2022
@keithm-xmos keithm-xmos transferred this issue from xmos/fwk_voice Feb 25, 2022
@keithm-xmos keithm-xmos transferred this issue from another repository Feb 25, 2022
@keithm-xmos keithm-xmos added the size:S small label Apr 12, 2022
@keithm-xmos keithm-xmos transferred this issue from xmos/xcore_iot May 19, 2022
@keithm-xmos keithm-xmos transferred this issue from another repository May 23, 2022
@mbanth
Copy link
Collaborator

mbanth commented Aug 9, 2022

@keithm-xmos, what help do you want/need for this issue? It may be possible to share portions of the DFU code for sln_voice and xvf3800.

@keithm-xmos
Copy link
Contributor Author

In order for DFU to work nicely in an RTOS application, we have requested a change to the bootloader. In a nutshell, the bootloader will support a user defined function for reading an image from flash. The bootloader will load images from flash as normal if this user defined function is not provided - keeping it backwards compatible. It is not clear if or when this bootloader enhancement will be implemented.

Alternatively, now that we will soon have a new QSPI flash library distributed with the XTC tools, we plan to migrate the RTOS driver to use this library instead of the qspi_io library in fwk_io. The desire is to consolidate to a single QSPI library - the one that comes with the tools. Before this can happen we need to scope out the new library to see if it is a (near) drop in replacement or if more work is needed.

The consolidation to a single QSPI library is a desired enhancement regardless of how we approach DFU in an RTOS application. However, consolidation is a prerequisite if we do not get the requested changes to the bootloader. Also a requirement if we do not get the requested changes to the bootloader is a low-level mechanism to synchronize access to the new QSPI flash library. We will need to ensure that multiple clients are not reading/writing with the library at the same time. This could be an issue if one or more clients are bypassing the RTOS flash driver.

@keithm-xmos keithm-xmos removed the status:help wanted Extra attention is needed label Aug 19, 2022
@keithm-xmos
Copy link
Contributor Author

keithm-xmos commented Aug 19, 2022

Requires xmos/xcore_iot#501 and xmos/fwk_rtos#46

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:S small type:enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants