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

[BUG] KILL, : PRINTER HALTED Please Reset #26903

Closed
1 task done
DrUnKeNTiGeR opened this issue Mar 25, 2024 · 9 comments · Fixed by #26908
Closed
1 task done

[BUG] KILL, : PRINTER HALTED Please Reset #26903

DrUnKeNTiGeR opened this issue Mar 25, 2024 · 9 comments · Fixed by #26908

Comments

@DrUnKeNTiGeR
Copy link

Did you test the latest bugfix-2.1.x code?

Yes, and the problem still exists.

Bug Description

Marlin bugfix 2.1.x with ender 2 pro configs compiles and flashes fine when it boots i get KILL, : PRINTER HALTED Please Reset error we tried looking at terminal but octoprint nor prontor face will connect..

Bug Timeline

New when I flashed firmware

Expected behavior

To boot into a usable state

Actual behavior

Error at boot

Steps to Reproduce

  1. turn on printer
  2. wait
    Configuration.txt
    Configuration_adv.txt

Version of Marlin Firmware

bugfix 2.1.x

Printer model

Ender 2 Pro

Electronics

None

LCD/Controller

OEM

Other add-ons

Nome

Bed Leveling

MBL Manual Bed Leveling

Your Slicer

Prusa Slicer

Host Software

SD Card (headless)

Don't forget to include

  • A ZIP file containing your Configuration.h and Configuration_adv.h.

Additional information & file uploads

No response

@thisiskeithb
Copy link
Member

I’ll mark this as confirmed since we went through some troubleshooting on Discord. Unfortunately, I don’t have an HC32-based board, but it was clearly halting with current bugfix-2.1.x, but does not halt with Creality’s firmware for the same board/printer.

Tagging @shadow578 to see if they’ve had any reports with this board/printer or other HC32-based machines on current bugfix-2.1.x.

@DrUnKeNTiGeR
Copy link
Author

to make it work comment out #define KILL_PIN EXP1_04_PIN in pins_CREALITY_ENDER2P_V24S4.h

boots like a champ now!!

@shadow578
Copy link
Contributor

seems like the KILL_PIN in pins_CREALITY_ENDER2P_V24S4.h is set incorrectly, as both KILL_PIN and X_STEP_PIN are set to pin PC1.
Since commenting the KILL_PIN define makes the printer work, i'm assuming that it's the KILL_PIN that is wrong, not
X_STEP_PIN).


@DrUnKeNTiGeR could you do the following:

(1) check if LCD display, rotary encoder and beeper work?
If yes, then only the EXP1_04 pin is wrong.

(2) if possible confirm the pinout of the EXP1 header on your mainboard.
to do so, use a multimeter in continuity mode and test what pins on the MCU connect to the expansion header.
if the components in (1) all worked, you'd really only have to check pin 4, but it may be a good idea to use the other pins as practice.

please double-check the following:

  • the printer should be turned off
  • the lcd ribbon is disconnected from the expansion header while probing
  • the orientation of the expansion header is correct (next to pin 5, there should be a notch in the plastic)
  • the orientation of the MCU is correct (pin 1 is denoted using a 'dot' on the chip, and the text should be right-side up; see image below)
  • the text on the MCU should read HC32F460KETA or HC32F460KCTA
  • the MCU should have 16 pins on either side, for a total of 64 pins.

EXP1 pinout:

      ------
PC6  | 1  2 | PC7
PA2  | 3  4 | PC1 (?!)
PB13   5  6 | PB14
PB15 | 7  8 | PB12
GND  | 9 10 | 5V
      ------

MCU reference image, pin one is marked by a dot on the chip as well as a triangle on the pcb.

hc32f460 on aquila mainboard

HC32F460KETA pinout reference:

HC32F460KETA pinout reference

@thisiskeithb
Copy link
Member

I think PC1 was added to the LCD pins incorrectly in 6257962 / #26730 (comment)

I planned to disable KILL_PIN once I’m back at a computer and can submit a PR.

@ellensp
Copy link
Contributor

ellensp commented Mar 25, 2024

According to a hires scan of the board I found online https://github.com/n4bstar/ce2p

the back of the boards says this

back

Marlin pins are flipped left to right, as the image is from the back.

According to this page https://gist.github.com/SteveGotthardt/33bdef27d1430f62b4fee5d723d0cde9
These are the pins (converted for marlin)

      ------
PC6  | 1  2 | PC7 
PA2  | 3  4 | PA3 
PB13   5  6 | PB14
 NC  | 7  8 | PB12
GND  | 9 10 | 5V
      ------
#define EXP1_01_PIN                         PC6
#define EXP1_02_PIN                         PC7
#define EXP1_03_PIN                         PA2
#define EXP1_04_PIN                         PA3
#define EXP1_05_PIN                         PB13
#define EXP1_06_PIN                         PB14
#define EXP1_07_PIN                         -1
#define EXP1_08_PIN                         PB12


Update Marlin/src/pins/hc32f4/pins_CREALITY_ENDER2P_V24S4.h with this new information

@ellensp
Copy link
Contributor

ellensp commented Mar 25, 2024

@DrUnKeNTiGeR please confirm the LCD still work with these new pins and it doesn't HALT

@DrUnKeNTiGeR
Copy link
Author

yes lcd and speaker work, i havent printed yet but those do work fine.

@ellensp
Copy link
Contributor

ellensp commented Mar 26, 2024

PR created, will be added to Marlin code base soon.

Copy link

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.

@github-actions github-actions bot locked and limited conversation to collaborators May 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants