hardware/config.scad

@@ -157,6 +157,21 @@ y_carriage_post_hole_off = 2;
 y_carriage_post_center = 1;
 y_carriage_post_screw_hole = y_carriage_post_screw + nut_gap;
 y_carriage_post_len = y_carriage_rail_dist - rail_wheel_height / 2 + 10;
+echo("y_carriage_post_len", y_carriage_post_len);
+
+eccentric_height_main = 8.5;
+eccentric_height_inset = 2.4;
+eccentric_dia_inset = 7.3;
+eccentric_dia_main = 11.5;
+eccentric_hole_off = 0.75;
+eccentric_screw_hole = y_carriage_post_screw_hole + 2 * eccentric_hole_off;
+eccentric_height_add = y_carriage_post_len - eccentric_height_main;
+eccentric_inset_wall = 15.0;
+eccentric_inset_hole = eccentric_height_inset + 0.6;
+eccentric_screw_hole_h = 6.5;
+eccentric_screw_hole_dia = 15.0 + 2 * eccentric_hole_off;
+
+use_custom_eccentric = true;
 
 y_carriage_wheel_off_x = 20.4;
 y_carriage_wheel_off_y = 28;
@@ -211,7 +226,7 @@ x_axis_rail_len = x_axis_travel_len + y_carriage_y + motor_mount_length + belt_t
 point_that_reaches_everywhere_x = -7.5;
 point_that_reaches_everywhere_y = -17.5;
 
-use_anim = true;
+use_anim = false;
 x_axis_position = 1.0;
 y_axis_position = 0.5;
 
@@ -236,3 +251,24 @@ right_support_len = 200;
 right_support_off = 50;
 
 echo("distance between bottom support rails", x_axis_rail_len - left_support_off - right_support_off - 40, -left_support_off, -right_support_off);
+
+endstop_pcb_width = 16.0;
+endstop_pcb_height = 40.0;
+endstop_pcb_depth = 1.6;
+endstop_pcb_hole = 3.2;
+endstop_pcb_hole_dist_x = 3.2;
+endstop_pcb_hole_dist_y = 3.2;
+endstop_pcb_hole_dist = 19.0;
+endstop_pcb_switchpoint = 4.5;
+endstop_mount_width = 12;
+endstop_mount_depth = 5;
+endstop_mount_slot_width = 6;
+endstop_mount_slot_depth = 4;
+endstop_mount_slot_length = 8;
+endstop_mount_add_height = 15;
+endstop_mount_nut_hole = 6.5;
+endstop_pcb_mount_off_h = y_carriage_rail_dist + endstop_pcb_switchpoint + 2; // TODO is now -3
+endstop_pcb_mount_off_d = 18; // TODO was 8
+endstop_pcb_mount_off_w = endstop_mount_slot_length;
+
+use_endstop_mount_v2 = false;

hardware/fuellfix.scad

@@ -3,7 +3,7 @@ use <table.scad>;
 use <gearbox.scad>;
 use <common.scad>;
 
-draw_assembly = false;
+draw_assembly = true;
 
 module print() {
     for (i = [1 : 4])
@@ -43,6 +43,11 @@ module print() {
     translate([-real_belt_pulley_dia - i, belt_tensioner_moving_len + 20, 0])
     rotate([0, -90, 0])
     motor_mount(2);
+    
+    for (i = [0, -100])
+    translate([-real_belt_pulley_dia - 140, belt_tensioner_moving_len + i, endstop_mount_depth])
+    rotate([-90, 0, 0])
+    endstop_mount();
 }
 
 top_rail_len = x_axis_rail_len + plate_x - 70;
@@ -70,8 +75,11 @@ if (draw_assembly) {
     fuellfix_assembly();
 } else {
     //belt_pulley(teethcount, real_belt_pulley_dia, 0);
-    x_carriage_holder();
-    //print();
+    //x_carriage_holder();
+    //y_carriage(0);
+    //endstop_mount();
+    
+    print();
 }
     
 echo(); echo(); echo();

hardware/table.scad

@@ -254,6 +254,121 @@ module plate() {
  ************** Y-Axis **************
  ************************************/
 
+module endstop_pcb() {
+    difference() {
+        cube([endstop_pcb_width, endstop_pcb_depth, endstop_pcb_height]);
+        
+        for (i = [0, 1])
+        translate([endstop_pcb_hole_dist_x, -1, endstop_pcb_height - endstop_pcb_hole_dist_y - i * endstop_pcb_hole_dist])
+        rotate([-90, 0, 0])
+        cylinder(d = endstop_pcb_hole, h = endstop_pcb_depth + 2);
+    }
+    
+    translate([0, 0, endstop_pcb_height - endstop_pcb_switchpoint])
+    rotate([0, -90, 0])
+    cylinder(d = endstop_pcb_hole, h = endstop_pcb_switchpoint);
+}
+
+module endstop_mount_v1() {
+    %color("yellow")
+    translate([-endstop_pcb_mount_off_w, -endstop_pcb_depth - endstop_pcb_mount_off_d, endstop_pcb_mount_off_h])
+    endstop_pcb();
+    
+    color("blue")
+    difference() {
+        cube([endstop_mount_width, endstop_mount_depth, 40]);
+        
+        for (i = [0, 1])
+        translate([endstop_mount_width / 2, -1, 10 + 20 * i])
+        rotate([-90, 0, 0])
+        cylinder(d = motor_mount_hole_size, h = endstop_mount_depth + 2);
+    }
+    
+    color("blue")
+    difference() {
+        union() {
+            hull() {
+                translate([-endstop_mount_slot_length, 0, 40 - endstop_mount_add_height])
+                cube([endstop_mount_slot_length, endstop_mount_depth, endstop_mount_add_height]);
+                
+                translate([0, 0, endstop_pcb_height - endstop_pcb_hole_dist_y - endstop_pcb_hole_dist])
+                translate([-endstop_pcb_mount_off_w, -endstop_pcb_mount_off_d, endstop_pcb_mount_off_h - endstop_mount_slot_width / 2])
+                cube([endstop_mount_slot_length, endstop_mount_slot_depth, endstop_mount_slot_width]);
+                
+                for (i = [0, 1])
+                translate([0, 0, endstop_pcb_height - endstop_pcb_hole_dist_y - i * endstop_pcb_hole_dist])
+                translate([-endstop_pcb_mount_off_w, -endstop_pcb_mount_off_d, endstop_pcb_mount_off_h - endstop_mount_slot_width / 2])
+                cube([endstop_mount_slot_length, endstop_mount_slot_depth, endstop_mount_slot_width]);
+            }
+        }
+        
+        translate([-endstop_pcb_mount_off_w, -endstop_pcb_depth - endstop_pcb_mount_off_d, endstop_pcb_mount_off_h])
+        for (i = [0, 1])
+        translate([endstop_pcb_hole_dist_x, -1, endstop_pcb_height - endstop_pcb_hole_dist_y - i * endstop_pcb_hole_dist])
+        rotate([-90, 0, 0]) {
+            cylinder(d = endstop_pcb_hole, h = endstop_pcb_depth + 20);
+            
+            translate([0, 0, 1 + endstop_pcb_depth + endstop_mount_slot_depth])
+            cylinder(d = endstop_mount_nut_hole, h = 20, $fn = 6);
+        }
+    }
+}
+
+module endstop_mount_v2() {
+    %color("yellow")
+    translate([-endstop_pcb_mount_off_w, -endstop_pcb_depth - endstop_pcb_mount_off_d, endstop_pcb_mount_off_h])
+    endstop_pcb();
+    
+    color("blue")
+    difference() {
+        cube([endstop_mount_width, endstop_mount_depth, 40]);
+        
+        for (i = [0, 1])
+        translate([endstop_mount_width / 2, -1, 10 + 20 * i])
+        rotate([-90, 0, 0])
+        cylinder(d = motor_mount_hole_size, h = endstop_mount_depth + 2);
+    }
+    
+    color("blue")
+    difference() {
+        union() {
+            hull() {
+                translate([-endstop_mount_slot_length, 0, 40 - endstop_mount_add_height])
+                cube([endstop_mount_slot_length, endstop_mount_depth, endstop_mount_add_height]);
+                
+                translate([0, 0, endstop_pcb_height - endstop_pcb_hole_dist_y - endstop_pcb_hole_dist])
+                translate([-endstop_pcb_mount_off_w, -endstop_pcb_mount_off_d, endstop_pcb_mount_off_h - endstop_mount_slot_width / 2])
+                cube([endstop_mount_slot_length, endstop_mount_slot_depth, endstop_mount_slot_width]);
+            }
+            
+            hull() {
+                for (i = [0, 1])
+                translate([0, 0, endstop_pcb_height - endstop_pcb_hole_dist_y - i * endstop_pcb_hole_dist])
+                translate([-endstop_pcb_mount_off_w, -endstop_pcb_mount_off_d, endstop_pcb_mount_off_h - endstop_mount_slot_width / 2])
+                cube([endstop_mount_slot_length, endstop_mount_slot_depth, endstop_mount_slot_width]);
+            }
+        }
+        
+        translate([-endstop_pcb_mount_off_w, -endstop_pcb_depth - endstop_pcb_mount_off_d, endstop_pcb_mount_off_h])
+        for (i = [0, 1])
+        translate([endstop_pcb_hole_dist_x, -1, endstop_pcb_height - endstop_pcb_hole_dist_y - i * endstop_pcb_hole_dist])
+        rotate([-90, 0, 0]) {
+            cylinder(d = endstop_pcb_hole, h = endstop_pcb_depth + 20);
+            
+            translate([0, 0, 1 + endstop_pcb_depth + endstop_mount_slot_depth])
+            cylinder(d = endstop_mount_nut_hole, h = 20, $fn = 6);
+        }
+    }
+}
+
+module endstop_mount() {
+    if (use_endstop_mount_v2) {
+        endstop_mount_v2();
+    } else {
+        endstop_mount_v1();
+    }
+}
+
 module belt_mount(h) {
     difference() {
         hull() {
@@ -273,6 +388,15 @@ module belt_mount(h) {
     }
 }
 
+module eccentric() {
+    translate([0, 0, eccentric_height_add]) {
+        cylinder(d = eccentric_dia_main, h = eccentric_height_main);
+        
+        translate([0, 0, -eccentric_height_inset])
+        cylinder(d = eccentric_dia_inset, h = eccentric_height_inset);
+    }
+}
+
 module y_carriage_post() {
     rotate([0, 0, 90])
     difference() {
@@ -290,12 +414,21 @@ module y_carriage_posts() {
     for (i = [-1, 1])
     for (j = [-1, 1])
     translate([i * y_carriage_wheel_x_dist / 2, j * y_carriage_wheel_y_dist / 2, 0]) {
-        color("blue")
-        y_carriage_post();
+        if (use_custom_eccentric) {
+            color("yellow")
+            eccentric();
         
-        %color("yellow")
-        translate([0, y_carriage_post_hole_off, y_carriage_pulley_off])
-        rail_wheel();
+            %color("yellow")
+            translate([0, eccentric_hole_off, y_carriage_pulley_off])
+            rail_wheel();
+        } else {
+            color("blue")
+            y_carriage_post();
+        
+            %color("yellow")
+            translate([0, y_carriage_post_hole_off, y_carriage_pulley_off])
+            rail_wheel();
+        }
     }
 }
 
@@ -307,6 +440,12 @@ module y_carriage(axis = 0) {
     if (axis == 1) {
         echo("t-nut", "x-carriage", "2");
     }
+    
+    if (use_custom_eccentric) {
+        echo("eccentric", "carriage", "4");
+        echo("screw", "carriage", "M5x30", "4");
+        echo("nut", "carriage", "M5", "4");
+    }
 
     color("green")
     difference() {
@@ -318,6 +457,14 @@ module y_carriage(axis = 0) {
             translate([0, i * (y_carriage_y + belt_mount_depth) / 2, 0])
             scale([1, -i, 1])
             belt_mount(y_carriage_h);
+            
+            // additional "meat" for eccentric inset
+            translate([y_carriage_x / 2, y_carriage_y / 2, y_carriage_h])
+            for (i = [-1, 1])
+            for (j = [-1, 1])
+            translate([i * y_carriage_wheel_x_dist / 2, j * y_carriage_wheel_y_dist / 2, 0])
+            if (use_custom_eccentric)
+            cylinder(d = eccentric_inset_wall, h = eccentric_height_add);
         }
         
         translate([y_carriage_x / 2, y_carriage_y / 2, 0])
@@ -335,10 +482,19 @@ module y_carriage(axis = 0) {
             cylinder(d = y_carriage_post_screw_hole, h = y_carriage_h + 2);
         }
         
+        // holes for wheel posts
         translate([y_carriage_x / 2, y_carriage_y / 2, 0])
         for (i = [-1, 1])
         for (j = [-1, 1])
-        translate([i * y_carriage_wheel_x_dist / 2, j * y_carriage_wheel_y_dist / 2, -1]) {
+        translate([i * y_carriage_wheel_x_dist / 2, j * y_carriage_wheel_y_dist / 2, -1])
+        if (use_custom_eccentric) {
+            cylinder(d = eccentric_screw_hole, h = y_carriage_h + 2);
+            
+            translate([0, 0, y_carriage_h + eccentric_height_add + 1 - eccentric_inset_hole])
+            cylinder(d = eccentric_dia_inset, h = eccentric_inset_hole + 1);
+            
+            cylinder(d = eccentric_screw_hole_dia, h = eccentric_screw_hole_h + 1);
+        } else {
             cylinder(d = y_carriage_post_screw_hole, h = y_carriage_h + 2);
         }
     }
@@ -360,6 +516,10 @@ module y_axis() {
     rotate([180, 0, 0])
     motor_mount(2);
     
+    translate([10 + endstop_mount_depth, y_axis_rail_len - endstop_mount_width, -20])
+    rotate([0, 0, 90])
+    endstop_mount();
+    
     belt_tensioner(2, y_axis_rail_len, 0);
 }
 
@@ -396,6 +556,7 @@ module x_carriage() {
     rotate([0, 180, -90])
     y_carriage(1);
     
+    /*
     color("purple")
     for (i = [-1, 1])
     translate([y_carriage_y / 2, 0, 0])
@@ -403,6 +564,7 @@ module x_carriage() {
     translate([y_carriage_y / 2 - x_carriage_holder_l, 0, 0])
     translate([0, -x_carriage_holder_w / 2, 20 + y_carriage_h + y_carriage_rail_dist])
     x_carriage_holder();
+    */
 }
 
 module x_axis() {
@@ -417,6 +579,9 @@ module x_axis() {
     rotate([0, 180, 90])
     motor_mount(2);
     
+    translate([x_axis_rail_len - endstop_mount_width, -10 - endstop_mount_depth, -20])
+    endstop_mount();
+    
     rotate([0, 0, -90])
     belt_tensioner(2, x_axis_rail_len, 0);
 }
@@ -441,21 +606,43 @@ module assembly_x_axis() {
     assembly_y_axis_plate();
 }
 
+bottom_support_len = 500;
+bottom_support_off = 80;
+
+use_double_supports = false;
+
+module rail_foot() {
+    
+}
+
 module assembly() {
     translate([-x_axis_rail_len / 2, 0, 20]) {
         assembly_x_axis();
         
-        color("grey")
-        for (i = [1, -1])
-        scale([1, i, 1])
-        translate([left_support_off, -left_support_len - 10, -20])
-        rail_2020_y(left_support_len, "left support");
-        
-        color("grey")
-        for (i = [1, -1])
-        scale([1, i, 1])
-        translate([x_axis_rail_len - 20 - right_support_off, -right_support_len - 10, -20])
-        rail_2020_y(right_support_len, "right support");
+        if (use_double_supports) {
+            color("grey")
+            for (i = [1, -1])
+            scale([1, i, 1])
+            translate([left_support_off, -left_support_len - 10, -20])
+            rail_2020_y(left_support_len, "left support");
+            
+            color("grey")
+            for (i = [1, -1])
+            scale([1, i, 1])
+            translate([x_axis_rail_len - 20 - right_support_off, -right_support_len - 10, -20])
+            rail_2020_y(right_support_len, "right support");
+            
+            // TODO feet
+        } else {
+            color("grey")
+            translate([x_axis_rail_len / 2, 0, -40])
+            for (i = [1, -1])
+            scale([i, 1, 1])
+            translate([x_axis_rail_len / 2 - bottom_support_off, -bottom_support_len / 2, 0])
+            rail_2020_y(bottom_support_len, "bottom supports");
+            
+            // TODO feet
+        }
     }
 }
 
@@ -485,8 +672,8 @@ module xy_table() {
 //y_carriage_post();
 //x_carriage_holder();
 
-translate([0, -50, 0])
-y_carriage(0);
+//translate([0, -50, 0])
+//y_carriage(0);
 //y_carriage(1);
 //y_carriage(2);
 //x_carriage();
@@ -495,4 +682,4 @@ y_carriage(0);
 //x_axis();
 
 //assembly();
-//xy_table();
+xy_table();

include/common.h

@@ -1,6 +1,9 @@
 #ifndef _COMMON_H_
 #define _COMMON_H_
 
+#define CLEAR_STORE_INTERRUPTS() uint8_t prev_int_reg = SREG; cli()
+#define RESTORE_INTERRUPTS() SREG = prev_int_reg
+
 void async_beep(int time, int freq);
 void blocking_beep(int time, int freq, int repeat = 0);
 void common_run(unsigned long t);

include/config.h

@@ -72,12 +72,12 @@
 #define E_MAX_SPEED 50.0 // in percent/s
 
 // homing speeds
-#define XY_FAST_HOME_SPEED 2.5 // in mm/s
-#define XY_SLOW_HOME_SPEED 1.0 // in mm/s
-#define Z_FAST_HOME_SPEED 2.0 // in mm/s
+#define XY_FAST_HOME_SPEED 25.0 // in mm/s
+#define XY_SLOW_HOME_SPEED 5.0 // in mm/s
+#define Z_FAST_HOME_SPEED 5.0 // in mm/s
 #define Z_SLOW_HOME_SPEED 1.0 // in mm/s
 #define E_FAST_HOME_SPEED 10.0 // in percent/s
-#define E_SLOW_HOME_SPEED 5.0 // in percent/s
+#define E_SLOW_HOME_SPEED 2.0 // in percent/s
 
 // accelerations
 #define XY_MAX_ACCEL 100.0 // in mm/s^2
@@ -91,12 +91,20 @@
 #define E_AXIS_MOVEMENT_DIR 1.0
 
 // homing back-off
-#define XY_BACK_OFF_DISTANCE 10.0 // in mm
-#define Z_BACK_OFF_DISTANCE 5.0 // in mm
-#define E_BACK_OFF_DISTANCE 2.0 // in mm
+#define XY_BACK_OFF_DISTANCE 5.0 // in mm
+#define Z_BACK_OFF_DISTANCE 2.0 // in mm
+#define E_BACK_OFF_DISTANCE 1.0 // in mm
 
 #define XY_HOME_BACK_OFF_TIME (XY_BACK_OFF_DISTANCE / XY_FAST_HOME_SPEED * 1000)
 #define Z_HOME_BACK_OFF_TIME (Z_BACK_OFF_DISTANCE / Z_FAST_HOME_SPEED * 1000)
 #define E_HOME_BACK_OFF_TIME (E_BACK_OFF_DISTANCE / E_FAST_HOME_SPEED * 1000)
 
+// epsilon around expected endstop position where we dont abort on hits
+#define X_AXIS_EPSILON 20.0 // TODO
+#define Y_AXIS_EPSILON 20.0 // TODO
+#define Z_AXIS_EPSILON 1.0
+#define E_AXIS_EPSILON 1.0
+
+#define ENCODER_RPM_VALUE_FACTOR 10.0
+
 #endif // _CONFIG_H_

include/config_pins.h

@@ -41,12 +41,12 @@
 
 #define LED_PIN            13
 
-#define FAN_PIN            9
+#define FAN_PIN            8
 
 #define PS_ON_PIN          12	//ATX , awake=LOW, SLEEP=High
 
 #define HEATER_0_PIN	10  // Extruder Heater
-#define HEATER_1_PIN	8
+#define HEATER_1_PIN	9
 
 #define TEMP_0_PIN		13   // ANALOG NUMBERING
 #define TEMP_1_PIN		14   // ANALOG NUMBERING

include/encoder.h

@@ -5,6 +5,7 @@ void encoder_init(void);
 void encoder_run(void);
 
 int encoder_change(void);
+int encoder_rpm(void);
 int encoder_click(void);
 int kill_switch(void);
 

include/statemachine.h

@@ -12,11 +12,12 @@ template <typename T, size_t N>
 void array_insert_at_pos(Array<T, N> *arr, T value, size_t pos);
 
 struct StateMachineInput {
-    StateMachineInput(int c, int e, int k, int m)
-            : click(c), encoder(e), kill(k), motors_done(m) { };
+    StateMachineInput(int c, int e, int r, int k, int m)
+            : click(c), encoder(e), rpm(r), kill(k), motors_done(m) { };
 
     int click;
     int encoder;
+    int rpm;
     int kill;
     int motors_done;
 };
@@ -54,6 +55,8 @@ public:
     void onEnter(EnterFuncPtr func);
     void whenIn(InFuncPtr func);
 
+    void updateText(void);
+
     virtual void enterState(void);
     virtual void inState(StateMachineInput smi);
 
@@ -151,8 +154,10 @@ public:
     void setHeading(const char *_heading);
     void setText(const char *_text);
 
+    typedef void(*EnterFuncPtr)(void);
     typedef void(*UpdateFuncPtr)(size_t index, T value);
 
+    void onEnter(EnterFuncPtr func);
     void onLiveUpdate(UpdateFuncPtr func);
     void onUpdate(UpdateFuncPtr func);
 
@@ -168,6 +173,7 @@ private:
     const char *heading;
     size_t pos;
     bool editing;
+    EnterFuncPtr onEnterFunc;
     UpdateFuncPtr updateFunc, updateLiveFunc;
 };
 

include/states.h

@@ -9,4 +9,6 @@ void states_run(struct StateMachineInput smi);
 State *states_get(void);
 void states_go_to(State *state);
 
+extern StateText sm_error;
+
 #endif // _STATES_H_

include/steppers.h

@@ -5,8 +5,12 @@ void steppers_init(void);
 bool steppers_run(void);
 
 bool steppers_homed(void);
+char steppers_homing_axis(void);
 void steppers_start_homing(void);
 
+float steppers_current_pos(int axis);
+
+// target pos
 float steppers_pos_x(void);
 float steppers_pos_y(void);
 float steppers_pos_z(void);
@@ -29,4 +33,6 @@ void steppers_set_accel_y(float accel);
 void steppers_set_accel_z(float accel);
 void steppers_set_accel_e(float accel);
 
+bool steppers_home_switch(int axis);
+
 #endif // _STEPPERS_H_

platformio.ini

@@ -24,6 +24,7 @@ lib_deps =
     https://github.com/waspinator/AccelStepper
     https://github.com/mathertel/RotaryEncoder
     https://github.com/janelia-arduino/Array
+    https://github.com/PaulStoffregen/TimerOne
 
 [env:ramps_lcd12864]
 platform = atmelavr
@@ -38,3 +39,4 @@ lib_deps =
     https://github.com/waspinator/AccelStepper
     https://github.com/mathertel/RotaryEncoder
     https://github.com/janelia-arduino/Array
+    https://github.com/PaulStoffregen/TimerOne

src/debounce.cpp

@@ -19,6 +19,13 @@ int Debouncer::poll() {
             if (currentState == LOW) {
                 ret = 1;
             }
+
+            /*
+            Serial.print("Debounce: ");
+            Serial.print(pin);
+            Serial.print(" = ");
+            Serial.println(state);
+            */
         }
     }
 

src/encoder.cpp

@@ -63,6 +63,10 @@ int encoder_change(void) {
     return diff;
 }
 
+int encoder_rpm(void) {
+    return encoder.getRPM();
+}
+
 int encoder_click(void) {
     int r = click_state;
 

src/lcd.cpp

@@ -25,6 +25,8 @@ LiquidCrystal lcd(LCD_PINS_RS, LCD_PINS_ENABLE, LCD_PINS_D4, LCD_PINS_D5, LCD_PI
 
 #include <U8g2lib.h>
 
+//#define ENABLE_DYNAMIC_SCROLLING
+
 #define LCD_HEADING_LINES 1
 #define LCD_HEADING_WIDTH 15
 #define LCD_TEXT_LINES 5
@@ -111,6 +113,8 @@ void lcd_shorten_multiline(String s[], int n, int w, String *remainder) {
     last_scroll_time = millis();
 }
 
+#ifdef ENABLE_DYNAMIC_SCROLLING
+
 void lcd_scroll_multiline(String s[], int n, int w, String &pre, String &post, int &dir) {
     if (dir == 0) {
         // switch directions if done scrolling in current direction
@@ -180,6 +184,8 @@ void lcd_scrolling(void) {
     }
 }
 
+#endif // ENABLE_DYNAMIC_SCROLLING
+
 int lcd_text_lines(void) {
     return LCD_TEXT_LINES;
 }
@@ -221,10 +227,12 @@ void lcd_init(void) {
 }
 
 void lcd_loop(void) {
+#ifdef ENABLE_DYNAMIC_SCROLLING
     if ((millis() - last_scroll_time) >= SCROLL_DELAY) {
         last_scroll_time = millis();
         lcd_scrolling();
     }
+#endif // ENABLE_DYNAMIC_SCROLLING
 
 #ifdef USE_FULL_GRAPHIC_LCD
     if (redraw) {

src/main.cpp

@@ -1,4 +1,5 @@
 #include <Arduino.h>
+#include <TimerOne.h>
 
 #include "config.h"
 #include "config_pins.h"
@@ -9,32 +10,9 @@
 #include "steppers.h"
 #include "states.h"
 
-void setup() {
-    pinMode(LED_PIN, OUTPUT);
-    digitalWrite(LED_PIN, HIGH);
-
-    pinMode(BEEPER, OUTPUT);
-    blocking_beep(100, 1000);
-    
-    Serial.begin(115200);
-    Serial.println(F("Initializing Fuellfix v2"));
-    Serial.print(F("Version: "));
-    Serial.println(FIRMWARE_VERSION);
-
-    Serial.println(F("Init data"));
-    data_init();
-
-    Serial.println(F("Init encoder"));
-    encoder_init();
-
-    Serial.println(F("Init LCD"));
-    lcd_init();
-
-    Serial.println(F("Init stepper motors"));
-    steppers_init();
-
-    // ----------------------------------
+static volatile bool still_running = false;
 
+void print_data(void) {
     Serial.println(F("XY:"));
 
     Serial.print(F("\t X: "));
@@ -110,8 +88,42 @@ void setup() {
     Serial.println(F(" percent/s^2"));
 
     Serial.println();
+}
 
-    // ----------------------------------
+void fast_loop(void) {
+    still_running = steppers_run();
+    encoder_run();
+}
+
+void setup() {
+    pinMode(LED_PIN, OUTPUT);
+    digitalWrite(LED_PIN, HIGH);
+
+    pinMode(BEEPER, OUTPUT);
+    blocking_beep(100, 1000);
+
+    // turn on fan connected to D8 for electronics
+    pinMode(FAN_PIN, OUTPUT);
+    digitalWrite(FAN_PIN, HIGH);
+
+    Serial.begin(115200);
+    Serial.println(F("Initializing Fuellfix v2"));
+    Serial.print(F("Version: "));
+    Serial.println(FIRMWARE_VERSION);
+
+    Serial.println(F("Init data"));
+    data_init();
+
+    Serial.println(F("Init encoder"));
+    encoder_init();
+
+    Serial.println(F("Init LCD"));
+    lcd_init();
+
+    Serial.println(F("Init stepper motors"));
+    steppers_init();
+
+    print_data();
 
     Serial.println(F("ready, showing splash screen"));
     digitalWrite(LED_PIN, LOW);
@@ -125,19 +137,28 @@ void setup() {
 
     blocking_beep(100, 2000);
     Serial.println(F("starting main loop"));
+
+    Timer1.initialize(1000); // 1000us -> 1kHz
+    Timer1.attachInterrupt(fast_loop);
 }
 
 void loop() {
-    unsigned long t = millis();
-    common_run(t);
-    bool still_running = steppers_run();
-    encoder_run();
+    common_run(millis());
+
+    CLEAR_STORE_INTERRUPTS();
 
     int click = encoder_click();
+    int change = encoder_change();
+    int rpm = encoder_rpm();
     int kill = kill_switch();
+    bool motors_done = !still_running;
+
+    RESTORE_INTERRUPTS();
 
-    struct StateMachineInput smi(click, encoder_change(), kill, !still_running);
+    struct StateMachineInput smi(click, change, rpm, kill, motors_done);
     states_run(smi);
 
     lcd_loop();
+
+    delay(10);
 }

src/statemachine.cpp

@@ -45,11 +45,7 @@ void StateText::whenIn(InFuncPtr func) {
     whenInFunc = func;
 }
 
-void StateText::enterState(void) {
-    if (onEnterFunc != NULL) {
-        onEnterFunc();
-    }
-
+void StateText::updateText(void) {
     lcd_clear();
     if (heading != NULL) {
         lcd_set_heading(heading);
@@ -61,6 +57,14 @@ void StateText::enterState(void) {
     }
 }
 
+void StateText::enterState(void) {
+    if (onEnterFunc != NULL) {
+        onEnterFunc();
+    }
+
+    updateText();
+}
+
 void StateText::inState(struct StateMachineInput smi) {
     if (whenInFunc != NULL) {
         whenInFunc(smi);
@@ -325,7 +329,10 @@ void StateValue<T>::enterState(void) {
 template <typename T>
 void StateValue<T>::inState(StateMachineInput smi) {
     if (smi.encoder != 0) {
-        value -= smi.encoder;
+        float vf = smi.encoder;
+        vf *= 1.0 + ((float)smi.rpm / ENCODER_RPM_VALUE_FACTOR);
+        int v = vf;
+        value -= v;
         if (value < min) {
             value = min;
         }
@@ -353,6 +360,7 @@ template class StateValue<float>;
 template <typename T, size_t N>
 StateValues<T, N>::StateValues(State *_parent) : State(_parent) {
     heading = NULL;
+    onEnterFunc = NULL;
     updateFunc = NULL;
     updateLiveFunc = NULL;
     pos = 0;
@@ -376,6 +384,11 @@ void StateValues<T, N>::setHeading(const char *_heading) {
 }
 
 template <typename T, size_t N>
+void StateValues<T, N>::onEnter(EnterFuncPtr func) {
+    onEnterFunc = func;
+}
+
+template <typename T, size_t N>
 void StateValues<T, N>::onUpdate(UpdateFuncPtr func) {
     updateFunc = func;
 }
@@ -425,6 +438,9 @@ void StateValues<T, N>::display(void) {
 template <typename T, size_t N>
 void StateValues<T, N>::enterState(void) {
     pos = 0;
+    if (onEnterFunc != NULL) {
+        onEnterFunc();
+    }
     display();
 }
 
@@ -432,16 +448,23 @@ template <typename T, size_t N>
 void StateValues<T, N>::inState(StateMachineInput smi) {
     if (editing) {
         if (smi.encoder != 0) {
-            *(values[pos]) -= smi.encoder;
+            float vf = smi.encoder;
+            vf *= 1.0 + ((float)smi.rpm / ENCODER_RPM_VALUE_FACTOR);
+            int v = vf;
+            *(values[pos]) -= v;
+
             if (*(values[pos]) < mins[pos]) {
                 *(values[pos]) = mins[pos];
             }
+
             if (*(values[pos]) > maxs[pos]) {
                 *(values[pos]) = maxs[pos];
             }
+
             if (updateLiveFunc != NULL) {
                 updateLiveFunc(pos, *(values[pos]));
             }
+
             display();
         }
         if (smi.click) {
@@ -475,6 +498,7 @@ void StateValues<T, N>::inState(StateMachineInput smi) {
                         updateFunc(i, *(values[i]));
                     }
                 }
+
                 if (getChild() != NULL) {
                     states_go_to(getChild());
                 } else if (getParent() != NULL) {

src/states.cpp

@@ -9,6 +9,10 @@
 #include "statemachine.h"
 #include "states.h"
 
+//#define DISABLE_HOMING_STATE
+#define ENABLE_MOVEMENT_TEST_STATE
+#define ENABLE_INPUT_TEST_STATE
+
 // --------------------------------------
 
 StateText sm_ask_homing = StateText();
@@ -52,6 +56,18 @@ StateValue<float> sm_conf_accel_xy = StateValue<float>(&sm_config, data_options(
 StateValue<float> sm_conf_accel_z = StateValue<float>(&sm_config, data_options()->accel_z, 1.0, Z_MAX_ACCEL);
 StateValue<float> sm_conf_accel_e = StateValue<float>(&sm_config, data_options()->accel_e, 1.0, E_MAX_ACCEL);
 
+StateText sm_error = StateText(NULL);
+
+#ifdef ENABLE_MOVEMENT_TEST_STATE
+StateMenu sm_movement_test = StateMenu(&sm_menu);
+StateText sm_move_x_test = StateText(&sm_movement_test);
+StateText sm_move_y_test = StateText(&sm_movement_test);
+#endif // ENABLE_MOVEMENT_TEST_STATE
+
+#ifdef ENABLE_INPUT_TEST_STATE
+StateText sm_input_test = StateText(&sm_menu);
+#endif // ENABLE_INPUT_TEST_STATE
+
 // --------------------------------------
 
 State *current_state = NULL;
@@ -73,18 +89,33 @@ void states_init(void) {
     sm_do_homing.setHeading("Homing...");
 
     sm_do_homing.onEnter([]() {
-        //steppers_start_homing();
+#ifndef DISABLE_HOMING_STATE
+        steppers_start_homing();
+#endif // DISABLE_HOMING_STATE
     });
 
     sm_do_homing.whenIn([](StateMachineInput smi) {
-        //if (smi.motors_done) {
-        //    if (steppers_homed()) {
+#ifndef DISABLE_HOMING_STATE
+        if (smi.motors_done) {
+            if (steppers_homed()) {
+#endif // DISABLE_HOMING_STATE
+
                 async_beep(HOMING_BEEP_TIME, HOMING_BEEP_FREQ);
                 states_go_to(&sm_menu);
-        //    }
-        //}
 
-        // TODO update text with current axis
+#ifndef DISABLE_HOMING_STATE
+            }
+        }
+#endif // DISABLE_HOMING_STATE
+
+        static char last_axis = ' ';
+        char axis = steppers_homing_axis();
+        if ((axis != ' ') && (axis != last_axis)) {
+            last_axis = axis;
+            strbuf = String(F("Currently homing ")) + axis + String(F(" axis"));
+            sm_do_homing.setText(strbuf.c_str());
+            sm_do_homing.updateText();
+        }
     });
 
     // ----------------------------------
@@ -132,6 +163,7 @@ void states_init(void) {
 
     sm_auto.setTitle("Filling Menu");
 
+    // TODO
     sm_presets.setTitle("Use Preset");
     sm_presets.setPrefix("Preset ");
     sm_presets.dataCount([]() {
@@ -144,16 +176,27 @@ void states_init(void) {
 
     sm_wiz_move.setTitle("Initial Movement");
     sm_wiz_move.onEnter([]() {
-        //steppers_start_homing();
+        steppers_start_homing();
     });
     sm_wiz_move.whenIn([](StateMachineInput smi) {
-        //if (smi.motors_done) {
-        //    if (steppers_homed()) {
+#ifndef DISABLE_HOMING_STATE
+        if (smi.motors_done) {
+            if (steppers_homed()) {
+#endif // DISABLE_HOMING_STATE
                 states_go_to(&sm_wiz_count);
-        //    }
-        //}
-
-        // TODO update text with current axis
+#ifndef DISABLE_HOMING_STATE
+            }
+        }
+#endif // DISABLE_HOMING_STATE
+
+        static char last_axis = ' ';
+        char axis = steppers_homing_axis();
+        if ((axis != ' ') && (axis != last_axis)) {
+            last_axis = axis;
+            strbuf = String(F("Currently homing ")) + axis + String(F(" axis"));
+            sm_wiz_move.setText(strbuf.c_str());
+            sm_wiz_move.updateText();
+        }
     });
 
     sm_wiz_count.setTitle("Container Count");
@@ -163,6 +206,10 @@ void states_init(void) {
     sm_wiz_first_pos.setTitle("First Container Position");
     sm_wiz_first_pos.setData(0, "X: ", &wizard_first_x, X_AXIS_MIN, X_AXIS_MAX);
     sm_wiz_first_pos.setData(1, "Y: ", &wizard_first_y, Y_AXIS_MIN, Y_AXIS_MAX);
+    sm_wiz_first_pos.onEnter([]() {
+        wizard_first_x = steppers_current_pos(0);
+        wizard_first_y = steppers_current_pos(1);
+    });
     sm_wiz_first_pos.onLiveUpdate([](size_t i, float v) {
         if (i == 0) {
             steppers_move_x(v);
@@ -181,6 +228,10 @@ void states_init(void) {
     sm_wiz_last_pos.setTitle("Last Container Position");
     sm_wiz_last_pos.setData(0, "X: ", &wizard_last_x, X_AXIS_MIN, X_AXIS_MAX);
     sm_wiz_last_pos.setData(1, "Y: ", &wizard_last_y, Y_AXIS_MIN, Y_AXIS_MAX);
+    sm_wiz_last_pos.onEnter([]() {
+        wizard_last_x = steppers_current_pos(0);
+        wizard_last_y = steppers_current_pos(1);
+    });
     sm_wiz_last_pos.onLiveUpdate([](size_t i, float v) {
         if (i == 0) {
             steppers_move_x(v);
@@ -215,12 +266,14 @@ void states_init(void) {
         }
     });
 
+    // TODO
     sm_mod_preset.setTitle("Modify Preset");
     sm_mod_preset.setPrefix("Preset ");
     sm_mod_preset.dataCount([]() {
         return (int)data_preset_count();
     });
 
+    // TODO
     sm_del_preset.setTitle("Delete Preset");
     sm_del_preset.setPrefix("Preset ");
     sm_del_preset.dataCount([]() {
@@ -290,6 +343,65 @@ void states_init(void) {
         steppers_set_accel_e(v);
     });
 
+    sm_error.setChild(&sm_ask_homing);
+    sm_error.setTitle("Axis Error");
+    sm_error.setHeading("Axis Error");
+    sm_error.setText("Endstop has been hit!");
+
+#ifdef ENABLE_MOVEMENT_TEST_STATE
+    sm_movement_test.setTitle("Movement Test");
+
+    sm_move_x_test.setTitle("X Axis");
+    sm_move_x_test.setHeading("X Move Test");
+    sm_move_x_test.whenIn([](StateMachineInput smi) {
+        static bool s = false;
+        if (smi.motors_done) {
+            s = !s;
+            steppers_move_x(s ? (X_AXIS_MAX - 10.0) : (X_AXIS_MIN + 10.0));
+            if (smi.click) {
+                states_go_to(&sm_movement_test);
+            }
+        }
+    });
+
+    sm_move_y_test.setTitle("Y Axis");
+    sm_move_y_test.setHeading("Y Move Test");
+    sm_move_y_test.whenIn([](StateMachineInput smi) {
+        static bool s = false;
+        if (smi.motors_done) {
+            s = !s;
+            steppers_move_y(s ? (Y_AXIS_MAX - 10.0) : (Y_AXIS_MIN + 10.0));
+            if (smi.click) {
+                states_go_to(&sm_movement_test);
+            }
+        }
+    });
+#endif // ENABLE_MOVEMENT_TEST_STATE
+
+#ifdef ENABLE_INPUT_TEST_STATE
+    sm_input_test.setTitle("Input Test");
+    sm_input_test.setHeading("Input Test");
+
+    sm_input_test.whenIn([](StateMachineInput smi) {
+        String s = "Endstops: ";
+        for (int i = 0; i < 4; i++) {
+            if (steppers_home_switch(i)) {
+                s += '1';
+            } else {
+                s += '0';
+            }
+            if (i < 3) {
+                s += ' ';
+            }
+        }
+        if (s != strbuf) {
+            strbuf = s;
+            sm_input_test.setText(strbuf.c_str());
+            sm_input_test.updateText();
+        }
+    });
+#endif // ENABLE_INPUT_TEST_STATE
+
     // ----------------------------------
 
     states_go_to(&sm_ask_homing);

src/steppers.cpp

@@ -3,8 +3,10 @@
 
 #include "config.h"
 #include "config_pins.h"
+#include "common.h"
 #include "lcd.h"
 #include "data.h"
+#include "states.h"
 #include "steppers.h"
 
 static AccelStepper stepper_x(AccelStepper::DRIVER, X_STEP_PIN, X_DIR_PIN, 0, 0, false);
@@ -30,8 +32,8 @@ enum stepper_states {
     step_homed
 };
 
-static stepper_states state = step_disabled;
-static unsigned long steppers_home_move_start_time = 0;
+static volatile stepper_states state = step_disabled;
+static volatile unsigned long steppers_home_move_start_time = 0;
 
 void steppers_init(void) {
     pinMode(X_ENDSTOP_PIN, INPUT_PULLUP);
@@ -53,24 +55,45 @@ void steppers_init(void) {
     pinMode(E0_DIR_PIN, OUTPUT);
 
     stepper_x.setEnablePin(X_ENABLE_PIN);
-    stepper_x.setMaxSpeed(XY_MAX_SPEED * XY_STEPS_PER_MM);
+    stepper_x.setPinsInverted(false, false, true);
     steppers_set_speed_x(data_options()->speed_x);
     steppers_set_accel_x(data_options()->accel_x);
 
     stepper_y.setEnablePin(Y_ENABLE_PIN);
+    stepper_y.setPinsInverted(false, false, true);
     steppers_set_speed_y(data_options()->speed_y);
     steppers_set_accel_y(data_options()->accel_y);
 
     stepper_z.setEnablePin(Z_ENABLE_PIN);
+    stepper_z.setPinsInverted(false, false, true);
     steppers_set_speed_z(data_options()->speed_z);
     steppers_set_accel_z(data_options()->accel_z);
 
     stepper_e.setEnablePin(E0_ENABLE_PIN);
+    stepper_e.setPinsInverted(false, false, true);
     steppers_set_speed_e(data_options()->speed_e);
     steppers_set_accel_e(data_options()->accel_e);
 }
 
 static void steppers_initiate_home(int axis, int phase) {
+    Serial.print(F("steppers_initiate_home("));
+    if (axis == 0) {
+        Serial.print('X');
+    } else if (axis == 1) {
+        Serial.print('Y');
+    } else if (axis == 2) {
+        Serial.print('Z');
+    } else if (axis == 3) {
+        Serial.print('E');
+    } else {
+        Serial.print(axis);
+    }
+    Serial.print(F(", "));
+    Serial.print(phase);
+    Serial.println(F(")"));
+
+    CLEAR_STORE_INTERRUPTS();
+
     steppers_home_move_start_time = millis();
 
     if (axis == 0) {
@@ -124,21 +147,52 @@ static void steppers_initiate_home(int axis, int phase) {
     } else {
         Serial.println(F("home_init error: invalid axis"));
     }
+
+    RESTORE_INTERRUPTS();
 }
 
-static bool steppers_home_switch(int axis) {
+bool steppers_home_switch(int axis) {
+    bool r = true;
+    CLEAR_STORE_INTERRUPTS();
+
     if (axis == 0) {
-        return digitalRead(X_ENDSTOP_PIN) == LOW;
+        r = digitalRead(X_ENDSTOP_PIN) == LOW;
     } else if (axis == 1) {
-        return digitalRead(Y_ENDSTOP_PIN) == LOW;
+        r = digitalRead(Y_ENDSTOP_PIN) == LOW;
     } else if (axis == 2) {
-        return digitalRead(Z_ENDSTOP_PIN) == LOW;
+        r = digitalRead(Z_ENDSTOP_PIN) == LOW;
     } else if (axis == 3) {
-        return digitalRead(E_ENDSTOP_PIN) == LOW;
+        r = digitalRead(E_ENDSTOP_PIN) == LOW;
     } else {
         Serial.println(F("home_switch error: invalid axis"));
     }
-    return true;
+
+    RESTORE_INTERRUPTS();
+    return r;
+}
+
+float steppers_current_pos(int axis) {
+    float r = 0.0;
+    CLEAR_STORE_INTERRUPTS();
+
+    if (axis == 0) {
+        float v = stepper_x.currentPosition();
+        r = v / XY_STEPS_PER_MM / X_AXIS_MOVEMENT_DIR;
+    } else if (axis == 1) {
+        float v = stepper_y.currentPosition();
+        r = v / XY_STEPS_PER_MM / Y_AXIS_MOVEMENT_DIR;
+    } else if (axis == 2) {
+        float v = stepper_z.currentPosition();
+        r = v / Z_STEPS_PER_MM / Z_AXIS_MOVEMENT_DIR;
+    } else if (axis == 3) {
+        float v = stepper_e.currentPosition();
+        r = E_PERCENT_TO_MM(v / E_STEPS_PER_PERCENT / E_AXIS_MOVEMENT_DIR);
+    } else {
+        Serial.println(F("steppers_pos error: invalid axis"));
+    }
+
+    RESTORE_INTERRUPTS();
+    return r;
 }
 
 bool steppers_run(void) {
@@ -158,6 +212,13 @@ bool steppers_run(void) {
     } else if (state == step_homing_x_slow) {
         if (steppers_home_switch(0)) {
             stepper_x.setSpeed(0);
+
+#if (X_MIN_PIN == -1)
+            stepper_x.setCurrentPosition(X_AXIS_MAX * XY_STEPS_PER_MM * X_AXIS_MOVEMENT_DIR);
+#else
+            stepper_x.setCurrentPosition(X_AXIS_MIN * XY_STEPS_PER_MM * X_AXIS_MOVEMENT_DIR);
+#endif
+
             steppers_initiate_home(1, 0);
         } else {
             stepper_x.runSpeed();
@@ -178,7 +239,17 @@ bool steppers_run(void) {
     } else if (state == step_homing_y_slow) {
         if (steppers_home_switch(1)) {
             stepper_y.setSpeed(0);
-            steppers_initiate_home(3, 0);
+
+#if (Y_MIN_PIN == -1)
+            stepper_y.setCurrentPosition(Y_AXIS_MAX * XY_STEPS_PER_MM * Y_AXIS_MOVEMENT_DIR);
+#else
+            stepper_y.setCurrentPosition(Y_AXIS_MIN * XY_STEPS_PER_MM * Y_AXIS_MOVEMENT_DIR);
+#endif
+
+            //steppers_initiate_home(3, 0);
+            // TODO
+            state = step_homed;
+            return false;
         } else {
             stepper_y.runSpeed();
         }
@@ -198,6 +269,13 @@ bool steppers_run(void) {
     } else if (state == step_homing_z_slow) {
         if (steppers_home_switch(2)) {
             stepper_z.setSpeed(0);
+
+#if (Z_MIN_PIN == -1)
+            stepper_z.setCurrentPosition(Z_AXIS_MAX * Z_STEPS_PER_MM * Z_AXIS_MOVEMENT_DIR);
+#else
+            stepper_z.setCurrentPosition(Z_AXIS_MIN * Z_STEPS_PER_MM * Z_AXIS_MOVEMENT_DIR);
+#endif
+
             steppers_initiate_home(0, 0);
         } else {
             stepper_z.runSpeed();
@@ -218,24 +296,80 @@ bool steppers_run(void) {
     } else if (state == step_homing_e_slow) {
         if (steppers_home_switch(3)) {
             stepper_e.setSpeed(0);
+
+#if (E_MIN_PIN == -1)
+            stepper_e.setCurrentPosition(E_MM_TO_PERCENT(E_AXIS_MAX) * E_STEPS_PER_PERCENT * E_AXIS_MOVEMENT_DIR);
+#else
+            stepper_e.setCurrentPosition(E_MM_TO_PERCENT(E_AXIS_MIN) * E_STEPS_PER_PERCENT * E_AXIS_MOVEMENT_DIR);
+#endif
+
             state = step_homed;
             return false;
         } else {
             stepper_e.runSpeed();
         }
     } else if (state != step_disabled) {
-        for (int i = 0; i < 4; i++) {
-            if (steppers_home_switch(i)) {
-                Serial.print(F("ERROR: endstop hit on "));
-                Serial.println(i);
-
-                // TODO proper error handling?
-                lcd_clear();
-                lcd_set_heading("ERROR");
-                lcd_set_text("Endstop hit. Aborting!");
-                while (1) { }
+        if (state == step_homed) {
+            for (int i = 0; i < 4; i++) {
+                Serial.print(i);
+                Serial.print(": ");
+                Serial.println(steppers_current_pos(i));
+
+                float compare = 0.0, epsilon = 0.0;
+                if (i == 0) {
+                    epsilon = X_AXIS_EPSILON;
+#if (X_MIN_PIN == -1)
+                    compare = X_AXIS_MAX;
+#else
+                    compare = X_AXIS_MIN;
+#endif
+                } else if (i == 1) {
+                    epsilon = Y_AXIS_EPSILON;
+#if (Y_MIN_PIN == -1)
+                    compare = Y_AXIS_MAX;
+#else
+                    compare = Y_AXIS_MIN;
+#endif
+                } else if (i == 2) {
+                    epsilon = Z_AXIS_EPSILON;
+#if (Z_MIN_PIN == -1)
+                    compare = Z_AXIS_MAX;
+#else
+                    compare = Z_AXIS_MIN;
+#endif
+                } else if (i == 3) {
+                    epsilon = E_AXIS_EPSILON;
+#if (E_MIN_PIN == -1)
+                    compare = E_AXIS_MAX;
+#else
+                    compare = E_AXIS_MIN;
+#endif
+                }
+
+                if (fabs(steppers_current_pos(i) - compare) > epsilon) {
+                    if (steppers_home_switch(i)) {
+                        Serial.print(F("Endstop hit at "));
+                        Serial.println(steppers_current_pos(i));
+
+                        steppers_kill();
+
+                        if (i == 0) {
+                            sm_error.setText("Enstop hit on X axis");
+                        } else if (i == 1) {
+                            sm_error.setText("Enstop hit on Y axis");
+                        } else if (i == 2) {
+                            sm_error.setText("Enstop hit on Z axis");
+                        } else if (i == 3) {
+                            sm_error.setText("Enstop hit on E axis");
+                        } else {
+                            sm_error.setText("Enstop hit on unknown axis");
+                        }
+                        states_go_to(&sm_error);
+                    }
+                }
             }
         }
+
         boolean x = stepper_x.run();
         boolean y = stepper_y.run();
         boolean z = stepper_z.run();
@@ -248,14 +382,48 @@ bool steppers_run(void) {
 }
 
 bool steppers_homed(void) {
-    return (state == step_homed);
+    CLEAR_STORE_INTERRUPTS();
+    bool r = (state == step_homed);
+    RESTORE_INTERRUPTS();
+    return r;
+}
+
+char steppers_homing_axis(void) {
+    CLEAR_STORE_INTERRUPTS();
+
+    char r = ' ';
+    if ((state == step_homing_x_fast) || (state == step_homing_x_back) || (state == step_homing_x_slow)) {
+        r = 'X';
+    } else if ((state == step_homing_y_fast) || (state == step_homing_y_back) || (state == step_homing_y_slow)) {
+        r = 'Y';
+    } else if ((state == step_homing_z_fast) || (state == step_homing_z_back) || (state == step_homing_z_slow)) {
+        r = 'Z';
+    } else if ((state == step_homing_e_fast) || (state == step_homing_e_back) || (state == step_homing_e_slow)) {
+        r = 'E';
+    }
+
+    RESTORE_INTERRUPTS();
+    return r;
 }
 
 void steppers_start_homing(void) {
-    steppers_initiate_home(2, 0);
+    CLEAR_STORE_INTERRUPTS();
+
+    stepper_x.enableOutputs();
+    stepper_y.enableOutputs();
+    stepper_z.enableOutputs();
+    stepper_e.enableOutputs();
+
+    //steppers_initiate_home(2, 0);
+    // TODO
+    steppers_initiate_home(0, 0);
+
+    RESTORE_INTERRUPTS();
 }
 
 static int steppers_move_axis(AccelStepper &axis, long pos) {
+    CLEAR_STORE_INTERRUPTS();
+
     if (state == step_disabled) {
         Serial.println(F("Enabling stepper drivers"));
 
@@ -271,6 +439,8 @@ static int steppers_move_axis(AccelStepper &axis, long pos) {
     }
 
     axis.moveTo(pos);
+
+    RESTORE_INTERRUPTS();
     return 0;
 }
 
@@ -357,26 +527,36 @@ int steppers_move_e(float pos) {
 }
 
 float steppers_pos_x(void) {
+    CLEAR_STORE_INTERRUPTS();
     float v = stepper_x.targetPosition();
+    RESTORE_INTERRUPTS();
     return v / XY_STEPS_PER_MM / X_AXIS_MOVEMENT_DIR;
 }
 
 float steppers_pos_y(void) {
+    CLEAR_STORE_INTERRUPTS();
     float v = stepper_y.targetPosition();
+    RESTORE_INTERRUPTS();
     return v / XY_STEPS_PER_MM / Y_AXIS_MOVEMENT_DIR;
 }
 
 float steppers_pos_z(void) {
+    CLEAR_STORE_INTERRUPTS();
     float v = stepper_z.targetPosition();
+    RESTORE_INTERRUPTS();
     return v / Z_STEPS_PER_MM / Z_AXIS_MOVEMENT_DIR;
 }
 
 float steppers_pos_e(void) {
+    CLEAR_STORE_INTERRUPTS();
     float v = stepper_e.targetPosition();
+    RESTORE_INTERRUPTS();
     return E_PERCENT_TO_MM(v / E_STEPS_PER_PERCENT / E_AXIS_MOVEMENT_DIR);
 }
 
 void steppers_kill(void) {
+    CLEAR_STORE_INTERRUPTS();
+
     stepper_x.setCurrentPosition(0);
     stepper_y.setCurrentPosition(0);
     stepper_z.setCurrentPosition(0);
@@ -388,6 +568,8 @@ void steppers_kill(void) {
     stepper_e.disableOutputs();
 
     state = step_not_homed;
+
+    RESTORE_INTERRUPTS();
 }
 
 void steppers_set_speed_x(float speed) {
@@ -397,7 +579,10 @@ void steppers_set_speed_x(float speed) {
     if (speed > XY_MAX_SPEED) {
         speed = XY_MAX_SPEED;
     }
+
+    CLEAR_STORE_INTERRUPTS();
     stepper_x.setMaxSpeed(speed * XY_STEPS_PER_MM);
+    RESTORE_INTERRUPTS();
 }
 
 void steppers_set_speed_y(float speed) {
@@ -407,7 +592,10 @@ void steppers_set_speed_y(float speed) {
     if (speed > XY_MAX_SPEED) {
         speed = XY_MAX_SPEED;
     }
+
+    CLEAR_STORE_INTERRUPTS();
     stepper_y.setMaxSpeed(speed * XY_STEPS_PER_MM);
+    RESTORE_INTERRUPTS();
 }
 
 void steppers_set_speed_z(float speed) {
@@ -417,7 +605,10 @@ void steppers_set_speed_z(float speed) {
     if (speed > Z_MAX_SPEED) {
         speed = Z_MAX_SPEED;
     }
+
+    CLEAR_STORE_INTERRUPTS();
     stepper_z.setMaxSpeed(speed * Z_STEPS_PER_MM);
+    RESTORE_INTERRUPTS();
 }
 
 void steppers_set_speed_e(float speed) {
@@ -427,7 +618,10 @@ void steppers_set_speed_e(float speed) {
     if (speed > E_MAX_SPEED) {
         speed = E_MAX_SPEED;
     }
+
+    CLEAR_STORE_INTERRUPTS();
     stepper_e.setMaxSpeed(speed * E_STEPS_PER_PERCENT);
+    RESTORE_INTERRUPTS();
 }
 
 void steppers_set_accel_x(float accel) {
@@ -437,7 +631,10 @@ void steppers_set_accel_x(float accel) {
     if (accel > XY_MAX_ACCEL) {
         accel = XY_MAX_ACCEL;
     }
+
+    CLEAR_STORE_INTERRUPTS();
     stepper_x.setAcceleration(accel * XY_STEPS_PER_MM);
+    RESTORE_INTERRUPTS();
 }
 
 void steppers_set_accel_y(float accel) {
@@ -447,7 +644,10 @@ void steppers_set_accel_y(float accel) {
     if (accel > XY_MAX_ACCEL) {
         accel = XY_MAX_ACCEL;
     }
+
+    CLEAR_STORE_INTERRUPTS();
     stepper_y.setAcceleration(accel * XY_STEPS_PER_MM);
+    RESTORE_INTERRUPTS();
 }
 
 void steppers_set_accel_z(float accel) {
@@ -457,7 +657,10 @@ void steppers_set_accel_z(float accel) {
     if (accel > Z_MAX_ACCEL) {
         accel = Z_MAX_ACCEL;
     }
+
+    CLEAR_STORE_INTERRUPTS();
     stepper_z.setAcceleration(accel * Z_STEPS_PER_MM);
+    RESTORE_INTERRUPTS();
 }
 
 void steppers_set_accel_e(float accel) {
@@ -467,5 +670,8 @@ void steppers_set_accel_e(float accel) {
     if (accel > E_MAX_ACCEL) {
         accel = E_MAX_ACCEL;
     }
+
+    CLEAR_STORE_INTERRUPTS();
     stepper_e.setAcceleration(accel * E_STEPS_PER_PERCENT);
+    RESTORE_INTERRUPTS();
 }