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

🎨 Adjust TMC SPI pins. Misc. fixes and cleanup #25558

Merged
merged 6 commits into from
Mar 24, 2023

Conversation

thinkyhead
Copy link
Member

@thinkyhead thinkyhead commented Mar 24, 2023

Various straightforward changes gathered from #24911 by @quiret. See individual commits for the types of changes. Highlights include:

  • Rename TMC_SW_* pins to TMC_SPI_* as they may apply to software or hardware SPI.
  • For AVR automatically set TMC_USE_SW_SPI based on the pins.
  • Apply helpful comments with links to pins headers and FastIO.
  • Wrap font includes to skip extraneous font data.
  • Rename LPC_SOFTWARE_SPI to SOFTWARE_SPI.
  • Update to atmelavr@~4.0.1 and -std=gnu++1z for AVR build. (And misc. INI cleanup.)

@thinkyhead thinkyhead added PR: General Cleanup T: HAL & APIs Topic related to the HAL and internal APIs. C: Build / Toolchain labels Mar 24, 2023
@thinkyhead thinkyhead force-pushed the bf2_some_24911 branch 3 times, most recently from 339abb0 to f6934a2 Compare March 24, 2023 08:49
thinkyhead and others added 6 commits March 24, 2023 17:01
Co-Authored-By: Martin Turski <turningtides@outlook.de>
Co-Authored-By: Martin Turski <turningtides@outlook.de>
Co-Authored-By: Martin Turski <turningtides@outlook.de>
Co-Authored-By: Martin Turski <turningtides@outlook.de>
Co-Authored-By: Martin Turski <turningtides@outlook.de>
Co-Authored-By: Martin Turski <turningtides@outlook.de>
@thinkyhead thinkyhead merged commit ec91d5a into MarlinFirmware:bugfix-2.1.x Mar 24, 2023
@thinkyhead thinkyhead deleted the bf2_some_24911 branch March 24, 2023 23:25
thinkyhead added a commit to MarlinFirmware/Configurations that referenced this pull request Mar 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: Build / Toolchain PR: General Cleanup T: HAL & APIs Topic related to the HAL and internal APIs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant