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

rp2040: Add spi0_gpio4_gpio3_gpio2 bus to support fysetc PITB V2 (#6683) #452

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

rogerlz
Copy link
Contributor

@rogerlz rogerlz commented Dec 3, 2024

The Fysetc PITB V2 board uses a spi bus config that is supported by the RP2040 chip, but not klipper, so this adds the relevant config to the file to allow you to run the tmc5160's on the board via hardware SPI. This resolves the issue of software spi not working on this board, which I was unable to fully understand.

I have also seen other users encounter similar bus config issues with the rp2040 setting up things like accelerometers and such with this pin layout.

As requested, this also uses the new convention for spi bus naming, while maintaining the old bus names for compatibility.

Klipper PR: Klipper3d/klipper#6683

@rogerlz rogerlz requested a review from a team as a code owner December 3, 2024 23:01
The Fysetc PITB V2 board uses a spi bus config that is supported by the
RP2040 chip, but not klipper, so this adds the relevant config to the file
to allow you to run the tmc5160's on the board via hardware SPI.  This
resolves the issue of software spi not working on this board, which I
was unable to fully understand.

I have also seen other users encounter similar bus config issues with
the rp2040 setting up things like accelerometers and such with this
pin layout.

As requested, this also uses the new convention for spi bus naming, while
maintaining the old bus names for compatibility.

Signed-off-by: Jessica Hunt <[email protected]>
@rogerlz rogerlz merged commit dd059dc into main Dec 5, 2024
2 checks passed
@rogerlz rogerlz deleted the mainline-pr-6683 branch December 5, 2024 19:54
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.

1 participant