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

MARLIN_EEPROM_SIZE with E2END as fallback #18059

Merged

Conversation

thinkyhead
Copy link
Member

@thinkyhead thinkyhead commented May 20, 2020

Using E2END as our primary define for available settings storage space has caveats:

  • Platforms are not obligated to provide E2END as the size of the EEPROM.
  • Pins must intercede to #undef E2END to redefine the value, losing the platform value. It takes extra work to tell if the value comes from Marlin or from the platform, and to check that the override was done in the right place, and so on.

This PR makes Marlin use MARLIN_EEPROM_SIZE to specify the total storage available for settings. (The name EEPROM_SIZE is used by some platforms, so that can't be used if we want it to be overridable in the environment.)

@thinkyhead thinkyhead force-pushed the bf2_eeprom_size_setting_PR branch 4 times, most recently from 4c83653 to 87d72e7 Compare May 21, 2020 00:37
@thinkyhead thinkyhead changed the title Use EEPROM_SIZE with E2END as fallback MARLIN_EEPROM_SIZE with E2END as fallback May 21, 2020
@thinkyhead thinkyhead force-pushed the bf2_eeprom_size_setting_PR branch 4 times, most recently from 0183d94 to 7c4d78b Compare May 22, 2020 07:12
@thinkyhead thinkyhead force-pushed the bf2_eeprom_size_setting_PR branch from 7c4d78b to 230de98 Compare May 22, 2020 07:14
@thinkyhead thinkyhead merged commit 461647f into MarlinFirmware:bugfix-2.0.x May 22, 2020
@thinkyhead thinkyhead deleted the bf2_eeprom_size_setting_PR branch May 22, 2020 07:15
vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request May 29, 2020
jmp0x0000 pushed a commit to jmp0x0000/Marlin that referenced this pull request Aug 7, 2020
HairingX pushed a commit to HairingX/Marlin that referenced this pull request Jun 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant