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

target: add weak aliasing to allow disabling targets #1044

Merged
merged 2 commits into from
Jul 26, 2022

Conversation

perigoso
Copy link
Contributor

@perigoso perigoso commented Jun 8, 2022

Cherry-picked from #1033

This PR doesn't actually allow (easily) the disabling targets, just lays some groundwork for it, and cleans a couple things up

Signed-off-by: Rafael Silva [email protected]

@dragonmux dragonmux self-requested a review June 8, 2022 16:47
@dragonmux dragonmux added this to the v1.9 release milestone Jun 8, 2022
@dragonmux dragonmux added the Enhancement General project improvement label Jun 8, 2022
@perigoso
Copy link
Contributor Author

perigoso commented Jun 8, 2022

I don't understand the chain of includes that made it so this fails with my changes, supposedly doesn't touch any of that

@perigoso
Copy link
Contributor Author

perigoso commented Jun 8, 2022

doesn't make much sense, it should have been failing before?

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.

This looks good and seems mostly ready for merge - please address the couple of points in target_probes.c and rebase against main and we'll merge it.

Thank you for the contribution!

src/target/target_probes.c Outdated Show resolved Hide resolved
src/target/target_probes.c Outdated Show resolved Hide resolved
src/target/target_probes.c Outdated Show resolved Hide resolved
@perigoso perigoso force-pushed the feature/target_disabling branch from 72a2a47 to 2709f67 Compare July 25, 2022 21:51
@perigoso perigoso requested a review from dragonmux July 25, 2022 21:54
@perigoso
Copy link
Contributor Author

perigoso commented Jul 25, 2022

I thought about adding a debug message to warn that a probe method was called that has been disabled at build time, but left it for a future commit/pr if there's interest to not block this

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, merging.

This'll definitely make, eg, just dropping all LPC targets out of firmware for debug builds.. considerably easier - thank you!

@dragonmux dragonmux merged commit bf30580 into blackmagic-debug:main Jul 26, 2022
@perigoso perigoso deleted the feature/target_disabling branch November 24, 2022 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement General project improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants