Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warn of possible endstop/DIAG homing conflicts #23050

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Marlin/src/gcode/gcode.h
Original file line number Diff line number Diff line change
Expand Up @@ -1109,7 +1109,7 @@ class GcodeSuite {
static void M913();
static void M913_report(const bool forReplay=true);
#endif
#if ENABLED(USE_SENSORLESS)
#if USE_SENSORLESS
static void M914();
static void M914_report(const bool forReplay=true);
#endif
Expand Down
9 changes: 9 additions & 0 deletions Marlin/src/inc/Warnings.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -488,3 +488,12 @@
#if HOMING_Z_WITH_PROBE && IS_CARTESIAN && DISABLED(Z_SAFE_HOMING)
#error "Z_SAFE_HOMING is recommended when homing with a probe. Enable Z_SAFE_HOMING or comment out this line to continue."
#endif

//
// Warn users of potential endstop/DIAG pin conflicts to prevent homing issues when not using sensorless homing
//
#if !USE_SENSORLESS && ENABLED(USES_DIAG_JUMPERS)
#warning "Motherboard DIAG jumpers must be removed when SENSORLESS_HOMING is disabled."
#elif !USE_SENSORLESS && ENABLED(USES_DIAG_PINS)
#warning "Driver DIAG pins must be physically removed unless SENSORLESS_HOMING is enabled. (See https://bit.ly/2ZPRlt0)"
#endif
4 changes: 3 additions & 1 deletion Marlin/src/pins/lpc1768/pins_BTT_SKR_V1_3.h
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,9 @@
*/

#define BOARD_INFO_NAME "BTT SKR V1.3"
#define LPC1768_IS_SKRV1_3 1

#define LPC1768_IS_SKRV1_3
#define USES_DIAG_JUMPERS

//
// Trinamic Stallguard pins
Expand Down
2 changes: 2 additions & 0 deletions Marlin/src/pins/lpc1768/pins_BTT_SKR_V1_4.h
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@
#define BOARD_CUSTOM_BUILD_FLAGS -DLPC_PINCFG_UART3_P4_28
#endif

#define USES_DIAG_PINS

//
// EEPROM
//
Expand Down
2 changes: 2 additions & 0 deletions Marlin/src/pins/lpc1768/pins_MKS_SGEN_L.h
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,8 @@
#define BOARD_INFO_NAME "MKS SGen-L"
#define BOARD_WEBSITE_URL "github.com/makerbase-mks/MKS-SGEN_L"

#define USES_DIAG_JUMPERS

//
// Servos
//
Expand Down
2 changes: 2 additions & 0 deletions Marlin/src/pins/lpc1769/pins_BTT_SKR_E3_TURBO.h
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,8 @@
#define BOARD_INFO_NAME "BTT SKR E3 Turbo"
#endif

#define USES_DIAG_JUMPERS

// Onboard I2C EEPROM
#define I2C_EEPROM
#define MARLIN_EEPROM_SIZE 0x1000 // 4KB (AT24C32)
Expand Down
2 changes: 2 additions & 0 deletions Marlin/src/pins/lpc1769/pins_MKS_SGEN_L_V2.h
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,8 @@
#define BOARD_INFO_NAME "MKS SGEN_L V2"
#define BOARD_WEBSITE_URL "github.com/makerbase-mks"

#define USES_DIAG_JUMPERS

//
// EEPROM, MKS SGEN_L V2.0 hardware has 4K EEPROM on the board
//
Expand Down
2 changes: 2 additions & 0 deletions Marlin/src/pins/stm32f1/pins_BTT_SKR_MINI_E3_common.h
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@
// Release PB3/PB4 (E0 STP/DIR) from JTAG pins
#define DISABLE_JTAG

#define USES_DIAG_JUMPERS

// Ignore temp readings during development.
//#define BOGUS_TEMPERATURE_GRACE_PERIOD 2000

Expand Down
5 changes: 3 additions & 2 deletions Marlin/src/pins/stm32f1/pins_MKS_ROBIN_E3_common.h
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,11 @@

#include "env_validate.h"

#define BOARD_NO_NATIVE_USB

#define BOARD_WEBSITE_URL "github.com/makerbase-mks"

#define BOARD_NO_NATIVE_USB
#define USES_DIAG_JUMPERS

//#define DISABLE_DEBUG
#define DISABLE_JTAG

Expand Down
1 change: 1 addition & 0 deletions Marlin/src/pins/stm32f1/pins_MKS_ROBIN_NANO_V2.h
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@
#define BOARD_INFO_NAME "MKS Robin nano V2.0"

#define BOARD_NO_NATIVE_USB
#define USES_DIAG_PINS

// Avoid conflict with TIMER_SERVO when using the STM32 HAL
#define TEMP_TIMER 5
Expand Down
8 changes: 5 additions & 3 deletions Marlin/src/pins/stm32f4/pins_BTT_BTT002_V1_0.h
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,11 @@

#define BOARD_INFO_NAME "BTT BTT002 V1.0"

#define USES_DIAG_PINS

// Ignore temp readings during development.
//#define BOGUS_TEMPERATURE_GRACE_PERIOD 2000

// Use one of these or SDCard-based Emulation will be used
#if NO_EEPROM_SELECTED
//#define SRAM_EEPROM_EMULATION // Use BackSRAM-based EEPROM emulation
Expand All @@ -41,9 +46,6 @@
#define FLASH_EEPROM_LEVELING
#endif

// Ignore temp readings during development.
//#define BOGUS_TEMPERATURE_GRACE_PERIOD 2000

//
// Limit Switches
//
Expand Down
2 changes: 2 additions & 0 deletions Marlin/src/pins/stm32f4/pins_BTT_E3_RRF.h
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,8 @@
#define BOARD_INFO_NAME "BTT E3 RRF"
#endif

#define USES_DIAG_JUMPERS

// Add-on board for IDEX conversion
//#define BTT_E3_RRF_IDEX_BOARD

Expand Down
9 changes: 4 additions & 5 deletions Marlin/src/pins/stm32f4/pins_BTT_GTR_V1_0.h
Original file line number Diff line number Diff line change
Expand Up @@ -31,15 +31,14 @@

#define BOARD_INFO_NAME "BTT GTR V1.0"

#define USES_DIAG_JUMPERS
#define HAS_OTG_USB_HOST_SUPPORT // USB Flash Drive support
#define M5_EXTENDER // The M5 extender is attached

// Onboard I2C EEPROM
#define I2C_EEPROM
#define MARLIN_EEPROM_SIZE 0x2000 // 8KB (24C64 ... 64Kb = 8KB)

// USB Flash Drive support
#define HAS_OTG_USB_HOST_SUPPORT

#define M5_EXTENDER // The M5 extender is attached

//
// Servos
//
Expand Down
6 changes: 3 additions & 3 deletions Marlin/src/pins/stm32f4/pins_BTT_OCTOPUS_V1_common.h
Original file line number Diff line number Diff line change
Expand Up @@ -23,15 +23,15 @@

#include "env_validate.h"

#define HAS_OTG_USB_HOST_SUPPORT // USB Flash Drive support
#define USES_DIAG_JUMPERS

// Onboard I2C EEPROM
#define I2C_EEPROM
#define MARLIN_EEPROM_SIZE 0x8000 // 32KB (24C32A)
#define I2C_SCL_PIN PB8
#define I2C_SDA_PIN PB9

// USB Flash Drive support
#define HAS_OTG_USB_HOST_SUPPORT

// Avoid conflict with TIMER_TONE
#define STEP_TIMER 10

Expand Down
5 changes: 3 additions & 2 deletions Marlin/src/pins/stm32f4/pins_BTT_SKR_PRO_common.h
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@

#include "env_validate.h"

#define USES_DIAG_JUMPERS

// If you have the BigTreeTech driver expansion module, enable BTT_MOTOR_EXPANSION
// https://github.com/bigtreetech/BTT-Expansion-module/tree/master/BTT%20EXP-MOT
//#define BTT_MOTOR_EXPANSION
Expand All @@ -47,8 +49,7 @@
#define FLASH_EEPROM_LEVELING
#endif

// USB Flash Drive support
#define HAS_OTG_USB_HOST_SUPPORT
#define HAS_OTG_USB_HOST_SUPPORT // USB Flash Drive support

//
// Servos
Expand Down
3 changes: 1 addition & 2 deletions Marlin/src/pins/stm32f4/pins_BTT_SKR_V2_0_common.h
Original file line number Diff line number Diff line change
Expand Up @@ -47,8 +47,7 @@
#define FLASH_EEPROM_LEVELING
#endif

// USB Flash Drive support
#define HAS_OTG_USB_HOST_SUPPORT
#define HAS_OTG_USB_HOST_SUPPORT // USB Flash Drive support

// Avoid conflict with TIMER_TONE
#define STEP_TIMER 10
Expand Down
3 changes: 1 addition & 2 deletions Marlin/src/pins/stm32f4/pins_LERDGE_K.h
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,7 @@
#define MARLIN_EEPROM_SIZE 0x10000
#endif

// USB Flash Drive support
#define HAS_OTG_USB_HOST_SUPPORT
#define HAS_OTG_USB_HOST_SUPPORT // USB Flash Drive support

//
// Servos
Expand Down
3 changes: 1 addition & 2 deletions Marlin/src/pins/stm32f4/pins_LERDGE_S.h
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,7 @@
#define STEP_TIMER 4
#define TEMP_TIMER 2

// USB Flash Drive support
#define HAS_OTG_USB_HOST_SUPPORT
#define HAS_OTG_USB_HOST_SUPPORT // USB Flash Drive support

//
// Servos
Expand Down
3 changes: 1 addition & 2 deletions Marlin/src/pins/stm32f4/pins_LERDGE_X.h
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,7 @@
#define I2C_SDA_PIN PB9
#define MARLIN_EEPROM_SIZE 0x10000 // FM24CL64 F-RAM 64K (8Kx8)

// USB Flash Drive support
#define HAS_OTG_USB_HOST_SUPPORT
#define HAS_OTG_USB_HOST_SUPPORT // USB Flash Drive support

//
// Servos
Expand Down
4 changes: 2 additions & 2 deletions Marlin/src/pins/stm32f4/pins_MKS_MONSTER8.h
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,8 @@

#define BOARD_INFO_NAME "MKS Monster8 V1.x"

// USB Flash Drive support
#define HAS_OTG_USB_HOST_SUPPORT
#define HAS_OTG_USB_HOST_SUPPORT // USB Flash Drive support
#define USES_DIAG_JUMPERS

//#define DISABLE_DEBUG

Expand Down
6 changes: 2 additions & 4 deletions Marlin/src/pins/stm32f4/pins_MKS_ROBIN_NANO_V3.h
Original file line number Diff line number Diff line change
Expand Up @@ -32,22 +32,20 @@

#define BOARD_INFO_NAME "MKS Robin Nano V3"

#define USES_DIAG_JUMPERS

#ifndef X_CS_PIN
#define X_CS_PIN PD5
#endif

#ifndef Y_CS_PIN
#define Y_CS_PIN PD7
#endif

#ifndef Z_CS_PIN
#define Z_CS_PIN PD4
#endif

#ifndef E0_CS_PIN
#define E0_CS_PIN PD9
#endif

#ifndef E1_CS_PIN
#define E1_CS_PIN PD8
#endif
Expand Down
3 changes: 1 addition & 2 deletions Marlin/src/pins/stm32f4/pins_MKS_ROBIN_NANO_V3_common.h
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,7 @@
// MKS Robin Nano V3, MKS Eagle pinmap
//

// USB Flash Drive support
#define HAS_OTG_USB_HOST_SUPPORT
#define HAS_OTG_USB_HOST_SUPPORT // USB Flash Drive support

// Avoid conflict with TIMER_TONE
#define STEP_TIMER 10
Expand Down
3 changes: 1 addition & 2 deletions Marlin/src/pins/stm32f4/pins_MKS_ROBIN_PRO_V2.h
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,7 @@
#define I2C_EEPROM
#define MARLIN_EEPROM_SIZE 0x1000 // 4KB

// USB Flash Drive support
#define HAS_OTG_USB_HOST_SUPPORT
#define HAS_OTG_USB_HOST_SUPPORT // USB Flash Drive support

//
// Release PB4 (Y_ENABLE_PIN) from JTAG NRST role
Expand Down
3 changes: 1 addition & 2 deletions Marlin/src/pins/stm32h7/pins_BTT_SKR_SE_BX.h
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,7 @@
#define I2C_EEPROM
#define MARLIN_EEPROM_SIZE 0x1000 // 4KB (24C32 ... 32Kb = 4KB)

// USB Flash Drive support
#define HAS_OTG_USB_HOST_SUPPORT
#define HAS_OTG_USB_HOST_SUPPORT // USB Flash Drive support

//
// Limit Switches
Expand Down