Skip to content

drivers/segger: Add a kconfig to override Segger SystemView target sources version#16753

Merged
xiaoxiang781216 merged 1 commit into
apache:masterfrom
simbit18:simbit18-segger
Jul 22, 2025
Merged

drivers/segger: Add a kconfig to override Segger SystemView target sources version#16753
xiaoxiang781216 merged 1 commit into
apache:masterfrom
simbit18:simbit18-segger

Conversation

@simbit18

Copy link
Copy Markdown
Contributor

Summary

update:
CMakeLists.txt
Make.defs
Kconfig

It allows the same version of SystemView target sources to be used for CMake and Make, to avoid discrepancies in the future.

#16620 (comment)

Impact

Impact on user: NO

Impact on build: NO

Impact on hardware: NO

Impact on documentation: NO

Impact on security: NO

Impact on compatibility: NO

Testing

CI

====================================================================================
Configuration/Tool: sim/segger
2025-07-22 10:32:20
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
  [1/1] Normalize sim/segger
====================================================================================

https://github.com/simbit18/nuttx_test_pr/actions/runs/16441393116/job/46463008660#logs

…urces version

update:
    CMakeLists.txt
    Make.defs
    Kconfig

It allows the same version of SystemView target sources to be used for CMake and Make, to avoid discrepancies in the future.

Signed-off-by: simbit18 <simbit18@gmail.com>
@github-actions github-actions Bot added Area: Drivers Drivers issues Size: S The size of the change in this PR is small labels Jul 22, 2025
@xiaoxiang781216 xiaoxiang781216 merged commit d7a05df into apache:master Jul 22, 2025
39 checks passed
@simbit18 simbit18 deleted the simbit18-segger branch July 22, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Drivers Drivers issues Size: S The size of the change in this PR is small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants