End stops won't stop motors from stopping when homing.

I am kind of new to marlin and 3d printing. I have been tinkering here and there with it but I cannot figure out what is wrong. I have a mini kossel 3d printer. I have endstops on the top of the x, y, and z axis to make homing easier at the top. The problem I am having is when i press home, it moves up but once it reaches the endstops, the lights trigger but they fail to stop all 3 motors from spinning making a horrible noise and i am always forced to unplug from the power source. I have included a snapshot of the part of firmware i think is relevant and a picture of my 3d printer set up. Please help! Also, when i send the m119 command it says,
SENDING:M119
Reporting endstop status
x_max: open
y_max: open
z_min: TRIGGERED
z_max: open


Firmware....
// coarse Endstop Settings
#define ENDSTOPPULLUPS // Comment this out (using // at the start of the line) to disable the endstop pullup resistors

#ifndef ENDSTOPPULLUPS
// fine endstop settings: Individual pullups. will be ignored if ENDSTOPPULLUPS is defined
// #define ENDSTOPPULLUP_XMAX
// #define ENDSTOPPULLUP_YMAX
// #define ENDSTOPPULLUP_ZMAX
// #define ENDSTOPPULLUP_XMIN
// #define ENDSTOPPULLUP_YMIN
// #define ENDSTOPPULLUP_ZMIN
#endif

#ifdef ENDSTOPPULLUPS
#define ENDSTOPPULLUP_XMAX
#define ENDSTOPPULLUP_YMAX
#define ENDSTOPPULLUP_ZMAX
#define ENDSTOPPULLUP_XMIN
#define ENDSTOPPULLUP_YMIN
#define ENDSTOPPULLUP_ZMIN
#endif

// The pullups are needed if you directly connect a mechanical endswitch between the signal and ground pins.
const bool X_MIN_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
const bool Y_MIN_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
const bool Z_MIN_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool X_MAX_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
const bool Y_MAX_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
const bool Z_MAX_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
//#define DISABLE_MAX_ENDSTOPS
//#define DISABLE_MIN_ENDSTOPS

// Disable max endstops for compatibility with endstop checking routine
#if defined(COREXY) && !defined(DISABLE_MAX_ENDSTOPS)
#define DISABLE_MAX_ENDSTOPS
#endif

// For Inverting Stepper Enable Pins (Active Low) use 0, Non Inverting (Active High) use 1
#define X_ENABLE_ON 0
#define Y_ENABLE_ON 0
#define Z_ENABLE_ON 0
#define E_ENABLE_ON 0 // For all extruders

// Disables axis when it's not being used.
#define DISABLE_X false
#define DISABLE_Y false
#define DISABLE_Z false
#define DISABLE_E false // For all extruders

#define INVERT_X_DIR false // for Mendel set to false, for Orca set to true
#define INVERT_Y_DIR false // for Mendel set to true, for Orca set to false
#define INVERT_Z_DIR false // for Mendel set to false, for Orca set to true
#define INVERT_E0_DIR false // for direct drive extruder v9 set to true, for geared extruder set to false
#define INVERT_E1_DIR false // for direct drive extruder v9 set to true, for geared extruder set to false
#define INVERT_E2_DIR false // for direct drive extruder v9 set to true, for geared extruder set to false

// ENDSTOP SETTINGS:
// Sets direction of endstops when homing; 1=MAX, -1=MIN
#define X_HOME_DIR 1
#define Y_HOME_DIR 1
#define Z_HOME_DIR 1

#define min_software_endstops true // If true, axis won't move to coordinates less than HOME_POS.
#define max_software_endstops true // If true, axis won't move to coordinates greater than the defined lengths below.

// Travel limits after homing
#define X_MAX_POS DELTA_PRINTABLE_RADIUS
#define X_MIN_POS -DELTA_PRINTABLE_RADIUS
#define Y_MAX_POS DELTA_PRINTABLE_RADIUS
#define Y_MIN_POS -DELTA_PRINTABLE_RADIUS
#define Z_MAX_POS MANUAL_Z_HOME_POS
#define Z_MIN_POS 0

#define X_MAX_LENGTH (X_MAX_POS - X_MIN_POS)
#define Y_MAX_LENGTH (Y_MAX_POS - Y_MIN_POS)
#define Z_MAX_LENGTH (Z_MAX_POS - Z_MIN_POS)


