diff --git a/Marlin/src/Marlin.cpp b/Marlin/src/Marlin.cpp index cc755d4cf051f75df4f59900d1cdd0c1f13e6fa1..4abdf4f9a2bf2964cf7f8d0fcca612599deb2f2a 100644 --- a/Marlin/src/Marlin.cpp +++ b/Marlin/src/Marlin.cpp @@ -972,11 +972,8 @@ void loop() { #if ENABLED(SDSUPPORT) card.checkautostart(); - #endif - - #if ENABLED(SDSUPPORT) && (ENABLED(ULTIPANEL) || ENABLED(EXTENSIBLE_UI)) - if (abort_sd_printing) { - abort_sd_printing = false; + + if (card.abort_sd_printing) { card.stopSDPrint( #if SD_RESORT true @@ -992,7 +989,7 @@ void loop() { card.removeJobRecoveryFile(); #endif } - #endif // SDSUPPORT && (ENABLED(ULTIPANEL) || ENABLED(EXTENSIBLE_UI)) + #endif // SDSUPPORT if (commands_in_queue < BUFSIZE) get_available_commands(); advance_command_queue(); diff --git a/Marlin/src/gcode/gcode.cpp b/Marlin/src/gcode/gcode.cpp index 4de056de7a3dc8d2a334e2889e3a953ab05164a8..17d7d51dd33d0384891b7fbab0abb7eceaafbc7c 100644 --- a/Marlin/src/gcode/gcode.cpp +++ b/Marlin/src/gcode/gcode.cpp @@ -584,6 +584,10 @@ void GcodeSuite::process_parsed_command( case 504: M504(); break; // M504: Validate EEPROM contents #endif + #if ENABLED(SDSUPPORT) + case 524: M524(); break; // M524: Abort the current SD print job + #endif + #if ENABLED(ABORT_ON_ENDSTOP_HIT_FEATURE_ENABLED) case 540: M540(); break; // M540: Set abort on endstop hit for SD printing #endif diff --git a/Marlin/src/gcode/gcode.h b/Marlin/src/gcode/gcode.h index 8a8a9b0076d2d78f18204cb1526685da7983e9a5..560cf2e1225d9faada7a20891c80540fe4046857 100644 --- a/Marlin/src/gcode/gcode.h +++ b/Marlin/src/gcode/gcode.h @@ -202,6 +202,7 @@ * M501 - Restore parameters from EEPROM. (Requires EEPROM_SETTINGS) * M502 - Revert to the default "factory settings". ** Does not write them to EEPROM! ** * M503 - Print the current settings (in memory): "M503 S<verbose>". S0 specifies compact output. + * M524 - Abort the current SD print job (started with M24) * M540 - Enable/disable SD card abort on endstop hit: "M540 S<state>". (Requires ABORT_ON_ENDSTOP_HIT_FEATURE_ENABLED) * M600 - Pause for filament change: "M600 X<pos> Y<pos> Z<raise> E<first_retract> L<later_retract>". (Requires ADVANCED_PAUSE_FEATURE) * M603 - Configure filament change: "M603 T<tool> U<unload_length> L<load_length>". (Requires ADVANCED_PAUSE_FEATURE) @@ -719,6 +720,10 @@ private: static void M504(); #endif + #if ENABLED(SDSUPPORT) + static void M524(); + #endif + #if ENABLED(ABORT_ON_ENDSTOP_HIT_FEATURE_ENABLED) static void M540(); #endif diff --git a/Marlin/src/gcode/sdcard/M20-M30_M32-M34_M928.cpp b/Marlin/src/gcode/sdcard/M20-M30_M32-M34_M524_M928.cpp similarity index 97% rename from Marlin/src/gcode/sdcard/M20-M30_M32-M34_M928.cpp rename to Marlin/src/gcode/sdcard/M20-M30_M32-M34_M524_M928.cpp index 6a982b74b0ac7b9c30416e0471962ce2aaa66a5b..81cca4c0d3a282b9f7a336af923007a6bcfd41b6 100644 --- a/Marlin/src/gcode/sdcard/M20-M30_M32-M34_M928.cpp +++ b/Marlin/src/gcode/sdcard/M20-M30_M32-M34_M524_M928.cpp @@ -244,6 +244,13 @@ void GcodeSuite::M32() { #endif // SDCARD_SORT_ALPHA && SDSORT_GCODE +/** + * M524: Abort the current SD print job (started with M24) + */ +void GcodeSuite::M524() { + if (IS_SD_PRINTING()) card.abort_sd_printing = true; +} + /** * M928: Start SD Write */ diff --git a/Marlin/src/lcd/extensible_ui/ui_api.cpp b/Marlin/src/lcd/extensible_ui/ui_api.cpp index e13ca08758ab5cae679292e089260d48e0aec6af..47b635f54fbc0910dfddf4b5faf58c2b7d022f46 100644 --- a/Marlin/src/lcd/extensible_ui/ui_api.cpp +++ b/Marlin/src/lcd/extensible_ui/ui_api.cpp @@ -37,10 +37,6 @@ #if ENABLED(SDSUPPORT) #include "../../sd/cardreader.h" #include "../../feature/emergency_parser.h" - - bool abort_sd_printing; // =false -#else - constexpr bool abort_sd_printing = false; #endif #if ENABLED(PRINTCOUNTER) @@ -476,7 +472,7 @@ namespace UI { void stopPrint() { #if ENABLED(SDSUPPORT) wait_for_heatup = wait_for_user = false; - abort_sd_printing = true; + card.abort_sd_printing = true; UI::onStatusChanged(PSTR(MSG_PRINT_ABORTED)); #endif } diff --git a/Marlin/src/lcd/ultralcd.cpp b/Marlin/src/lcd/ultralcd.cpp index 85a2180bb158e68f18a534f51557f9bc7c3f6907..8da41adbb45a06cfd32217419aef6dcd0582dd9e 100644 --- a/Marlin/src/lcd/ultralcd.cpp +++ b/Marlin/src/lcd/ultralcd.cpp @@ -895,11 +895,9 @@ void lcd_quick_feedback(const bool clear_buttons) { lcd_reset_status(); } - bool abort_sd_printing; // =false - void lcd_sdcard_stop() { wait_for_heatup = wait_for_user = false; - abort_sd_printing = true; + card.abort_sd_printing = true; lcd_setstatusPGM(PSTR(MSG_PRINT_ABORTED), -1); lcd_return_to_status(); } diff --git a/Marlin/src/lcd/ultralcd.h b/Marlin/src/lcd/ultralcd.h index 2b262f6af1e3b3f7b9e3fd1b919c94a758262293..b1d9490ecf801704643a795d3d7d71b003990667 100644 --- a/Marlin/src/lcd/ultralcd.h +++ b/Marlin/src/lcd/ultralcd.h @@ -288,10 +288,5 @@ void lcd_reselect_last_file(); #endif -#if (ENABLED(EXTENSIBLE_UI) || ENABLED(ULTIPANEL)) && ENABLED(SDSUPPORT) - extern bool abort_sd_printing; -#else - constexpr bool abort_sd_printing = false; -#endif #endif // ULTRALCD_H diff --git a/Marlin/src/sd/cardreader.cpp b/Marlin/src/sd/cardreader.cpp index 68ebe99d4aa5d905e472ce9629f5388e157e4e98..91a3a8d04c12e78e19f6a742002acc8806ca66c2 100644 --- a/Marlin/src/sd/cardreader.cpp +++ b/Marlin/src/sd/cardreader.cpp @@ -351,7 +351,7 @@ void CardReader::stopSDPrint( #if ENABLED(ADVANCED_PAUSE_FEATURE) did_pause_print = 0; #endif - sdprinting = false; + sdprinting = abort_sd_printing = false; if (isFileOpen()) file.close(); #if SD_RESORT if (re_sort) presort(); diff --git a/Marlin/src/sd/cardreader.h b/Marlin/src/sd/cardreader.h index 42e35656fb9e8330f08a7e50c2510d87885cbebf..a92196f9869c7324502d6f0edbd337baa61ce38c 100644 --- a/Marlin/src/sd/cardreader.h +++ b/Marlin/src/sd/cardreader.h @@ -149,7 +149,7 @@ public: FORCE_INLINE char* longest_filename() { return longFilename[0] ? longFilename : filename; } public: - bool saving, logging, sdprinting, cardOK, filenameIsDir; + bool saving, logging, sdprinting, cardOK, filenameIsDir, abort_sd_printing; char filename[FILENAME_LENGTH], longFilename[LONG_FILENAME_LENGTH]; int8_t autostart_index; private: