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

Port demo to Teensy 3.6 #222

Merged
merged 4 commits into from
Jun 6, 2020
Merged

Conversation

stapelberg
Copy link
Contributor

@stapelberg stapelberg commented Jun 3, 2020

Getting the mcuconf and Makefile changes right is non-trivial for a beginner, so
this will be helpful for others who want to get started with ChibiOS on a Teensy 3.6.

I have split out the mechanical copying of the old demo into one commit, and the required changes for working on the Teensy 3.6 into the other commit, so I recommend looking only at one at a time for a better picture of what is going on :)

cc @walkerstop — thanks for your work on the Teensy 3.6 support in ChibiOS! Maybe you could take a quick look at this PR, too. Perhaps you have an idea about tickless mode not working? See the TODO in chconf.h:

// TODO: tickless mode is currently broken: e.g. chVTGetSystemTimeX() will
// always return 0, making all timers hang.

Edit: this is now ready to review.

Getting the mcuconf and Makefile changes right is non-trivial for a beginner, so
this will be helpful for others who want to get started with ChibiOS on a Teensy 3.6.
@stapelberg stapelberg force-pushed the demo_teensy36 branch 2 times, most recently from 449731b to ab9e587 Compare June 3, 2020 19:44
@stapelberg
Copy link
Contributor Author

Okay, after including the HSRUN fix (largely just including PJRC’s code), the firmware works on my Teensy 3.6 and jenkins passes :)

This is now ready to review.

@fpoussin fpoussin merged commit 7fefab1 into ChibiOS:chibios-20.3.x Jun 6, 2020
@fpoussin
Copy link
Member

fpoussin commented Jun 6, 2020

All good, thank you

@stapelberg stapelberg deleted the demo_teensy36 branch June 6, 2020 12:12
@stapelberg
Copy link
Contributor Author

Awesome, thanks for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants