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

Supporting REPRAP_DISCOUNT_SMART_CONTROLLER #23522

Closed
wants to merge 1 commit into from

Conversation

fduraibi
Copy link
Contributor

Adding support for the REPRAP_DISCOUNT_SMART_CONTROLLER LCD module with the Ramps 1.4 Plus board

Description

The current code only supports the REPRAPWORLD_GRAPHICAL_LCD (LCD12864) and doesn't support the REPRAP_DISCOUNT_SMART_CONTROLLER (LCD2004).
This PR, which I have tested and currently running on my printer, fixes the issue by assigning the right pins to the board. I have also updated the comment field with the Pin-out of the boards connectors EXP1 and EXP2.

Benefits

Fixes the pins for the LCD board REPRAP_DISCOUNT_SMART_CONTROLLER when used with the RAMPS 1.4 Plus

Related Issues

#23501

Images

This image shows the board pin-out

LCD_Interface

The LCD pin mapping

LCD2004_IO_Ports

Adding support for the REPRAP_DISCOUNT_SMART_CONTROLLER LCD module with the Ramps 1.4 Plus board

This closes the issue MarlinFirmware#23501
@github-actions
Copy link

Thanks for your contribution! Unfortunately we can't accept PRs directed at release branches. We make patches to the bugfix branches and only later do we push them out as releases.
Please redo this PR starting with the bugfix-2.0.x branch and be careful to target bugfix-2.0.x when resubmitting the PR.
It may help to set your fork's default branch to bugfix-2.0.x.
See this page for full instructions.

@github-actions github-actions bot closed this Jan 14, 2022
@fduraibi fduraibi deleted the patch-1 branch January 14, 2022 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant