MarlinFirmware / Marlin

Marlin is an optimized firmware for RepRap 3D printers based on the Arduino platform. Many commercial 3D printers come with Marlin installed. Check with your vendor if you need source code for your specific machine.
https://marlinfw.org
GNU General Public License v3.0
16.19k stars 19.22k forks source link

[BUG] Incorrect PIN assignment in pins_RAMPS.h #15377

Closed julianwelbyeverard-zz closed 4 years ago

julianwelbyeverard-zz commented 5 years ago

Description

Incorrect PIN assignment in pins_RAMPS.h (both bugfix 1.1.x and bugfix 2.0.x branches)

When using a MKS_MINI_12864 display on a BOARD_MKS_GEN_L board that the assignment pin number in pins_RAMPS.h

#define KILL_PIN          41

should be

#define KILL_PIN          64

both my Display and Board are Original and not clones

Steps to Reproduce

set the board to

#define MOTHERBOARD BOARD_MKS_GEN_L

Display to

#define MKS_MINI_12864

Compile and upload

Expected behaviour: The display should show boot screen and information etc

Actual behaviour: The does not show boot screen and information etc

Additional Information

Change the pin in pins_RAMPS as above #define KILL_PIN 64 and display now works (note: I am not sure of the effect elsewhere of this change but in my tests, all seems ok) (Hardware, MKS GEN L board, MKS MINI 12864, TMC2208_STANDALONE on XYZ and A4988 on E, Dual Z on E1 TMC2208_STANDALONE , BLTouch as per example setup)

thinkyhead commented 5 years ago
julianwelbyeverard-zz commented 5 years ago

That I don't know as I only have MKS Gen-L boards, I found this late last night and have not had the time dig through code in more depth, hopefully, someone might be able to check on another board,

GMagician commented 5 years ago

I have a mega2560 with ramps and kill button on lcd is working (as I remember) so I think this doesn't apply to all ramp boards

thisiskeithb commented 5 years ago

Using the latest bugfix-2.0.x commit (665e45e0bad4254352330884a846c21aad2a2ff0) and using the Tevo Michelangelo config (BOARD_MKS_GEN_L & MKS_MINI_12864) & flashing it to my MKS GEN-L v1.0 with MKS Mini 12864 v2.0, the kill pin works fine and the bootscreen is shown/the LCD works fine as well.

thinkyhead commented 5 years ago

@julianwelbyeverard — Is there any extra version information silkscreened onto your board? It sounds like you might have a unicorn.

julianwelbyeverard-zz commented 5 years ago

@julianwelbyeverard — Is there any extra version information silkscreened onto your board? It sounds like you might have a unicorn.

IMG_1143 IMG_1144 IMG_1145

Attached is both photos of the front and back of the boards.. hope this helps

boelle commented 4 years ago

@julianwelbyeverard did you try what @thisiskeithb suggested?

and have you tried latest bugfix 2.0.x ?

github-actions[bot] commented 4 years ago

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.