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-util to v0.8 #29

Merged
merged 2 commits into from
Feb 13, 2019
Merged

Update dfu-util to v0.8 #29

merged 2 commits into from
Feb 13, 2019

Conversation

chrissbarr
Copy link
Contributor

@chrissbarr chrissbarr commented Oct 5, 2018

Updating dfu-util and ensuring consistency across platforms.

  • win was v0.1+svn, updated to v0.8 - folder structure changed to match other platforms.
  • linux is v0.8, no change needed.
  • linux64 was v0.7, updated to v0.8.
  • macosx is v0.8, no change needed.

I settled on v0.8 even though v0.9 is available because it looks like v0.9 only has x64 Windows binaries, which might cause problems for anyone running x86. v0.8 seems to work fine and was already in use for linux and macosx platforms anyway.

Things that I'd like some input on:

  • I moved the updated dfu-util in the win dir to its own dfu-util subdir, as this is how linux, linux64 and macosx are structured. This keeps things tidier (there are a few dfu-util files) and consistent across platforms. Does anyone know a reason not to do that?

Updating dfu-util and ensuring consistency across platforms.

Windows was v0.1, now is v0.8 - structure changed to match other platforms.
Linux was v0.8, no change needed.
Linux64 was v0.7, updated to v0.8.
OSX was v0.8, no change needed.
Not entirely sure that this is associated (or solely associated) with dfu-util.
@fpistm fpistm self-requested a review October 8, 2018 06:54
@fpistm fpistm added the enhancement New feature or request label Oct 8, 2018
@fpistm
Copy link
Member

fpistm commented Oct 8, 2018

This repo was originally based on https://github.com/rogerclarkmelbourne/Arduino_STM32/tree/master/tools/
So all things you raised comes from this legacy.
Anyway, it's nice to align the folder tree across all host OS.
I've just do the same for stm32flash.
About v0.9 of dfu utils, I don't know if we need to move on this version.
I can rebuild all if this is requested.

@chrissbarr
Copy link
Contributor Author

This repo was originally based on https://github.com/rogerclarkmelbourne/Arduino_STM32/tree/master/tools/
So all things you raised comes from this legacy.

That makes sense, from memory STM32GENERIC (which I think also inherited the tools setup from Roger) also had outdated dfu-util versions initially. I see Roger has added dfu-util v0.9 to his repo for "F4 DFU testing", but hasn't replaced the older dfu-util version with it yet - possibly because v0.9 is x64 only, as I mentioned previously.

I think v0.8 works well enough. There is a changelog here - I don't see anything that would be needed in the STM32 upload process. I think the main problem was that the version for Windows here was very old (v0.1) - it looks like anything after v0.5 (where they added DfuSe support) should work with most STM32 chips.

@rogerclarkmelbourne
Copy link
Contributor

rogerclarkmelbourne commented Oct 9, 2018

@chrissbarr

There was definitely a problem with the version of DFU which supported DFUSe, as it would not work with the STM32duino booloader, (I can't remember the details)
Hence I kept the old version of DFUUtil which was know to work on most platforms.

BTW.
I suspect in the end, a lot of people may switch to use the new HID based bootloader that Vassilis now administers, but AFIK there is only an upload program for Windows, (or there are only binaries (exe) for Windows)

And people seem fairly OK with using the dfu based bootloader at the moment, as you still need to load drivers (or udev rules) for the Serial, and hence the installation would be no easier even with the HID bootloader

@fpistm fpistm merged commit d160199 into stm32duino:master Feb 13, 2019
@fpistm fpistm mentioned this pull request Feb 13, 2019
@chrissbarr chrissbarr deleted the dfu-util_update branch May 30, 2019 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants