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

Wanhao One+ SD detect pin #20724

Merged

Conversation

antman2
Copy link
Contributor

@antman2 antman2 commented Jan 9, 2021

Description

Define SD_DETECT_PIN for Wanhao One+ board to enable SD card support.

Benefits

Prior to this Marlin was not detecting SD card insertion on a Cocoon Create ModelMaker (a Wanhao Duplicator i3 Mini rebrand).

Configurations

This allows Configuration.h's SDSUPPORT define to function as expected:

/**
 * SD CARD
 *
 * SD Card support is disabled by default. If your controller has an SD slot,
 * you must uncomment the following option or it won't work.
 */
#define SDSUPPORT

Related Issues

  • Wanhao Duplicator i3 Mini & Rebrands Support #12425
  • Wanhao Duplicator i3 Mini & Rebrands Configuration #14559

Additional Information

An old Cocoon Create ModelMaker joined the printer family this week. After backing up Marlin 1.1.4 and upgrading to 2.0.7.2 it was observed that the SD card was no longer working. Continuity probing showed the Card Detect connected to m2560 pin 49 which, in src/HAL/AVR/fastio/fastio_1280.h maps to Port D6/Logical Pin 83.

@thinkyhead thinkyhead merged commit d78f292 into MarlinFirmware:bugfix-2.0.x Jan 10, 2021
@thinkyhead thinkyhead changed the title Define SD_DETECT_PIN for Wanhao One+ board to enable SD card support. Wanhao One+ SD detect pin Jan 10, 2021
kpishere pushed a commit to kpishere/Marlin that referenced this pull request Feb 19, 2021
zillarob pushed a commit to zillarob/Marlin that referenced this pull request Feb 25, 2021
W4tel-BiDi pushed a commit to W4tel-BiDi/Marlin that referenced this pull request Apr 5, 2021
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Apr 29, 2021
thinkyhead pushed a commit that referenced this pull request Apr 30, 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.

None yet

2 participants