search

MarlinFirmware / Marlin

Marlin is an optimized firmware for RepRap 3D printers based on the Arduino platform. Many commercial 3D printers come with Marlin installed. Check with your vendor if you need source code for your specific machine.
https://marlinfw.org
GNU General Public License v3.0
16.3k stars 19.25k forks source link

TMC2226 not included as sensorless homing or probing driver in Sanity Check #19997

Closed hackebike closed 4 years ago

hackebike commented 4 years ago

I am using TMC 2226 as drivers with sensorless homing and probing. Tried to compiled for SKR GTR and got this error: config files: Archive.zip Workaround, use 2209 instead of 2226 in the DRIVER_TYPE section and it compiles.

`In file included from Marlin/src/HAL/STM32/../../inc/MarlinConfig.h:49, from Marlin/src/HAL/STM32/HAL.cpp:28: Marlin/src/HAL/STM32/../../inc/SanityCheck.h:1307:8: error: #error "SENSORLESS_PROBING requires a TMC2130/2160/2209/5130/5160 driver on Z." 1307 | #error "SENSORLESS_PROBING requires a TMC2130/2160/2209/5130/5160 driver on Z." | ^~~~~ Marlin/src/HAL/STM32/../../inc/SanityCheck.h:2591:6: error: #error "SENSORLESS_PROBING requires a TMC stepper driver with StallGuard on Z." 2591 | #error "SENSORLESS_PROBING requires a TMC stepper driver with StallGuard on Z." | ^~~~~ Marlin/src/HAL/STM32/../../inc/SanityCheck.h:2612:4: error: #error "SENSORLESS_PROBING requires TMC2130, TMC2160, TMC2209, TMC2660, or TMC5160 stepper drivers." 2612 | #error "SENSORLESS_PROBING requires TMC2130, TMC2160, TMC2209, TMC2660, or TMC5160 stepper drivers." | ^~~~~ In file included from Marlin/src/HAL/STM32/../../inc/MarlinConfig.h:49, from Marlin/src/HAL/STM32/HAL_SPI.cpp:25: Marlin/src/HAL/STM32/../../inc/SanityCheck.h:1307:8: error: #error "SENSORLESS_PROBING requires a TMC2130/2160/2209/5130/5160 driver on Z." 1307 | #error "SENSORLESS_PROBING requires a TMC2130/2160/2209/5130/5160 driver on Z." | ^~~~~ In file included from Marlin/src/HAL/STM32/../../inc/MarlinConfig.h:49, from Marlin/src/HAL/STM32/MarlinSerial.cpp:22: Marlin/src/HAL/STM32/../../inc/SanityCheck.h:1307:8: error: #error "SENSORLESS_PROBING requires a TMC2130/2160/2209/5130/5160 driver on Z." 1307 | #error "SENSORLESS_PROBING requires a TMC2130/2160/2209/5130/5160 driver on Z." | ^~~~~ Marlin/src/HAL/STM32/../../inc/SanityCheck.h:2591:6: error: #error "SENSORLESS_PROBING requires a TMC stepper driver with StallGuard on Z." 2591 | #error "SENSORLESS_PROBING requires a TMC stepper driver with StallGuard on Z." | ^~~~~ Marlin/src/HAL/STM32/../../inc/SanityCheck.h:2612:4: error: #error "SENSORLESS_PROBING requires TMC2130, TMC2160, TMC2209, TMC2660, or TMC5160 stepper drivers." 2612 | #error "SENSORLESS_PROBING requires TMC2130, TMC2160, TMC2209, TMC2660, or TMC5160 stepper drivers." | ^~~~~ In file included from Marlin/src/HAL/STM32/../../inc/MarlinConfig.h:49, from Marlin/src/HAL/STM32/Sd2Card_sdio_stm32duino.cpp:23: Marlin/src/HAL/STM32/../../inc/SanityCheck.h:1307:8: error: #error "SENSORLESS_PROBING requires a TMC2130/2160/2209/5130/5160 driver on Z." 1307 | #error "SENSORLESS_PROBING requires a TMC2130/2160/2209/5130/5160 driver on Z." | ^~~~~ Marlin/src/HAL/STM32/../../inc/SanityCheck.h:2591:6: error: #error "SENSORLESS_PROBING requires a TMC stepper driver with StallGuard on Z." 2591 | #error "SENSORLESS_PROBING requires a TMC stepper driver with StallGuard on Z." | ^~~~~ Marlin/src/HAL/STM32/../../inc/SanityCheck.h:2612:4: error: #error "SENSORLESS_PROBING requires TMC2130, TMC2160, TMC2209, TMC2660, or TMC5160 stepper drivers." 2612 | #error "SENSORLESS_PROBING requires TMC2130, TMC2160, TMC2209, TMC2660, or TMC5160 stepper drivers."

`

thisiskeithb commented 4 years ago

TMC2226 is not a valid option. Use TMC2209: https://github.com/MarlinFirmware/Marlin/issues/18328#issuecomment-647557206

This Issue Queue is for Marlin bug reports and development-related issues, and we prefer not to handle user-support questions here. (As noted on this page.) For best results getting help with configuration and troubleshooting, please use the following resources:

After seeking help from the community, if the consensus points to a bug in Marlin, then you should post a bug report.

hackebike commented 4 years ago

So TMC 2226 is not planned to be supported and it should be set as TMC 2209 as before.

thisiskeithb commented 4 years ago

As far as Marlin is concerned, the TMC2226 is just a TMC2209 in a different package. Use TMC2209 or TMC2209_STANDALONE for each *_DRIVER_TYPE.

I'll try to put in a PR later adding an explanation under the TMC section that TMC2225 = TMC2208 and TMC2226 = TMC2209.

github-actions[bot] commented 3 years ago

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.