Please help.

Turn on the printer

hold down a endstop manually

then send a m119

Did the correct endstop change to Triggered

Check all 3 endstops manually


NB each endstops only stops the motor on that axis, so make sure you have those lined up, correct endstop for each motor

Here is my new code updated.

// The pullups are needed if you directly connect a mechanical endswitch between the signal and ground pins.
const bool X_MIN_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool Y_MIN_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool Z_MIN_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
const bool X_MAX_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool Y_MAX_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool Z_MAX_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
//#define DISABLE_MAX_ENDSTOPS
//#define DISABLE_MIN_ENDSTOPS




When I press home, each tower moves up like eaxctly one unit. So the good thing at this point is that is isnt going all the way up and crashing and it is going in the right direction. The bad thing still is that when i do get to the top, the endstops light comes on but does not make the motor stop. The motor acts like its not there.


Here is the m119 when it is open.
Reporting endstop status
x_max: TRIGGERED
y_max: TRIGGERED
z_min: open
z_max: TRIGGERED





Here is the m119 for the x end stop pushed.
Reporting endstop status
x_max: TRIGGERED
y_max: TRIGGERED
z_min: open
z_max: TRIGGERED





here is the m119 for the y end stop pushed
x_max: TRIGGERED
y_max: TRIGGERED
z_min: TRIGGERED
z_max: TRIGGERED



here is the m119 for the z end stop pushed.
x_max: TRIGGERED
y_max: TRIGGERED
z_min: open
z_max: TRIGGERED



Side note. A few weeks back, i accidently blew the voltage regulator and bought a whole new arduino and ramps1.4. However, I am still using the same endstops because I felt like they wouldn't be damaged by it.

From your results
           Xmax      Ymax      Zmin      Zmax
open     TRIGGERED TRIGGERED   open    TRIGGERED
x        TRIGGERED TRIGGERED   open    TRIGGERED
y        TRIGGERED TRIGGERED TRIGGERED TRIGGERED
z        TRIGGERED TRIGGERED   open    TRIGGERED

So according to this

X does nothing
Y triggers the Zmin!
Z does nothing

Are you sure you have plugged the end stops into the MAX positions on the ramps?

ie ramps has enstops pins for min and max and you must use the max endstops on a delta


Edited 4 time(s). Last edit at 05/21/2016 11:57PM by Dust.

Okay, I found out after reading your message that my endstops were all plugged into the MIN. of every axis. After finding that out, I plugged all the endstops into the correct MAX. positions and this is what i got.



Here is the m119 when it is open.
Reporting endstop status
x_max: TRIGGERED
y_max: TRIGGERED
z_min: open
z_max: TRIGGERED





Here is the m119 for the x end stop pushed.
Reporting endstop status
x_max: TRIGGERED
y_max: open
z_min: open
z_max: TRIGGERED





here is the m119 for the y end stop pushed
x_max: TRIGGERED
y_max: TRIGGERED
z_min: open
z_max: open



here is the m119 for the z end stop pushed.
x_max: open
y_max: TRIGGERED
z_min: open
z_max: TRIGGERED


Here is my code for that as well that is currently uploaded.
//=============================Mechanical Settings===========================
//===========================================================================

// Uncomment the following line to enable CoreXY kinematics
// #define COREXY

// coarse Endstop Settings
#define ENDSTOPPULLUPS // Comment this out (using // at the start of the line) to disable the endstop pullup resistors

#ifndef ENDSTOPPULLUPS
// fine endstop settings: Individual pullups. will be ignored if ENDSTOPPULLUPS is defined
// #define ENDSTOPPULLUP_XMAX
// #define ENDSTOPPULLUP_YMAX
// #define ENDSTOPPULLUP_ZMAX
// #define ENDSTOPPULLUP_XMIN
// #define ENDSTOPPULLUP_YMIN
// #define ENDSTOPPULLUP_ZMIN
#endif

#ifdef ENDSTOPPULLUPS
#define ENDSTOPPULLUP_XMAX
#define ENDSTOPPULLUP_YMAX
#define ENDSTOPPULLUP_ZMAX
#define ENDSTOPPULLUP_XMIN
#define ENDSTOPPULLUP_YMIN
#define ENDSTOPPULLUP_ZMIN
#endif

