Skip to content

ESP-IDF Compatibility - Redo PR 194#317

Closed
thinkyhead wants to merge 27 commits intoteemuatlut:masterfrom
MarlinFirmware:redo_ESP_IDF_compat_pr_194
Closed

ESP-IDF Compatibility - Redo PR 194#317
thinkyhead wants to merge 27 commits intoteemuatlut:masterfrom
MarlinFirmware:redo_ESP_IDF_compat_pr_194

Conversation

@thinkyhead
Copy link
Contributor

In case this is still useful to someone, it is rebased on the fork we are maintaining for quicker turnaround. It might need some additional work to also support the new TMC2240. Evaluate and have fun playing with TMC and ESP-IDF together!

Based on PR #194 by @holgerpieta.


Generally I prefer the original ESP-IDF instead of the Arduino framework, because it seems to me less magical and more predictable. So I tried to make this library compile in ESP-IDF without Arduino. Took much less work than I feared, which I guess shows again that this library is very well made. Thanks to everybody involved.

What I did not implement in ESP-IDF is the SPI interface, because I do not have any experience in that field and at first glance I wasn't able to match the Arduino SPI interface and the ESP-IDF API. Maybe someone with more experience can have a look at that. But then I'm not actually sure the hardware SPI is actually required, because the SW solution exists. To make sure nobody uses it accidentally (and to make it compile) I conditionally removed the affected constructors when ESP-IDF platform is configured.

I was able to test it on my ESP32 board with a TCM2208 stepper, but I neither have Arduino in place to try it nor any other stepper driver. So I would probably a good idea if someone with Arduino and other stepper drivers would test it before merging it upstream.

thinkyhead and others added 27 commits November 25, 2024 20:48
Co-Authored-By: David Buezas <777196+dbuezas@users.noreply.github.com>
Co-Authored-By: z1996xm <102506464+z1996xm@users.noreply.github.com>
Co-authored-by: Scott Lahteine <thinkyhead@users.noreply.github.com>
Co-Authored-By: KonstantinKalinskiy <114812187+konstantinkalinskiy@users.noreply.github.com>
Co-authored-by: Scott Lahteine <thinkyhead@users.noreply.github.com>
Co-Authored-By: holgerpieta <38852807+holgerpieta@users.noreply.github.com>
@thinkyhead thinkyhead closed this Oct 29, 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.

6 participants