Add support for SEGGER RTT STDIO

anaoum commented

2022-03-30 18:05:50 -05:00

(Migrated from github.com)

Fixes #774

This PR adds support for using SEGGER RTT as an STDIO driver.

The code for the stdio_rtt implementation is quite simple and copies the structure of the other STDIO drivers. RTT support can be enabled in the same way as the other STDIO drivers, simply add pico_enable_stdio_rtt(${TARGET_NAME} 1) to your CMakeLists.txt and call stdio_init_all() in your initialisation code.

The official SEGGER_RTT code is included in the lib directory (the latest version, V762c). I have copied all files exactly as-is, despite many of the files being unnecessary/code samples/etc. I did this to make updates to newer versions more simple (just overwrite the entire directory with the latest version). Unfortunately SEGGER does not distribute this code as a git repo which could have been included as a submodule.

Happy to make improvements/adjustments to this implementation based on feedback.

Fixes #774 This PR adds support for using SEGGER RTT as an STDIO driver. The code for the stdio_rtt implementation is quite simple and copies the structure of the other STDIO drivers. RTT support can be enabled in the same way as the other STDIO drivers, simply add `pico_enable_stdio_rtt(${TARGET_NAME} 1)` to your CMakeLists.txt and call `stdio_init_all()` in your initialisation code. The official SEGGER_RTT code is included in the lib directory (the latest version, V762c). I have copied all files exactly as-is, despite many of the files being unnecessary/code samples/etc. I did this to make updates to newer versions more simple (just overwrite the entire directory with the latest version). Unfortunately SEGGER does not distribute this code as a git repo which could have been included as a submodule. Happy to make improvements/adjustments to this implementation based on feedback.

👍 5

m-u-xyz commented

2022-05-04 04:33:51 -05:00

(Migrated from github.com)

Why don't you re-package the Segger code as a git repository yourself (or clone (and update?) https://github.com/PromyLOPh/rtt)?

Did you ask them to please publish their code to github?

Why don't you re-package the Segger code as a git repository yourself (or clone (and update?) https://github.com/PromyLOPh/rtt)? Did you ask them to please publish their code to github?

anaoum commented

2022-05-04 16:38:03 -05:00

(Migrated from github.com)

I find the problem with third parties hosting a clone of the official code is that they become out of date very quickly. To be fair, this problem would be the same by keeping the code copy in the pico-sdk, so I'm not sure what the best solution is...

The version you linked is 5 years old. I found another repo that seems to do a better job of keeping the code up to date: https://github.com/adfernandes/segger-rtt. I am happy to update my PR to use this third-party repo, but I would also like a maintainer of this project to chime in and advise what approach would most likely get merged.

I emailed SEGGER this morning to see if they would consider publicly hosting the code on a git repo.

I find the problem with third parties hosting a clone of the official code is that they become out of date very quickly. To be fair, this problem would be the same by keeping the code copy in the pico-sdk, so I'm not sure what the best solution is... The version you linked is 5 years old. I found another repo that seems to do a better job of keeping the code up to date: https://github.com/adfernandes/segger-rtt. I am happy to update my PR to use this third-party repo, but I would also like a maintainer of this project to chime in and advise what approach would most likely get merged. I emailed SEGGER this morning to see if they would consider publicly hosting the code on a git repo.

👍 1

PhilippHaefele commented

2022-06-24 15:20:03 -05:00

(Migrated from github.com)

I was just searching for something else, but i think they already have the RTT source code on GitHub. Didn't checked if the version is the newest one (they always change the header on a release, but code often doesn't have any changes)

https://github.com/SEGGERMicro/RTT

Hope i could help a little bit

I was just searching for something else, but i think they already have the RTT source code on GitHub. Didn't checked if the version is the newest one (they always change the header on a release, but code often doesn't have any changes) https://github.com/SEGGERMicro/RTT Hope i could help a little bit

adfernandes commented

2022-07-27 20:26:07 -05:00

(Migrated from github.com)

FYSA I keep the unmodified RTT sources maintained and properly tagged here: https://github.com/adfernandes/segger-rtt (been doing it for a while)

I'm reasonably certain that Segger doesn't "maintain" any release of RTT outside of the JLink releases. Other projects that Segger does follow, like the Zephyr RTOS from the Linux Foundation, use relatively old libraries, too.

The attitude, much like the SEGGERMicro repository linked above, is to do a code dump "once in a while", ignoring bugfix and patches.

So about once a month (waves hands in the air) I download the latest Segger JLink software distribution and upload to my repository.

FYSA I keep the unmodified RTT sources maintained and properly tagged here: https://github.com/adfernandes/segger-rtt (been doing it for a while) I'm reasonably certain that Segger doesn't "maintain" any release of RTT outside of the JLink releases. Other projects that Segger _does_ follow, like the Zephyr RTOS from the Linux Foundation, use relatively old libraries, too. The attitude, much like the `SEGGERMicro` repository linked above, is to do a code dump "once in a while", ignoring bugfix and patches. So about once a month _(waves hands in the air)_ I download the latest Segger JLink software distribution and upload to my repository.

👍 1

rgrr commented

2023-03-11 04:53:14 -06:00

(Migrated from github.com)

This PR has been added to the 1.6.0 milestone. Anything speaking against merging this into develop in the not too far future?

👀 1

adfernandes commented

2023-04-16 12:19:15 -05:00

(Migrated from github.com)

FWIW, I just pushed Version V7.86h (2023-04-12) to my unmodified, pristine mirror of Seggger's RTT distribution.

Segger's release notes are usually pretty complete, and there were some minor changes to to RTT backend in recent history.

@anaoum I would ask (please!) that the most recent-possible RTT sources, like mine from above, be merged into the Pico SDK since this can make a difference as the debuggers and protocol (slowly) evolve!

FWIW, I just pushed **Version V7.86h (2023-04-12)** to my [_unmodified_, pristine mirror of Seggger's RTT distribution](https://github.com/adfernandes/segger-rtt). Segger's [release notes](https://www.segger.com/downloads/jlink/ReleaseNotes_JLink.html) are usually pretty complete, and there **were** some minor changes to to RTT backend in recent history. @anaoum I would ask (please!) that the most recent-possible RTT sources, like mine from above, be merged into the Pico SDK since this _can_ make a difference as the debuggers and protocol (slowly) evolve!

👍 2

kilograham commented

2023-05-26 09:06:50 -05:00

(Migrated from github.com)

This PR has been added to the 1.6.0 milestone. Anything speaking against merging this into develop in the not too far future?

I think we would want to avoid copying a bunch of Segger code into this repo... so would prefer to build with code from a submodule and have that submodule be optional

> This PR has been added to the 1.6.0 milestone. Anything speaking against merging this into develop in the not too far future? I think we would want to avoid copying a bunch of Segger code into this repo... so would prefer to build with code from a submodule and have that submodule be optional

👍 2

adfernandes commented

2023-05-30 22:46:11 -05:00

(Migrated from github.com)

@kilograham, I agree with your "use a submodule" sentiment 100%

The issue is that there is no official source, as far as I know, RTT repository - that's why I maintain this, which is very carefully exactly what Segger distributes.

@anaoum, if you agree, I'd be happy to rework your PR to use a submodule...

@kilograham, I agree with your "use a submodule" sentiment 100% The issue is that there is no **official** source, as far as I know, RTT repository - that's why I maintain [this](https://github.com/adfernandes/segger-rtt), which is very carefully exactly what Segger distributes. @anaoum, if you agree, I'd be happy to rework your PR to use a submodule...

anaoum commented

2023-05-31 15:08:18 -05:00

(Migrated from github.com)

@anaoum, if you agree, I'd be happy to rework your PR to use a submodule...

no objection from me. I would have done it myself but am quite time poor at the moment.

> @anaoum, if you agree, I'd be happy to rework your PR to use a submodule... no objection from me. I would have done it myself but am quite time poor at the moment.

adfernandes commented

2023-06-04 00:17:28 -05:00

(Migrated from github.com)

(ping @kilograham and @anaoum)

Superseded by #1411

(ping @kilograham and @anaoum) **Superseded** by #1411

adfernandes commented

2023-06-07 00:37:42 -05:00

(Migrated from github.com)

@kilograham, I have vendored in RTT and removed the submodule.

However, no matter what I do, I cannot apply the compiler flags.

set_source_files_properties(SEGGER/RTT/SEGGER_RTT.c
    PROPERTIES COMPILE_OPTIONS "-Wno-cast-qual;-Wno-cast-align")

You'll see that I have

include(CMakePrintHelpers)

cmake_print_properties(
    SOURCES SEGGER/RTT/SEGGER_RTT.c
    PROPERTIES COMPILE_OPTIONS
)

and cmake does appear to apply those properties, but ... they're not in the actual compile commands, and the build fails.

Any idea what might be wrong?

@kilograham, I have vendored in `RTT` and removed the submodule. **However**, no matter what I do, I _cannot_ apply the compiler flags. ``` set_source_files_properties(SEGGER/RTT/SEGGER_RTT.c PROPERTIES COMPILE_OPTIONS "-Wno-cast-qual;-Wno-cast-align") ``` You'll see that I have ``` include(CMakePrintHelpers) cmake_print_properties( SOURCES SEGGER/RTT/SEGGER_RTT.c PROPERTIES COMPILE_OPTIONS ) ``` and `cmake` **does** appear to apply those properties, but ... they're not in the _actual_ compile commands, and the build fails. Any idea what might be wrong?

peterharperuk commented

2023-06-07 02:32:42 -05:00

(Migrated from github.com)

You'll have to specify the full path to the source file for set_source_files_properties to take effect.

peterharperuk commented

2023-06-07 03:44:36 -05:00

(Migrated from github.com)

I pushed my suggested changes that seem to fix the issue - hope that was ok

rgrr commented

2023-06-07 14:09:22 -05:00

(Migrated from github.com)

question: why don't you put the SysView files into the SEGGER subdirectory. Those are part of the more or less generic RTT support and would allow a user to easyly integrate SysView support.

kilograham commented

2023-06-07 14:21:10 -05:00

(Migrated from github.com)

closed in favor of #1411

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Rows
Columns

Add support for SEGGER RTT STDIO #775

Pull request closed