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.

Sign up for GitHub

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

Jump to bottom

Add support for Real Time Transfer (RTT) via OpenOCD #610

Merged
merged 4 commits into from
Apr 12, 2021
Merged

Add support for Real Time Transfer (RTT) via OpenOCD #610

merged 4 commits into from
Apr 12, 2021

Conversation

salkinium
Copy link
Member

@salkinium salkinium commented Apr 9, 2021 •
edited
Loading

OpenOCD v0.11 added basic support for RTT, which is a great non-blocking alternative to semihosting and works on any Cortex-M target without requiring any peripheral UART drivers.

This adds a basic implementation based on the OpenOCD implementation:

  • RTT control structure, channels and buffers implemented
  • µC -> OpenOCD tested in hardware
  • OpenOCD -> µC tested in hardware
  • Proper channel buffer config validation
  • Build system Integration: Better Telnet client
    • SCons
    • Makefile
    • CMake
  • Documentation

cc @rleh @chris-durand

@salkinium salkinium force-pushed the feature/rtt branch 3 times, most recently from 2298de1 to cbff4ab Compare April 10, 2021 07:52
@salkinium salkinium marked this pull request as ready for review April 10, 2021 08:01
chris-durand
chris-durand reviewed Apr 10, 2021
View reviewed changes
Copy link
Member

@chris-durand chris-durand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice!

src/modm/platform/uart/rtt/rtt.cpp.in Outdated Show resolved Hide resolved
src/modm/platform/uart/rtt/rtt.cpp.in Outdated Show resolved Hide resolved
@salkinium salkinium force-pushed the feature/rtt branch 3 times, most recently from 355ca7e to cb7c4db Compare April 11, 2021 13:56
@salkinium salkinium added the ci:hal Triggers the exhaustive HAL compile CI jobs label Apr 12, 2021
@salkinium salkinium added ci:hal Triggers the exhaustive HAL compile CI jobs and removed ci:hal Triggers the exhaustive HAL compile CI jobs labels Apr 12, 2021
@salkinium salkinium merged commit 6f178d5 into modm-io:develop Apr 12, 2021
@salkinium salkinium added this to the 2021q2 milestone Jun 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chris-durand chris-durand chris-durand approved these changes

@rleh rleh rleh approved these changes

Assignees
No one assigned
Labels
advanced 🤯 ci:hal Triggers the exhaustive HAL compile CI jobs feature 🚧 toolchain ⚙️
Milestone
2021q2
Development

Successfully merging this pull request may close these issues.
3 participants
@salkinium @rleh @chris-durand