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
[1.1.x] Fix EEPROM CRC #8283
[1.1.x] Fix EEPROM CRC #8283
Conversation
Fix EEPROM reversion.
I verified this fixes the EEPROM CRC issue for bugfix-v1.1.x |
@@ -1088,7 +1088,7 @@ void MarlinSettings::postprocess() { | |||
stepperE4.setCurrent(val, R_SENSE, HOLD_MULTIPLIER); | |||
#endif | |||
#else | |||
for (uint8_t q = 11; --q;) EEPROM_READ(val); | |||
for (uint8_t q = 0; q < 11; q++) EEPROM_READ(val); |
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.
This doesn't change the "read order," ya know.
The only issue was that it should've been q--
instead of --q
to get the correct number of items.
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.
Yes... And that is why I reverted #8284. I didn't test the bugfix-v2.0.0 branch, but it looked like it was OK.
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.
Oh duh... I'm smart... sorry guys.
Fix EEPROM reversion.
Fix EEPROM reversion.
Changing the order in which the EEPROM is read changes the CRC. e642a64 changed the EEPROM read order without changing the write order. I reverted the problematic line so that I can read from the EEPROM again.