// The pullups are needed if you directly connect a mechanical endswitch between the signal and ground pins.
const bool X_MIN_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool Y_MIN_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool Z_MIN_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
const bool X_MAX_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool Y_MAX_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool Z_MAX_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
//#define DISABLE_MAX_ENDSTOPS
//#define DISABLE_MIN_ENDSTOPS

// Disable max endstops for compatibility with endstop checking routine
#if defined(COREXY) && !defined(DISABLE_MAX_ENDSTOPS)
#define DISABLE_MAX_ENDSTOPS
#endif

// For Inverting Stepper Enable Pins (Active Low) use 0, Non Inverting (Active High) use 1
#define X_ENABLE_ON 0
#define Y_ENABLE_ON 0
#define Z_ENABLE_ON 0
#define E_ENABLE_ON 0 // For all extruders

// Disables axis when it's not being used.
#define DISABLE_X false
#define DISABLE_Y false
#define DISABLE_Z false
#define DISABLE_E false // For all extruders

#define INVERT_X_DIR true // for Mendel set to false, for Orca set to true
#define INVERT_Y_DIR true // for Mendel set to true, for Orca set to false
#define INVERT_Z_DIR true // for Mendel set to false, for Orca set to true
#define INVERT_E0_DIR false // for direct drive extruder v9 set to true, for geared extruder set to false
#define INVERT_E1_DIR false // for direct drive extruder v9 set to true, for geared extruder set to false
#define INVERT_E2_DIR false // for direct drive extruder v9 set to true, for geared extruder set to false

// ENDSTOP SETTINGS:
// Sets direction of endstops when homing; 1=MAX, -1=MIN
#define X_HOME_DIR 1
#define Y_HOME_DIR 1
#define Z_HOME_DIR 1

#define min_software_endstops true // If true, axis won't move to coordinates less than HOME_POS.
#define max_software_endstops true // If true, axis won't move to coordinates greater than the defined lengths below.

// Travel limits after homing
#define X_MAX_POS DELTA_PRINTABLE_RADIUS
#define X_MIN_POS -DELTA_PRINTABLE_RADIUS
#define Y_MAX_POS DELTA_PRINTABLE_RADIUS
#define Y_MIN_POS -DELTA_PRINTABLE_RADIUS
#define Z_MAX_POS MANUAL_Z_HOME_POS
#define Z_MIN_POS 0

#define X_MAX_LENGTH (X_MAX_POS - X_MIN_POS)
#define Y_MAX_LENGTH (Y_MAX_POS - Y_MIN_POS)
#define Z_MAX_LENGTH (Z_MAX_POS - Z_MIN_POS)
//============================= Bed Auto Leveling ===========================

... now you can either switch the plugs according to your measurement (x->y, y->z, z->y) ... or the pins in the file (should be "pins.h") ... and invert all the endstops -- then it should be OK ...

---

Viktor
--------
Aufruf zum Projekt "Müll-freie Meere" - [reprap.org] -- Deutsche Facebook-Gruppe - [www.facebook.com]

Call for the project "garbage-free seas" - [reprap.org]

as VDX said..

you need to invert all max endstops

so find

const bool X_MAX_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool Y_MAX_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool Z_MAX_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.

change to

const bool X_MAX_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
const bool Y_MAX_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
const bool Z_MAX_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.

           Xmax      Ymax      Zmin      Zmax
open     TRIGGERED TRIGGERED   open    TRIGGERED
x        TRIGGERED   open      open    TRIGGERED
y        TRIGGERED TRIGGERED   open      open
z          open    TRIGGERED   open    TRIGGERED

And you have all the endstops plugged into the wrong ports

You have X plugged into Y, Y plugged into Z and Z plugged into X

you should end up with

           Xmax      Ymax      Zmin      Zmax
open       open      open      open      open
x        TRIGGERED   open      open      open
y          open    TRIGGERED   open      open
z          open      open      open    TRIGGERED

Edited 2 time(s). Last edit at 05/22/2016 06:06PM by Dust.

Awesome. I will try this out and I will post how it goes. Thank you again for both of your continued help and support!

I fixed the endstop issues. I typed m119 and this is what i got.
Here is the m119 when it is open.

Reporting endstop status
x_max: open
y_max: open
z_min: open
z_max: open




Here is the m119 for the x end stop pushed.
Reporting endstop status
x_max: TRIGGERED
y_max: open
z_min: open
z_max: open





