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

Updated CR-10 Max Configuration.h #981

Merged
merged 1 commit into from
Oct 22, 2023
Merged

Updated CR-10 Max Configuration.h #981

merged 1 commit into from
Oct 22, 2023

Conversation

DakotaCodes00
Copy link
Contributor

@DakotaCodes00 DakotaCodes00 commented Oct 20, 2023

Description

For some odd reason the CR-10 Max uses pin 19 for the BL Touch sensor pin and not the default Z- endstop pin or Z Probe Pin as defined in the ramps pin file.

Previously used this config and the BL Touch never triggered, explicitly using pin 19 fixed the issue.

I also just changed the machine name to be "CR-10 Max".

Hope this helps, thanks to all the contributors who make Marlin Possible, YOU ROCK! :b

Benefits

  • Makes BLTouch Probe work on the CR-10 Max
  • Makes the printer name "CR-10 Max"

Related Issues

  • None as far as I know

Biggest fix is the Z_MIN_PROBE_PIN

For some odd reason the CR-10 Max uses pin 19 for the BL Touch sensor pin and not the default Z- endstop pin or Z Probe Pin as defined in the ramps pin file.

Previously used this config and the BL Touch never triggered, explicitly using pin 19 fixed the issue.

I also just changed the machine name to be "CR-10 Max".

Hope this helps, thanks to all the contributors who make Marlin Possible, YOU ROCK! :b
@ellensp
Copy link
Contributor

ellensp commented Oct 20, 2023

Nice catch, thanks.

@DakotaCodes00
Copy link
Contributor Author

@ellensp No problem, glad I was able to help :)

@thinkyhead thinkyhead merged commit cdd02f0 into MarlinFirmware:import-2.1.x Oct 22, 2023
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.

None yet

3 participants