Klipper3d / klipper

Klipper is a 3d-printer firmware
GNU General Public License v3.0
9.41k stars 5.3k forks source link

STM32F407 support? #1733

Closed towe96 closed 4 years ago

towe96 commented 5 years ago

Biqu has just released the SKR Pro: https://www.aliexpress.com/item/33042699158.html

It appears to be very potent board, with expansive expandability, 6 stepper sockets, and an STM32F407 MCU. I think it's fair to say it's the single best board ever released as of this date.

Is support for the SKR Pro's MCU planned?

Nelanius commented 4 years ago

Hi @Spacemarine2018, I am having some issue getting my Skr pro with tmc5160s working. Currently my steppers only move in one direction and my steppers are wired AABB. Could you post your config so i could compare it to mine?

ghost commented 4 years ago

Here you go @Nelanius printer.cfg.txt I don't have the sensorless homing working, I just connected the physical switches, so the config looks misleading in that section.

ghost commented 4 years ago

Can anybody with a working TMC5160 V1.2 from BTT and an SKR Pro post the printer.cfg? I'm having the problem that I can write via SPI, but not read and I want to make sure that it's not a configuation error.

Also please check if you get meaningful values (not only 00000 or fffff) when doing DUMP_TMC STEPPER=stepper_x just to make sure you can actually read and not only write SPI.

MrKekson commented 4 years ago

Only for 2130, should be similar i guess, do you need it?

On Tue, 5 Nov 2019 at 13:41, Spacemarine2018 notifications@github.com wrote:

Can anybody with a working TMC5160 V1.2 from BTT and an SKR Pro post the printer.cfg? I'm having the problem that I can write via SPI, but not read and I want to make sure that it's not a configuation error.

Also please check if you get meaningful values (not only 00000 or fffff) when doing DUMP_TMC STEPPER=stepper_x just to make sure you can actually read and not only write SPI.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/KevinOConnor/klipper/issues/1733?email_source=notifications&email_token=AAWUDIWKHUND7L6I6D7W7NDQSFSYXA5CNFSM4HYPLMZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDCWE6I#issuecomment-549806713, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAWUDIT2VCPXUWID3SM27HDQSFSYXANCNFSM4HYPLMZQ .

-- Oli

tiaanv commented 4 years ago

@Spacemarine2018. Why not run them in UART mode? I ran my 2209s in UART mode. I'm pretty sure the technical bits on the 5160s are good ng to be the same. Only. Small thing that's incorrect on the specs for config. TX and RX are on the same pin... So in klipper you need to delete or comment out the TX pin line in the driver. Onfig sections

ghost commented 4 years ago

@MrKekson does it differ from the config example here? https://github.com/KevinOConnor/klipper/blob/master/config/generic-bigtreetech-skr-pro.cfg If it's basically the same, I don't need it, but thanks!

@tiaanv interesting idea, I never thought of using UART because everyone seems to use SPI. Any benefit/drawback since the TMC5160 supports both?

MrKekson commented 4 years ago

You are right, this file was not available when i've configured my machine, but that should be ok too!

On Tue, 5 Nov 2019 at 15:41, Spacemarine2018 notifications@github.com wrote:

@MrKekson https://github.com/MrKekson does it differ from the config example here? https://github.com/KevinOConnor/klipper/blob/master/config/generic-bigtreetech-skr-pro.cfg If it's basically the same, I don't need it, but thanks!

@tiaanv https://github.com/tiaanv interesting idea, I never thought of using UART because everyone seems to use SPI. Any benefit/drawback since the TMC5160 supports both?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/KevinOConnor/klipper/issues/1733?email_source=notifications&email_token=AAWUDIRCZ6QLBFF2RJFP6H3QSGAYRA5CNFSM4HYPLMZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDC5RZI#issuecomment-549837029, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAWUDISE67MHVZ7TBYXYV2TQSGAYRANCNFSM4HYPLMZQ .

-- Oli

KevinOConnor commented 4 years ago

I'm going to close this issue as I think the original issue (stm32f4 support) is now complete.

-Kevin