MarlinFirmware / Marlin

Marlin is an optimized firmware for RepRap 3D printers based on the Arduino platform. Many commercial 3D printers come with Marlin installed. Check with your vendor if you need source code for your specific machine.
https://marlinfw.org
GNU General Public License v3.0
16.22k stars 19.22k forks source link

[FR] BigTreeTech SKR Pro v1.1 #14484

Closed boubouexpress closed 4 years ago

boubouexpress commented 5 years ago

hello

request to add the BIGTREETECH SKR PRO V1.1 ARM cortex M4 32 bit STM32F407ZGT6 MCU

https://github.com/bigtreetech/BIGTREETECH-SKR-PRO-V1.1

merci

govido commented 5 years ago

I guess Bigtreetech will submit the changes needed as they advertise marlin 2.0 compatibility. Yesterday the new E3 mini got merged from them, so I am looking forward to it.

AnHardt commented 5 years ago

they advertise marlin 2.0 compatibility.

That doesn't mean anything to me. Doesn't the word advertisement say all you need to know? I remember a well know Kickstarter announcing a new Marlin compatible probe. No other single hardware item caused that deep changes in Marlins structure to make that "compatible" crap finally work. And worst - they continue to change their product, without a warning, to make it incompatible again.

Bigtreetechs history gives me some more hope. Let's hope for the best.

govido commented 5 years ago

https://github.com/bigtreetech/BIGTREETECH-SKR-PRO-V1.1/tree/master/firmware ;) It is ready

eightheads commented 5 years ago

https://github.com/bigtreetech/BIGTREETECH-SKR-PRO-V1.1/tree/master/firmware ;) It is ready

Except they do not compile currently

thisiskeithb commented 5 years ago

There’s an SKR Plus pins file in there too.

Wonder when BigTreeTech will announce that board.

thisiskeithb commented 5 years ago

See https://github.com/MarlinFirmware/Marlin/pull/14510, but it still doesn’t compile.

Msq001 commented 5 years ago

14510 this PR was closed because of a conflict files. I will creat a new PR for this board, and https://github.com/bigtreetech/BIGTREETECH-SKR-PRO-V1.1/tree/master/firmware this branch is work now, and Pay attention to the Readme.md

govido commented 5 years ago

As this board is completely new I didn't want to open a new issue directly. The things I noticed so far (my board arrived yesterday):

not marlin related:

If I should test something let me know. I have tmc2130, tmc2209 to play with, tft35, fysetc mini v2.1, bltouch

Msq001 commented 5 years ago

https://github.com/MarlinFirmware/Marlin/pull/14543

  1. this PR will support TMC2208/2209 uart mode
  2. I have tested REPRAP_DISCOUNT_FULL_GRAPHIC_SMART_CONTROLLER(LCD12864) and REPRAP_DISCOUNT_SMART_CONTROLLER(LCD2004), They all work properly.
  3. stm32 have not Serial port 0, it begin from Serial port 1
  4. I have tested bltouch in PA1 pins, it‘s ok.

I'll test your feedback one by one. Thank you very much!

govido commented 5 years ago

@Msq001 Nice, I will take a look again at the terminal output while compiling. I got errors regarding display and bltouch, or they are coming from somewhere else... If stm32 has no serial 0 there should be a comment added in configuration.h ;)

govido commented 5 years ago

@Msq001 I tested the bltouch and fysetc options again. I must have done something wrong in my first attempt, bltouch has no errors. But the display throws this error right at the final compile stage...

Compiling .pioenvs\BIGTREE_SKR_PRO\src\src\sd\usb_flashdrive\lib\message.cpp.o
Compiling .pioenvs\BIGTREE_SKR_PRO\src\src\sd\usb_flashdrive\lib\parsetools.cpp.o
Compiling .pioenvs\BIGTREE_SKR_PRO\src\src\sd\usb_flashdrive\usb_host.cpp.o
Linking .pioenvs\BIGTREE_SKR_PRO\firmware.elf
.pioenvs/BIGTREE_SKR_PRO/src/src/lcd/dogm/u8g_dev_uc1701_mini12864_HAL.cpp.o:(.data.u8g_dev_uc1701_mini12864_HAL_2x_sw_spi+0x8): undefined reference to `u8g_com_HAL_AVR_sw_sp_fn(_u8g_t*, unsigned char, unsigned char, void*)'
collect2.exe: error: ld returned 1 exit status
*** [.pioenvs\BIGTREE_SKR_PRO\firmware.elf] Error 1 
JPRauwolf commented 5 years ago

With the new new version over at https://github.com/bigtreetech/BIGTREETECH-SKR-PRO-V1.1 they seem to got the tmc2208/2209 over uart working, at least it does compile. My RepRap display stopped working though so there is probably more to do. Edit: broken link

gerleimarci commented 5 years ago

Please update the Neopixel library because Fysetc MINI12864 useless without it.

yellobello commented 5 years ago

Is it currently a better idea to use bigtreetech´s sources for the SKR Pro or use the current Marlin 2.0.x-bugfix Branch? maybe dev-2.1.x? i.e. have the sources from Bigtreetech been merged already?

thinkyhead commented 5 years ago

Please update the Neopixel library because Fysetc MINI12864 useless without it.

That project is at https://github.com/adafruit/Adafruit_NeoPixel

thinkyhead commented 5 years ago

i.e. have the sources from Bigtreetech been merged already?

They periodically push updates here.

Evg33 commented 5 years ago

SKR Pro and Neopixel demo: https://youtu.be/uGW9yKVrxyU

https://github.com/adafruit/Adafruit_NeoPixel/pull/206#issuecomment-531297764

Evg33 commented 5 years ago

Now working with FYSETC_MINI_12864_2_1

image image image

Evg33 commented 5 years ago

fysetc 12864 mini merged https://github.com/MarlinFirmware/Marlin/commit/61e3f119f4791aa22af905e25c86f4a2a3753bc9

Evg33 commented 5 years ago

Neopixel merged https://github.com/MarlinFirmware/Marlin/commit/12e0581f249d7aafb394a6b9ca511a9b5f337cba

Romano26 commented 4 years ago

Now working with FYSETC_MINI_12864_2_1

image image image

Hello you could send your files because I have the same configuration and I can not compile and I can not find the problem; thank you

LinoBarreca commented 4 years ago

@boelle close this. it's already done.

boelle commented 4 years ago

done and thanks for the headsup

boelle commented 4 years ago

witch such a big list one man is almost not enough :-)

github-actions[bot] commented 4 years ago

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.