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

Update to SystemView v3.58 #22

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

brycewilkins
Copy link

@brycewilkins brycewilkins commented Nov 29, 2024

This repository has suffered a few setbacks over past years with Zephyr related patches being lost (causing things to break for RTT/SystemView/Zephyr), and it has been reported as hard to review PRs due to Windows style line endings. There is also no instruction on how to update from the existing Segger source code to a newer release, leading to partial updates that make subsequent updates more difficult and error prone.

This PR attempts to improve things by:

  1. Changing all line endings to Unix style LF (\n)
  2. Updating (completely) to SystemView v3.58 Target sources
  3. Adding a README that identifies Segger "upstream" and suggests a procedure for updating to a new Segger release

To do:

Convert all files in repository to Unix style LF line endings (`\n`).
Trailing whitespace is also trimmed.

No functional changes.

Signed-off-by: Bryce Wilkins <[email protected]>
Integrate Zephyr changes with Segger SystemView target source v3.58

Signed-off-by: Bryce Wilkins <[email protected]>
Add README.md that identifies the origin of Segger source files and
provide a procedure for migrating Zephyr changes to new Segger releases.

Signed-off-by: Bryce Wilkins <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant