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

update pins pins_CREALITY_ENDER2P_V24S4, solves issue 26903 #26908

Conversation

ellensp
Copy link
Contributor

@ellensp ellensp commented Mar 26, 2024

Description

pins_CREALITY_ENDER2P_V24S4.h has errors and causes KILL pin to be falsely triggered.

Updated to correct pin from information found online. https://gist.github.com/SteveGotthardt/33bdef27d1430f62b4fee5d723d0cde9

Has been tested and confirmed to work.

Requirements

Creality Ender 2 Pro motherboard CR-FDM-V2.4.S4.170

Benefits

Correct Pins

Related Issues

#26903

@thisiskeithb thisiskeithb linked an issue Mar 26, 2024 that may be closed by this pull request
1 task
@sjasonsmith
Copy link
Contributor

@ellensp & @thisiskeithb, how confident are you that this fix is universal? Is it possible there are more than one board being used with this pins file, and it needs to be split up?

I don't know where the original broken values came from, which is why I'm concerned that fixing this for one user might be breaking it for someone else.

@ellensp
Copy link
Contributor Author

ellensp commented Apr 7, 2024

the motherboard is labeled, and the link has pictures with port pins on silkscreen

Picture matches images in orrginal #24338 that was added incorrectly in #26730

So I am very confident this is correct

@sjasonsmith
Copy link
Contributor

I looked through your links and it seems like due-diligence was done and this is correct.

@sjasonsmith sjasonsmith merged commit d1cf817 into MarlinFirmware:bugfix-2.1.x Apr 7, 2024
61 checks passed
@ellensp ellensp deleted the update-pins_CREALITY_ENDER2P_V24S4 branch April 7, 2024 05:58
RPGFabi pushed a commit to RPGFabi/Marlin that referenced this pull request Jun 15, 2024
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.

[BUG] KILL, : PRINTER HALTED Please Reset
2 participants