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

[1.1.x] Add MKS MINI 12864 support to GT2560 #8048

Merged
merged 2 commits into from Oct 25, 2017
Merged

[1.1.x] Add MKS MINI 12864 support to GT2560 #8048

merged 2 commits into from Oct 25, 2017

Conversation

madmo
Copy link

@madmo madmo commented Oct 21, 2017

  • Add pin configurations to pins_GT2560_REV_A for MKS MINI 12864
  • Update sample configuration to enable SDSUPPORT if MKS_MINI_12864 is selected

@@ -1735,7 +1735,7 @@
/**
* Customize common displays for GT2560
*/
#if ENABLED(ULTIMAKERCONTROLLER) || ENABLED(REPRAP_DISCOUNT_SMART_CONTROLLER) || ENABLED(G3D_PANEL)
#if ENABLED(ULTIMAKERCONTROLLER) || ENABLED(REPRAP_DISCOUNT_SMART_CONTROLLER) || ENABLED(G3D_PANEL) || ENABLED(MKS_MINI_12864)
#define SDSUPPORT // Force SD Card support on for these displays
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that this makes it harder for users to disable SDSUPPORT for debugging, which was the intent of having it as a separate option, disabled by default. In the long run, we may set a flag for displays that have SD built-in and throw out a warning to users who forget to enable SDSUPPORT on those displays.

@thinkyhead thinkyhead changed the title Changes to add MKS MINI 12864 support for GT2560 [1.1.x] Add MKS MINI 12864 support to GT2560 Oct 25, 2017
@thinkyhead thinkyhead merged commit a12d1b2 into MarlinFirmware:bugfix-1.1.x Oct 25, 2017
kForth pushed a commit to kForth/Marlin that referenced this pull request Jun 22, 2022
[1.1.x] Add MKS MINI 12864 support to GT2560
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