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

feat: Add BTT MMB v1.1 config #616

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vattern
Copy link

@vattern vattern commented Jun 27, 2024

BTT MMB v1.0 and v1.1 has PA8 and PB8 swapped.

Copy link

📌 This pull request has been marked as stale because it has not had activity in the past 30 days.
Please update the PR or comment to keep it active. Otherwise, this will be closed in 14 days.
We appreciate your contribution!

2 similar comments
Copy link

📌 This pull request has been marked as stale because it has not had activity in the past 30 days.
Please update the PR or comment to keep it active. Otherwise, this will be closed in 14 days.
We appreciate your contribution!

Copy link

📌 This pull request has been marked as stale because it has not had activity in the past 30 days.
Please update the PR or comment to keep it active. Otherwise, this will be closed in 14 days.
We appreciate your contribution!

Copy link
Owner

@Frix-x Frix-x left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello, thanks for the PR!

But there's some changes needed in the user template to be able to merge it into Klippain.

Comment on lines +1 to +23
[board_pins mmu_manufacturer]
mcu: mmu
aliases:
MCU_M1_STEP=PB15 , MCU_M1_DIR=PB14 , MCU_M1_EN=PB8 , MCU_M1_UART=PA10 , # MCU_M1_DIAG=PA3 ,
MCU_M2_STEP=PD2 , MCU_M2_DIR=PB13 , MCU_M2_EN=PD1 , MCU_M2_UART=PC7 , # MCU_M2_DIAG=PA4 ,
MCU_M3_STEP=PD0 , MCU_M3_DIR=PD3 , MCU_M3_EN=PA15 , MCU_M3_UART=PC6 , MCU_M3_DIAG=PB9 ,
MCU_M4_STEP=PB6 , MCU_M4_DIR=PB7 , MCU_M4_EN=PB5 , MCU_M4_UART=PA9 , # MCU_M4_DIAG=PA8 ,

MCU_MISO=PA6 , MCU_SCK=PA5 , MCU_MOSI=PA7 ,

MCU_STP1=PA3 , MCU_STP2=PA4 , MCU_STP3=PB9 , # Shared with Steppers Diag pins 1-2-4 so it's better to use STP 4-11 for others servos/sensors
MCU_STP4=PA8 , MCU_STP5=PC15 , MCU_STP6=PC13 , MCU_STP7=PC14 , MCU_STP8=PB12 , MCU_STP9=PB11 , MCU_STP10=PB10 , MCU_STP11=PB2 ,


MCU_MOT=PA0 ,
MCU_SENSOR=PA1 ,
MCU_RGB=PA2 ,

## I2C header
MCU_I2C_SDA=PB4 , MCU_I2C_SCL=PB3 ,

## SWD header
MCU_SWD_PA14=PA14 , MCU_SWD_PA13=PA13 ,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file need to be modified to use the pins aliases used in Klippain.

Look at this file for documentation: https://github.com/Frix-x/klippain/blob/main/docs/pinout.md
And as an example, you can look at the MMB v1.0 user template here: https://github.com/Frix-x/klippain/blob/main/user_templates/mcu_defaults/mmu/BTT_MMB_CAN_v1.0.cfg

@Frix-x Frix-x mentioned this pull request Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants