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

Enable BigTreeTech GTR onboard I2C EEPROM by default #18029

Merged

Conversation

thisiskeithb
Copy link
Member

@thisiskeithb thisiskeithb commented May 18, 2020

Description

The BigTreeTech GTR ships with a real/onboard EEPROM, so this PR enables it by default. Based on GTR pin file from the official GTR repo.

I bench tested a GTR with these changes and EEPROM settings were saved/retrieved/reset successfully, but it has been reported to work successfully in the GTR Facebook group as well.

Also: #undef E2END was used, otherwise there'd be a ton of redefined warnings.

Benefits

Use onboard/real EEPROM on the GTR.

Related Issues

Co-authored-by: NAPCAL <47440988+NAPCAL@users.noreply.github.com>
@thinkyhead
Copy link
Member

I'm working on replacing use of E2END with EEPROM_SIZE. Then the E2END eeprom size from the platform will just be the final fallback. This way we won't have to #undef E2END anywhere. Should be ready soon…

@thinkyhead thinkyhead merged commit 4e51240 into MarlinFirmware:bugfix-2.0.x May 20, 2020
@thisiskeithb thisiskeithb deleted the pr/fix_btt_gtr_eeprom branch May 20, 2020 20:57
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.

2 participants