here is the m119 for the y end stop pushed
x_max: open
y_max: TRIGGERED
z_min: open
z_max: open



here is the m119 for the z end stop pushed.
x_max: open
y_max: open
z_min: open
z_max: TRIGGERED


The problem I am having now is that when I push home, nothing happens. Nothing moves and the motors don't even respond...? I am lost!
Below is my code.


// coarse Endstop Settings
#define ENDSTOPPULLUPS // Comment this out (using // at the start of the line) to disable the endstop pullup resistors

#ifndef ENDSTOPPULLUPS
// fine endstop settings: Individual pullups. will be ignored if ENDSTOPPULLUPS is defined
// #define ENDSTOPPULLUP_XMAX
// #define ENDSTOPPULLUP_YMAX
// #define ENDSTOPPULLUP_ZMAX
// #define ENDSTOPPULLUP_XMIN
// #define ENDSTOPPULLUP_YMIN
// #define ENDSTOPPULLUP_ZMIN
#endif

#ifdef ENDSTOPPULLUPS
#define ENDSTOPPULLUP_XMAX
#define ENDSTOPPULLUP_YMAX
#define ENDSTOPPULLUP_ZMAX
#define ENDSTOPPULLUP_XMIN
#define ENDSTOPPULLUP_YMIN
#define ENDSTOPPULLUP_ZMIN
#endif

// The pullups are needed if you directly connect a mechanical endswitch between the signal and ground pins.
const bool X_MIN_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool Y_MIN_ENDSTOP_INVERTING = false; // set to true to invert the logic of the endstop.
const bool Z_MIN_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
const bool X_MAX_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
const bool Y_MAX_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
const bool Z_MAX_ENDSTOP_INVERTING = true; // set to true to invert the logic of the endstop.
//#define DISABLE_MAX_ENDSTOPS
//#define DISABLE_MIN_ENDSTOPS

// Disable max endstops for compatibility with endstop checking routine
#if defined(COREXY) && !defined(DISABLE_MAX_ENDSTOPS)
#define DISABLE_MAX_ENDSTOPS
#endif

// For Inverting Stepper Enable Pins (Active Low) use 0, Non Inverting (Active High) use 1
#define X_ENABLE_ON 1
#define Y_ENABLE_ON 1
#define Z_ENABLE_ON 1
#define E_ENABLE_ON 0 // For all extruders

// Disables axis when it's not being used.
#define DISABLE_X false
#define DISABLE_Y false
#define DISABLE_Z false
#define DISABLE_E false // For all extruders

#define INVERT_X_DIR true // for Mendel set to false, for Orca set to true
#define INVERT_Y_DIR true // for Mendel set to true, for Orca set to false
#define INVERT_Z_DIR true // for Mendel set to false, for Orca set to true
#define INVERT_E0_DIR false // for direct drive extruder v9 set to true, for geared extruder set to false
#define INVERT_E1_DIR false // for direct drive extruder v9 set to true, for geared extruder set to false
#define INVERT_E2_DIR false // for direct drive extruder v9 set to true, for geared extruder set to false

// ENDSTOP SETTINGS:
// Sets direction of endstops when homing; 1=MAX, -1=MIN
#define X_HOME_DIR 1
#define Y_HOME_DIR 1
#define Z_HOME_DIR 1

#define min_software_endstops true // If true, axis won't move to coordinates less than HOME_POS.
#define max_software_endstops true // If true, axis won't move to coordinates greater than the defined lengths below.

// Travel limits after homing
#define X_MAX_POS DELTA_PRINTABLE_RADIUS
#define X_MIN_POS -DELTA_PRINTABLE_RADIUS
#define Y_MAX_POS DELTA_PRINTABLE_RADIUS
#define Y_MIN_POS -DELTA_PRINTABLE_RADIUS
#define Z_MAX_POS MANUAL_Z_HOME_POS
#define Z_MIN_POS 0

#define X_MAX_LENGTH (X_MAX_POS - X_MIN_POS)
#define Y_MAX_LENGTH (Y_MAX_POS - Y_MIN_POS)
#define Z_MAX_LENGTH (Z_MAX_POS - Z_MIN_POS)

Endstops look good now.

Did you turn on the external power supply?

The power supply is on and nothing is moving? Hmm?

Do you want me to post my entire code for configuration.h?

