-
-
Notifications
You must be signed in to change notification settings - Fork 503
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
Use a single generic eep file for resetting EEPROM #75
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.
Tested this on DFU (atmega32u4 and atmega32u2), and a Pro Micro with caterina bootloader, all on Windows.
Seems to work perfectly!
Ugh, it doesn't want to work now, with additional testing. So, yeah, we do need the erase :( |
It might just need |
That seems to do the trick:
|
--force doesn't work for me Okay. I see the issue. We're using an old version of dfu-programmer in Windows. We need to use |
Nope, that still doesn't work on Windows :'( |
What version is it on Windows? The macOS version is 0.7.0, and |
Okay, I'm back at my VS env... Also, the dfu-programmer versions are definitely the same. |
0.7.2 for Windows. And it wasn't for me. To clarify with some testing.... This works fine for the 32u4, but the 32u2 .... it will not write, even with force. |
Though, I think for now, that should be fine. Though... if we keep with the Toolbox, the "reset eeprom" may be better as an option for flashing, rather than a separate thing. |
What's the output you get from trying to reset a 32U2? |
Nothing helpful.
And debugging info didn't dive anything useful |
Okay, it appears to still work... |
Description
Removed the MCU-specific
atmega32u4_eeprom_reset.hex
for a more universal.eep
file which sets the first byte to0x00
so that the first two bytes are no longer0xFEED
(which should trigger EEPROM reinit).Works on macOS with a DFU keyboard. Windows, and Caterina on macOS still need testing.
Types of Changes
Issues Fixed or Closed by This PR