Skip to content

logging: backend: add KConfig option for SWO sync packets #91745

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
Merged
merged 1 commit into from
Oct 15, 2025
Merged

logging: backend: add KConfig option for SWO sync packets #91745

merged 1 commit into from
Oct 15, 2025

Conversation

fbnors
Copy link
Contributor

@fbnors fbnors commented Jun 17, 2025

Add KConfig LOG_BACKEND_SWO_SYNC_PACKET to allow disabling SWO synchronization packets. Useful if the SWO is used as a UART replacement to avoid that the sync packets show up as special characters in a terminal application.

@github-actions github-actions bot requested review from dcpleung and nordic-krch June 17, 2025 07:44
@fbnors fbnors force-pushed the swo-sync-kconfig branch from 3090697 to 883f8fa Compare June 17, 2025 07:59
@fbnors fbnors force-pushed the swo-sync-kconfig branch from 883f8fa to 3a86b72 Compare June 24, 2025 12:17
nordic-krch
nordic-krch previously approved these changes Jun 25, 2025
View reviewed changes
kartben
kartben reviewed Jun 25, 2025
View reviewed changes
Copy link
Contributor

@kartben kartben left a comment

Choose a reason for hiding this comment

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

please add new kconfig option to 4.2 release notes - thanks!

@fbnors fbnors force-pushed the swo-sync-kconfig branch from 3a86b72 to f40699d Compare June 25, 2025 15:03
@github-actions github-actions bot added the Release Notes To be mentioned in the release notes label Jun 25, 2025
@fbnors
Copy link
Contributor Author

fbnors commented Jun 25, 2025 •
edited
Loading

please add new kconfig option to 4.2 release notes - thanks!

Done! :-)

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

kartben
kartben previously requested changes Jul 22, 2025
View reviewed changes
Copy link
Contributor

@kartben kartben left a comment

Choose a reason for hiding this comment

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

release note update needs to be made against 4.3 file

@fbnors fbnors force-pushed the swo-sync-kconfig branch from f40699d to 59862fd Compare July 28, 2025 14:09
@zephyrbot zephyrbot requested review from cfriedt and jhedberg July 28, 2025 14:10
@fbnors
Copy link
Contributor Author

fbnors commented Jul 28, 2025

release note update needs to be made against 4.3 file

Done

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@kartben kartben dismissed their stale review July 29, 2025 08:52

Addressed

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@fbnors
Copy link
Contributor Author

fbnors commented Sep 12, 2025

Rebased on main branch. Please review.

@fbnors
logging: backend: add KConfig option for SWO sync packets
44917a0 
Add KConfig LOG_BACKEND_SWO_SYNC_PACKET to allow disabling SWO
synchronization packets. Useful if the SWO is used as a UART replacement
to avoid that the sync packets show up as special characters in a terminal
application.

Signed-off-by: Fabian Otto <[email protected]>
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@fbnors
Copy link
Contributor Author

fbnors commented Oct 14, 2025

@nordic-krch @kartben
I rebased again. Could you please have a look and approve the PR?

@jhedberg jhedberg merged commit 8d09755 into zephyrproject-rtos:main Oct 15, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kartben kartben kartben approved these changes

@nordic-krch nordic-krch nordic-krch approved these changes

@dcpleung dcpleung Awaiting requested review from dcpleung

@danieldegrasse danieldegrasse Awaiting requested review from danieldegrasse

@dkalowsk dkalowsk Awaiting requested review from dkalowsk

@cfriedt cfriedt Awaiting requested review from cfriedt

@jhedberg jhedberg Awaiting requested review from jhedberg

Assignees

@nordic-krch nordic-krch

Labels

area: Logging Release Notes To be mentioned in the release notes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@fbnors @kartben @nordic-krch @jhedberg