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

Update DFU flashing and new docs #1040

Merged
merged 2 commits into from
May 9, 2022

Conversation

C-Glick
Copy link
Contributor

@C-Glick C-Glick commented May 9, 2022

From issue #923

With this update I've fixed a few issues with the automatic DFU flashing.

  • I add the $(srctree) variable to python script call
  • I added a flash_dfu_manual make target to flash the firmware assuming the crazyflie is already in DFU mode. This makes it easier for users to flash using DFU mode when the crazyflie cannot run the normal firmware to support the automatic reboot into DFU mode
  • Finally, I updated the documentation to reflect the new make targets. I decided to leave the warning at the top of the documentation as DFU mode can still be dangerous for new/inexperienced users.

One issue that I've noted is dfu-util complains about an invalid suffix signature. This doesn't seem to cause any issues now but could in the future.

dfu-util: Invalid DFU suffix signature
dfu-util: A valid DFU suffix will be required in a future dfu-util release!!!

I've also noticed that the make file complains about old recipes for flash_dfu, this may be due to the .PHONY target, I'm not familiar with what its purpose is so I left it as is.

Let me know if any changes need to be made before merging. Thanks!

Copy link
Member

@tobbeanton tobbeanton left a comment

Choose a reason for hiding this comment

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

I think it all looks good, many thanks!

The warnings:

dfu-util: Warning: Invalid DFU suffix signature
dfu-util: A valid DFU suffix will be required in a future dfu-util release!!!

I think is because we are flashing a binary file.

@tobbeanton tobbeanton merged commit 1703a9b into bitcraze:master May 9, 2022
@knmcguire knmcguire added this to the next release milestone May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants