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 TMC5160 driver to X/Y #289

Merged
merged 3 commits into from
Sep 10, 2023

Conversation

pvpacheco
Copy link
Contributor

Adds TMC5160 driver option to X and Y axis configuration option. Those are used to run steppers at 48v.

@Frix-x
Copy link
Owner

Frix-x commented Sep 3, 2023

Hi thanks for this PR! I planned to add support for TMC5160 in the next release, so I'm pretty happy with this :)

Just to understand and find the best way to integrate it, can you tell me which MCU you are using?
Indeed, the SPI bus used for all the currently supported MCUs is the software SPI. Here is an example for the TMC2240 config files:

[tmc2240 stepper_x]
cs_pin: X_TMCUART
spi_software_sclk_pin: DRIVER_SPI_SCK
spi_software_mosi_pin: DRIVER_SPI_MOSI
spi_software_miso_pin: DRIVER_SPI_MISO

So X_TMCCS and Y_TMCCS are currently not defined in any of the MCU files

@Frix-x Frix-x self-assigned this Sep 3, 2023
@Frix-x Frix-x added the enhancement New feature or request label Sep 3, 2023
@pvpacheco pvpacheco force-pushed the features/add-tmc5160-driver branch from 78133ec to 66b8bc5 Compare September 4, 2023 00:49
@pvpacheco
Copy link
Contributor Author

pvpacheco commented Sep 4, 2023

Hi, thank you for sharing klippain!

I'm using an Octopus Max EZ. Just made the change to use software SPI and UART param. Its already working on my setup.

What do you think?

@Frix-x
Copy link
Owner

Frix-x commented Sep 10, 2023

Thanks for this. I think it's ok for me, let's merge it :)

@Frix-x Frix-x changed the base branch from main to develop September 10, 2023 15:01
@Frix-x Frix-x merged commit b24e44a into Frix-x:develop Sep 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants