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

remove erroneous z2 define from pins_BTT_MANTA_M8P_V2_0 #27748

Conversation

ellensp
Copy link
Contributor

@ellensp ellensp commented Mar 17, 2025

Description

pins_BTT_MANTA_M8P_V2_0.h is a bit broken

It has a #define Z2_SERIAL_TX_PIN PB5 so Z2_SERIAL_TX_PIN is always on Motor4
But there are no other Z2 pins. So Z2 moves to first available unused E

This also breaks driver uarts for E0,E1 E2 and E3

This is what is defined

  #define X_SERIAL_TX_PIN                   PC13      // M1
  #define Y_SERIAL_TX_PIN                   PE3       // M2
  #define Z_SERIAL_TX_PIN                   PB9       // M3
  #define Z2_SERIAL_TX_PIN                  PB5       // M4
  #define E0_SERIAL_TX_PIN                  PG14      // M5
  #define E1_SERIAL_TX_PIN                  PG10      // M6
  #define E2_SERIAL_TX_PIN                  PD5       // M7
  #define E3_SERIAL_TX_PIN                  PC6       // M8

But this is what is needed

  #define X_SERIAL_TX_PIN                   PC13      // M1
  #define Y_SERIAL_TX_PIN                   PE3       // M2
  #define Z_SERIAL_TX_PIN                   PB9       // M3
  #define E0_SERIAL_TX_PIN                  PB5       // M4
  #define E1_SERIAL_TX_PIN                  PG14      // M5
  #define E2_SERIAL_TX_PIN                  PG10      // M6
  #define E3_SERIAL_TX_PIN                  PD5       // M7
  #define E4_SERIAL_TX_PIN                  PC6       // M8

Requirements

BTT_MANTA_M8P_V2_0 using stepper driver with UART

Benefits

Correct UART is used

Related Issues

  • Z2 stepper motor is only in sync with Z stepper motor when using 16 microsteps #27746
  • @thinkyhead thinkyhead force-pushed the update-BTT_MANTA_M8P_V2_0-tmc-uarts branch from 93d749c to 3b546ae Compare March 19, 2025 20:31
    @thinkyhead thinkyhead merged commit 58749a9 into MarlinFirmware:bugfix-2.1.x Mar 19, 2025
    65 checks passed
    @ellensp ellensp deleted the update-BTT_MANTA_M8P_V2_0-tmc-uarts branch March 20, 2025 00:04
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    Z2 stepper motor is only in sync with Z stepper motor when using 16 microsteps
    2 participants