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

ctxlink/platform: Add debug output control to platform. #2030

Merged

Conversation

sidprice
Copy link
Contributor

The ctxLink platform did not have the required enable/disable code for debug output. This PR adds that functionality.

Your checklist for this pull request

Closing issues

None

@sidprice sidprice force-pushed the add_debug_output_control_to_platform branch from 0c32e40 to bf1eefc Compare December 22, 2024 18:52
@dragonmux dragonmux added this to the v2.0 release milestone Dec 22, 2024
@dragonmux dragonmux added Bug Confirmed bug Enhancement General project improvement BMP Firmware Black Magic Probe Firmware (not PC hosted software) Foreign Host Board Non Native hardware to runing Black Magic firmware on labels Dec 22, 2024
Copy link
Member

@dragonmux dragonmux left a comment

Choose a reason for hiding this comment

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

LGTM, please rebase this on main and we'll get this merged. Thank you for the contribution!

@sidprice sidprice force-pushed the add_debug_output_control_to_platform branch from bf1eefc to 94d8b3e Compare December 22, 2024 23:26
@dragonmux dragonmux merged commit 94d8b3e into blackmagic-debug:main Dec 22, 2024
36 checks passed
@sidprice sidprice deleted the add_debug_output_control_to_platform branch December 22, 2024 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BMP Firmware Black Magic Probe Firmware (not PC hosted software) Bug Confirmed bug Enhancement General project improvement Foreign Host Board Non Native hardware to runing Black Magic firmware on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants