diff --git a/Marlin/src/HAL/HAL_AVR/MarlinSerial.cpp b/Marlin/src/HAL/HAL_AVR/MarlinSerial.cpp
index 025254481eb1481445cfa85eb4d6a19ac91b1c90..80a5b0baa7b7dc66a7ea8ec47f598f74598bdc76 100644
--- a/Marlin/src/HAL/HAL_AVR/MarlinSerial.cpp
+++ b/Marlin/src/HAL/HAL_AVR/MarlinSerial.cpp
@@ -69,6 +69,8 @@
uint8_t xon_xoff_state = XON_XOFF_CHAR_SENT | XON_CHAR;
#endif
+ void clear_command_queue();
+
#if ENABLED(SERIAL_STATS_DROPPED_RX)
uint8_t rx_dropped_bytes = 0;
#endif
@@ -388,7 +390,8 @@
// reading rx_buffer_head and updating rx_buffer_tail, the previous rx_buffer_head
// may be written to rx_buffer_tail, making the buffer appear full rather than empty.
CRITICAL_SECTION_START;
- rx_buffer.head = rx_buffer.tail;
+ rx_buffer.head = rx_buffer.tail = 0;
+ clear_command_queue();
CRITICAL_SECTION_END;
#if ENABLED(SERIAL_XON_XOFF)
diff --git a/Marlin/src/config/examples/FolgerTech/i3-2020/Configuration.h b/Marlin/src/config/examples/FolgerTech/i3-2020/Configuration.h
index c50d917a0221d698997537877bf408adc0d5869a..ed8d0ba8a1fed3f1623354654100d07009c734d5 100644
--- a/Marlin/src/config/examples/FolgerTech/i3-2020/Configuration.h
+++ b/Marlin/src/config/examples/FolgerTech/i3-2020/Configuration.h
@@ -555,7 +555,7 @@
* Override with M203
* X, Y, Z, E0 [, E1[, E2[, E3[, E4]]]]
*/
-#define DEFAULT_MAX_FEEDRATE { 250, 250, 2, 17 }
+#define DEFAULT_MAX_FEEDRATE { 250, 250, 4, 17 }
/**
* Default Max Acceleration (change/s) change = mm/s
@@ -587,7 +587,7 @@
*/
#define DEFAULT_XJERK 8.5
#define DEFAULT_YJERK 8.5
-#define DEFAULT_ZJERK 0.3
+#define DEFAULT_ZJERK 0.75
#define DEFAULT_EJERK 4.0
//===========================================================================
@@ -916,7 +916,7 @@
/**
* Enable the G26 Mesh Validation Pattern tool.
*/
- //#define G26_MESH_VALIDATION
+ #define G26_MESH_VALIDATION
#if ENABLED(G26_MESH_VALIDATION)
#define MESH_TEST_NOZZLE_SIZE 0.4 // (mm) Diameter of primary nozzle.
#define MESH_TEST_LAYER_HEIGHT 0.2 // (mm) Default layer height for the G26 Mesh Validation Tool.
diff --git a/Marlin/src/config/examples/FolgerTech/i3-2020/Configuration_adv.h b/Marlin/src/config/examples/FolgerTech/i3-2020/Configuration_adv.h
index a6cdbf0f0feffdc48ac6a5159df3d0b4992eb94a..6babccbe7aa613ac379301870b31fde7aa9f3388 100644
--- a/Marlin/src/config/examples/FolgerTech/i3-2020/Configuration_adv.h
+++ b/Marlin/src/config/examples/FolgerTech/i3-2020/Configuration_adv.h
@@ -1562,7 +1562,7 @@
* Fully assembled MAX7219 boards can be found on the internet for under $2(US).
* For example, see https://www.ebay.com/sch/i.html?_nkw=332349290049
*/
-//#define MAX7219_DEBUG
+#define MAX7219_DEBUG
#if ENABLED(MAX7219_DEBUG)
//#define MAX7219_CLK_PIN 64 // on RAMPS // Configuration of the 3 pins to control the display
//#define MAX7219_DIN_PIN 57 // on RAMPS
diff --git a/Marlin/src/feature/bedlevel/ubl/ubl.h b/Marlin/src/feature/bedlevel/ubl/ubl.h
index b215165e7a28ccc66c6302a96fcd90548796db35..9a0f5caf2f686a0a74a40b83ccf7b830440b20db 100644
--- a/Marlin/src/feature/bedlevel/ubl/ubl.h
+++ b/Marlin/src/feature/bedlevel/ubl/ubl.h
@@ -93,7 +93,7 @@ class unified_bed_leveling {
static bool g29_parameter_parsing();
static void find_mean_mesh_height();
static void shift_mesh_height();
- static void probe_entire_mesh(const float &rx, const float &ry, const bool do_ubl_mesh_map, const bool stow_probe, bool do_furthest);
+ static void probe_entire_mesh(const float &rx, const float &ry, const bool do_ubl_mesh_map, const bool stow_probe, bool do_furthest) _O0;
static void tilt_mesh_based_on_3pts(const float &z1, const float &z2, const float &z3);
static void tilt_mesh_based_on_probed_grid(const bool do_ubl_mesh_map);
static void g29_what_command();
@@ -117,8 +117,8 @@ class unified_bed_leveling {
static void save_ubl_active_state_and_disable();
static void restore_ubl_active_state_and_leave();
static void display_map(const int);
- static mesh_index_pair find_closest_mesh_point_of_type(const MeshPointType, const float&, const float&, const bool, uint16_t[16]);
- static mesh_index_pair find_furthest_invalid_mesh_point();
+ static mesh_index_pair find_closest_mesh_point_of_type(const MeshPointType, const float&, const float&, const bool, uint16_t[16]) _O0;
+ static mesh_index_pair find_furthest_invalid_mesh_point() _O0;
static void reset();
static void invalidate();
static void set_all_mesh_points_to_value(const float);
diff --git a/Marlin/src/feature/bedlevel/ubl/ubl_G29.cpp b/Marlin/src/feature/bedlevel/ubl/ubl_G29.cpp
index 9482e6b7b6a1c37a465afba477f5a440e4f5f44c..4acde7fe4a525f427cbdbe446c17488d41a94d50 100644
--- a/Marlin/src/feature/bedlevel/ubl/ubl_G29.cpp
+++ b/Marlin/src/feature/bedlevel/ubl/ubl_G29.cpp
@@ -36,6 +36,7 @@
#include "../../../module/planner.h"
#include "../../../module/probe.h"
#include "../../../gcode/gcode.h"
+ #include "../../../core/serial.h"
#include "../../../gcode/parser.h"
#include "../../../feature/bedlevel/bedlevel.h"
#include "../../../libs/least_squares_fit.h"
@@ -59,8 +60,6 @@
extern float meshedit_done;
extern long babysteps_done;
- //extern bool set_probe_deployed(bool);
- //extern void set_bed_leveling_enabled(bool);
#define SIZE_OF_LITTLE_RAISE 1
#define BIG_RAISE_NOT_NEEDED 0
@@ -720,7 +719,7 @@
}
}
}
- safe_delay(5);
+ safe_delay(15);
return false;
}
@@ -754,6 +753,7 @@
while (is_lcd_clicked()) idle();
lcd_external_control = false;
restore_ubl_active_state_and_leave();
+ lcd_quick_feedback(true);
safe_delay(50); // Debounce the Encoder wheel
return;
}
@@ -771,6 +771,9 @@
const float measured_z = probe_pt(rawx, rawy, stow_probe, g29_verbose_level); // TODO: Needs error handling
z_values[location.x_index][location.y_index] = measured_z;
}
+ MYSERIAL0.flush(); // G29 P2's take a long time to complete. PronterFace can
+ // over run the serial character buffer with M105's without
+ // this fix
} while (location.x_index >= 0 && --max_iterations);
@@ -853,6 +856,7 @@
do_blocking_move_to_z(Z_CLEARANCE_DEPLOY_PROBE);
lcd_external_control = false;
KEEPALIVE_STATE(IN_HANDLER);
+ lcd_quick_feedback(true);
ubl.restore_ubl_active_state_and_leave();
}
@@ -908,9 +912,12 @@
SERIAL_PROTOCOL_F(z_values[location.x_index][location.y_index], 6);
SERIAL_EOL();
}
+ MYSERIAL0.flush(); // G29 P2's take a long time to complete. PronterFace can
+ // over run the serial character buffer with M105's without
+ // this fix
} while (location.x_index >= 0 && location.y_index >= 0);
- if (do_ubl_mesh_map) display_map(g29_map_type);
+ if (do_ubl_mesh_map) display_map(g29_map_type); // show user where we're probing
restore_ubl_active_state_and_leave();
KEEPALIVE_STATE(IN_HANDLER);
@@ -1035,12 +1042,12 @@
static uint8_t ubl_state_at_invocation = 0;
- #ifdef UBL_DEVEL_DEBUGGING
+ #if ENABLED(UBL_DEVEL_DEBUGGING)
static uint8_t ubl_state_recursion_chk = 0;
#endif
void unified_bed_leveling::save_ubl_active_state_and_disable() {
- #ifdef UBL_DEVEL_DEBUGGING
+ #if ENABLED(UBL_DEVEL_DEBUGGING)
ubl_state_recursion_chk++;
if (ubl_state_recursion_chk != 1) {
SERIAL_ECHOLNPGM("save_ubl_active_state_and_disabled() called multiple times in a row.");
@@ -1056,7 +1063,7 @@
}
void unified_bed_leveling::restore_ubl_active_state_and_leave() {
- #ifdef UBL_DEVEL_DEBUGGING
+ #if ENABLED(UBL_DEVEL_DEBUGGING)
if (--ubl_state_recursion_chk) {
SERIAL_ECHOLNPGM("restore_ubl_active_state_and_leave() called too many times.");
#if ENABLED(NEWPANEL)
@@ -1110,11 +1117,12 @@
SERIAL_ECHOLNPAIR("MESH_MAX_Y " STRINGIFY(MESH_MAX_Y) "=", MESH_MAX_Y);
safe_delay(50);
SERIAL_ECHOLNPAIR("GRID_MAX_POINTS_X ", GRID_MAX_POINTS_X);
+ safe_delay(50);
SERIAL_ECHOLNPAIR("GRID_MAX_POINTS_Y ", GRID_MAX_POINTS_Y);
- safe_delay(25);
+ safe_delay(50);
SERIAL_ECHOLNPAIR("MESH_X_DIST ", MESH_X_DIST);
SERIAL_ECHOLNPAIR("MESH_Y_DIST ", MESH_Y_DIST);
- safe_delay(25);
+ safe_delay(50);
SERIAL_PROTOCOLPGM("X-Axis Mesh Points at: ");
for (uint8_t i = 0; i < GRID_MAX_POINTS_X; i++) {
@@ -1139,7 +1147,7 @@
SERIAL_EOL();
safe_delay(50);
- #ifdef UBL_DEVEL_DEBUGGING
+ #if ENABLED(UBL_DEVEL_DEBUGGING)
SERIAL_PROTOCOLLNPAIR("ubl_state_at_invocation :", ubl_state_at_invocation);
SERIAL_EOL();
SERIAL_PROTOCOLLNPAIR("ubl_state_recursion_chk :", ubl_state_recursion_chk);
@@ -1352,6 +1360,7 @@
lcd_return_to_status();
do_blocking_move_to_z(Z_CLEARANCE_BETWEEN_PROBES);
LCD_MESSAGEPGM(MSG_EDITING_STOPPED);
+ lcd_quick_feedback(true);
}
void unified_bed_leveling::fine_tune_mesh(const float &rx, const float &ry, const bool do_ubl_mesh_map) {
@@ -1417,6 +1426,9 @@
do_blocking_move_to_z(h_offset + new_z); // Move the nozzle as the point is edited
#endif
idle();
+ MYSERIAL0.flush(); // G29 P2's take a long time to complete. PronterFace can
+ // over run the serial character buffer with M105's without
+ // this fix
} while (!is_lcd_clicked());
if (!lcd_map_control) lcd_return_to_status();
@@ -1426,9 +1438,6 @@
// Let's work on specifying a proper API for the LCD ASAP, OK?
lcd_external_control = true;
- // this sequence to detect an is_lcd_clicked() debounce it and leave if it is
- // a Press and Hold is repeated in a lot of places (including G26_Mesh_Validation.cpp). This
- // should be redone and compressed.
if (click_and_hold(abort_fine_tune))
goto FINE_TUNE_EXIT;
diff --git a/Marlin/src/gcode/bedlevel/G26.cpp b/Marlin/src/gcode/bedlevel/G26.cpp
index 5e2bc46c96a6945770951de26766eae642c8e043..0cb4dd76040823f7c08644a687e4364338fdda57 100644
--- a/Marlin/src/gcode/bedlevel/G26.cpp
+++ b/Marlin/src/gcode/bedlevel/G26.cpp
@@ -437,6 +437,9 @@ inline bool turn_on_heaters() {
SERIAL_EOL();
}
idle();
+ MYSERIAL0.flush(); // G26 takes a long time to complete. PronterFace can
+ // over run the serial character buffer with M105's without
+ // this fix
}
#if ENABLED(ULTRA_LCD)
}
@@ -459,6 +462,10 @@ inline bool turn_on_heaters() {
SERIAL_EOL();
}
idle();
+
+ MYSERIAL0.flush(); // G26 takes a long time to complete. PronterFace can
+ // over run the serial character buffer with M105's without
+ // this fix
}
#if ENABLED(ULTRA_LCD)
@@ -680,12 +687,12 @@ void GcodeSuite::G26() {
set_bed_leveling_enabled(!parser.seen('D'));
if (current_position[Z_AXIS] < Z_CLEARANCE_BETWEEN_PROBES) {
-SERIAL_PROTOCOLLNPGM("! move nozzle to Z_CLEARANCE_BETWEEN_PROBES height.");
-SERIAL_ECHOLNPAIR(" Z at:", current_position[Z_AXIS]);
+// SERIAL_PROTOCOLLNPGM("! move nozzle to Z_CLEARANCE_BETWEEN_PROBES height.");
+// SERIAL_ECHOLNPAIR(" Z at:", current_position[Z_AXIS]);
do_blocking_move_to_z(Z_CLEARANCE_BETWEEN_PROBES);
stepper.synchronize();
set_current_from_destination();
-SERIAL_ECHOLNPAIR(" Z now at:", current_position[Z_AXIS]);
+// SERIAL_ECHOLNPAIR(" Z now at:", current_position[Z_AXIS]);
}
if (turn_on_heaters() != G26_OK) goto LEAVE;
@@ -711,14 +718,14 @@ SERIAL_ECHOLNPAIR(" Z now at:", current_position[Z_AXIS]);
// Move nozzle to the specified height for the first layer
set_destination_from_current();
-SERIAL_PROTOCOLLNPGM("! moving nozzle to 1st layer height.");
-SERIAL_ECHOLNPAIR(" Z1 at:", current_position[Z_AXIS]);
+//SERIAL_PROTOCOLLNPGM("! moving nozzle to 1st layer height.");
+//SERIAL_ECHOLNPAIR(" Z1 at:", current_position[Z_AXIS]);
destination[Z_AXIS] = g26_layer_height;
move_to(destination, 0.0);
- stepper.synchronize();
- set_destination_from_current();
-SERIAL_ECHOLNPAIR(" Z2 at:", current_position[Z_AXIS]);
+//stepper.synchronize();
+//set_destination_from_current();
+//SERIAL_ECHOLNPAIR(" Z2 at:", current_position[Z_AXIS]);
move_to(destination, g26_ooze_amount);
#if ENABLED(ULTRA_LCD)
@@ -823,11 +830,17 @@ SERIAL_ECHOLNPAIR(" Z2 at:", current_position[Z_AXIS]);
//}
print_line_from_here_to_there(rx, ry, g26_layer_height, xe, ye, g26_layer_height);
+ MYSERIAL0.flush(); // G26 takes a long time to complete. PronterFace can
+ // over run the serial character buffer with M105's without
+ // this fix
}
if (look_for_lines_to_connect())
goto LEAVE;
}
+ MYSERIAL0.flush(); // G26 takes a long time to complete. PronterFace can
+ // over run the serial character buffer with M105's without
+ // this fix
} while (--g26_repeats && location.x_index >= 0 && location.y_index >= 0);
LEAVE:
diff --git a/Marlin/src/gcode/queue.cpp b/Marlin/src/gcode/queue.cpp
index 8e5985e7aeba80b28e1a3009322e52a24d18230d..f613ab1362e17095eed8fceea27cf7b8bfa95a20 100644
--- a/Marlin/src/gcode/queue.cpp
+++ b/Marlin/src/gcode/queue.cpp
@@ -90,8 +90,7 @@ void queue_setup() {
* Clear the Marlin command queue
*/
void clear_command_queue() {
- cmd_queue_index_r = cmd_queue_index_w;
- commands_in_queue = 0;
+ cmd_queue_index_r = cmd_queue_index_w = commands_in_queue = 0;
}
/**