Its unlikely to be configuration, unless you have changed the motherboard line to be a controller you don't have

Also pictures of your controller may be useful

and output of m503 command, ie Print settings

Do you have a multi meter?

I have a multi meter and tested the pos and neg temrinals and 12v is being produced, so that should be good. I did the m503 command and below is what i got. I do not know what it means though.


SENDING:M503
echoteps per unit:
echo: M92 X79.00 Y79.00 Z79.00 E94.00
echo:Maximum feedrates (mm/s):
echo: M203 X200.00 Y200.00 Z200.00 E200.00
echo:Maximum Acceleration (mm/s2):
echo: M201 X9000 Y9000 Z9000 E9000
echo:Acceleration: S=acceleration, T=retract acceleration
echo: M204 S3000.00 T3000.00
echo:Advanced variables: S=Min feedrate (mm/s), T=Min travel feedrate (mm/s), B=minimum segment time (ms), X=maximum XY jerk (mm/s), Z=maximum Z jerk (mm/s), E=maximum E jerk (mm/s)
echo: M205 S0.00 T0.00 B20000 X20.00 Z20.00 E20.00
echo:Home offset (mm):
echo: M206 X0.00 Y0.00 Z0.00
echo:Endstop adjustement (mm):
echo: M666 X0.00 Y0.00 Z0.00
echoID settings:
echo: M301 P22.20 I1.08 D114.00

I wanted to see
" M92 X79.00 Y79.00 Z79.00 E94.00"

This is the steps per mm, if that had gotten set to 0, it wouldn't move.

Can you check on the driver modules that they are getting 12v

ie voltage between vmot and the gnd under it. (be careful not to short it)



Edited 1 time(s). Last edit at 05/24/2016 08:44PM by Dust.

They are all getting 12v. It is weird. A few days ago when i had the end stops all screwed up, the motors would turn. Now they don't move at all? It is strange.

Another thing I noticed is before, when the motors worked I could not move the pulleys because the motors were on and could not be moved. Now when i turn on the power supply, i can move the pulleys and the motors are not even active.

That means the enable lines on the drivers are not being enabled...

Make sure you ramps is fully seated on the mega and check you configuration.h has

#define MOTHERBOARD BOARD_RAMPS_14_EFB

or something with RAMPS in the name

Older firmware could still use a number, 33 or 34

This is what is in my Marlin. So i would switch my #dwfinne MOTHERBOARD 33----------> to #define MOTHERBOARD BOARD_RAMPS_14_EFB ?



// 33 = RAMPS 1.3 / 1.4 (Power outputs: Extruder, Fan, Bed)


#ifndef MOTHERBOARD
#define MOTHERBOARD 33
#endif

Thats an older version, 33 is fine.

Okay. Do you have any other ideas then as to why the motors are not activating? I made sure my ramps was fully pushed onto my arduino.

Also, What do you mean or how do you fix it when you said ------->That means the enable lines on the drivers are not being enabled...

Well that is the issue isnt it

The stepper modules don't seem to be being enabled, so the steppers never get energized so will never move.

The cause... is what we are looking for.

The thing is the enable pins come directly from the mega. So its either a firmware issue where by the incorrect pins are being used (controlled by what motherboard you select) or customization of the pins.h file.. or something else...

Or the mega has dead pins.
Or their is a hardware fault in between the mega and the ramps.
Or dead stepper drivers


I would take a step back, grab the ramps test firmware [reprap.org]
and install that. if the motors move its a firmware issue, if they don't move its a hardware issue

Please note the test firmware will switch on and off your heated-bed and extruder hot-end too besides moving all the motors and flashing the leds. Disconnect them if you do not want any heating while testing.

NB this test doesn't look at endstops so could bash a axis at the end of travel.

Edited 4 time(s). Last edit at 05/25/2016 09:39PM by Dust.

I might have found somthing. When plug it in, the motors tighten up like they are ready to be given commands and i try to move them manually and they wont budge which is a good thing. Then, I go and press home command and the become loose and can be moved. Then, the home screen says Err. MINTEMP

Okay I have good news! I have it homing and is working great! Motors are running and endstops are performing! Now the thing that is not right is when i home it, it functions but a message comes up that says ERR. MINTEMP

A thermistor is either not connected or has a broken connection.

That was it. For now that is it! Thank you so much for your help. I truly appreciate it.