-
Notifications
You must be signed in to change notification settings - Fork 1k
Add dfu-util as a standard upload method #718
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
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.
Some remarks about the PR.
Anyway, I will not merge this until the automatic reset in BL mode is not finished.
I'm also thinking about merge the STM32CubeProgrammer script and the dfu-util.
Co-Authored-By: Frederic Pillon <[email protected]>
Co-Authored-By: Frederic Pillon <[email protected]>
The automatic reset in BL mode PR looks cool. Let me know if there's anything I can do to help with that, too. |
Any updates on this, I also believe that dfu-util will be a great feature to have as a default. It looks like the dependencies have been merged, what is outstanding to get this pull merged? |
Same answer as before:
Unfortunately, I had no time to work on this soon but it will come. |
Is there anything I can do to help? My free time is a bit sporadic, and im not the most experienced in this field, but i want to help with the project in general. In the mean time I will have a look through the list of requested boards and see if i can get some of them added as PR's |
Unfortunately i just spent all night figuring out how to do exactly what jgillick had already done because i didn't know about this PR, and I was about to make the same PR. The automatic reset works on my board with :leave (which was removed) so I don't know what the holdup is. Please merge the work! |
@jerkey Which board was removed ? |
i was referring to this commit where ":leave" was removed from the dfu command. I don't understand why that commit was made, and you said "Anyway, I will not merge this until the automatic reset in BL mode is not finished" and since :leave makes the board start the user program after flashing, it seemed like the right thing to do but maybe this is accomplished somehow in the present PR. FYI the hardware will enter dfu mode if boot0 is pulled high during power/reset (via a button for example) and the :leave leaves dfu mode and runs the program. I don't see how you could improve on this, we don't have a DTR toggle or anything like that - the device doesn't even enable USB unless the user program chooses to do that. The user has to hold their dfu button and press reset to enter dfu mode. That or they can jump boot0 high and always enter dfu mode, but then their program only runs when they flash or otherwise use dfu-util to leave the bootloader. |
I close this PR as it is outdated due to several change in the boards.txt. |
Summary
This PR implements the following features:
dfu-util
as a standard menu upload option.This is a follow-up to (and dependant on) PR stm32duino/Arduino_Tools#46.
Motivation
The
dfu-util
program is already bundled with the stm32duino packages, so setting it as a primary supported upload tool reduces the need for outside software dependencies and gets the end-user setup and running quicker. Additionally, the STM32CubeProgrammer installer is known to have issues on the mac -- there are workarounds, but it's one more hurdle for the end-user to jump through.Validation