From 18dec75b560f78b91388d61c0fe67d94e41e5c00 Mon Sep 17 00:00:00 2001 From: Robert Pelnar Date: Mon, 29 Apr 2019 13:19:59 +0200 Subject: [PATCH 1/2] MK2.5 - watchdog enabled + test (command "D-1") --- Firmware/Dcodes.cpp | 8 ++++---- Firmware/Marlin_main.cpp | 4 ++-- Firmware/variants/1_75mm_MK25-RAMBo10a-E3Dv6full.h | 3 +++ Firmware/variants/1_75mm_MK25-RAMBo13a-E3Dv6full.h | 3 +++ 4 files changed, 12 insertions(+), 6 deletions(-) diff --git a/Firmware/Dcodes.cpp b/Firmware/Dcodes.cpp index 44dfc4654..d2016ded5 100644 --- a/Firmware/Dcodes.cpp +++ b/Firmware/Dcodes.cpp @@ -176,15 +176,15 @@ extern float axis_steps_per_unit[NUM_AXIS]; #endif //0 #define LOG(args...) -#ifdef DEBUG_DCODES - void dcode__1() { - printf("D-1 - Endless loop\n"); - cli(); + printf_P(PSTR("D-1 - Endless loop\n")); +// cli(); while (1); } +#ifdef DEBUG_DCODES + void dcode_0() { if (*(strchr_pointer + 1) == 0) return; diff --git a/Firmware/Marlin_main.cpp b/Firmware/Marlin_main.cpp index e23ca37e8..bb13f673a 100644 --- a/Firmware/Marlin_main.cpp +++ b/Firmware/Marlin_main.cpp @@ -3538,7 +3538,7 @@ void process_commands() else if (code_seen("RESET")) { //! PRUSA RESET // careful! if (farm_mode) { -#ifdef WATCHDOG +#if (defined(WATCHDOG) && (MOTHERBOARD == BOARD_EINSY_1_0a)) boot_app_magic = BOOT_APP_MAGIC; boot_app_flags = BOOT_APP_FLG_RUN; wdt_enable(WDTO_15MS); @@ -7174,9 +7174,9 @@ if((eSoundMode==e_SOUND_MODE_LOUD)||(eSoundMode==e_SOUND_MODE_ONCE)) { switch((int)code_value()) { -#ifdef DEBUG_DCODES case -1: //! D-1 - Endless loop dcode__1(); break; +#ifdef DEBUG_DCODES case 0: //! D0 - Reset dcode_0(); break; case 1: //! D1 - Clear EEPROM diff --git a/Firmware/variants/1_75mm_MK25-RAMBo10a-E3Dv6full.h b/Firmware/variants/1_75mm_MK25-RAMBo10a-E3Dv6full.h index b5e9c1af0..f1d2c785c 100644 --- a/Firmware/variants/1_75mm_MK25-RAMBo10a-E3Dv6full.h +++ b/Firmware/variants/1_75mm_MK25-RAMBo10a-E3Dv6full.h @@ -103,6 +103,9 @@ // New XYZ calibration #define NEW_XYZCAL +// Watchdog support +#define WATCHDOG + // Fan check #define FANCHECK diff --git a/Firmware/variants/1_75mm_MK25-RAMBo13a-E3Dv6full.h b/Firmware/variants/1_75mm_MK25-RAMBo13a-E3Dv6full.h index 05b5e0923..ac21da414 100644 --- a/Firmware/variants/1_75mm_MK25-RAMBo13a-E3Dv6full.h +++ b/Firmware/variants/1_75mm_MK25-RAMBo13a-E3Dv6full.h @@ -104,6 +104,9 @@ // New XYZ calibration #define NEW_XYZCAL +// Watchdog support +#define WATCHDOG + // Fan check #define FANCHECK From 8fadbf1b4407c556f006807946865409016a4126 Mon Sep 17 00:00:00 2001 From: Marek Bel Date: Mon, 29 Apr 2019 16:41:38 +0200 Subject: [PATCH 2/2] Do not call watchdog reset from updateTemperaturesFromRawValues(). --- Firmware/temperature.cpp | 5 ----- 1 file changed, 5 deletions(-) diff --git a/Firmware/temperature.cpp b/Firmware/temperature.cpp index e432d028a..45256bf42 100644 --- a/Firmware/temperature.cpp +++ b/Firmware/temperature.cpp @@ -1046,11 +1046,6 @@ static void updateTemperaturesFromRawValues() redundant_temperature = analog2temp(redundant_temperature_raw, 1); #endif - //Reset the watchdog after we know we have a temperature measurement. -#ifdef WATCHDOG - wdt_reset(); -#endif //WATCHDOG - CRITICAL_SECTION_START; temp_meas_ready = false; CRITICAL_SECTION_END;