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

bitbang: Fix FTBFS with GCC 10 #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

matthewtraughber
Copy link

This is required to build openocd-ht32 with a recent version of GCC.

GCC 10 defaults to -fno-common which breaks the sharing of bitbang_swd struct between bitbang drivers due to a missing extern.

Change-Id: I2b4122f7939cec91a72284006748f99a23548324
Signed-off-by: Andreas Fritiofson [email protected]
Reviewed-on: http://openocd.zylin.com/5592
Tested-by: jenkins
Reviewed-by: Antonio Borneo [email protected]
Reviewed-by: Jonathan McDowell [email protected]

nattgris and others added 2 commits October 24, 2022 14:57
GCC 10 defaults to -fno-common which breaks the sharing of bitbang_swd
struct between bitbang drivers due to a missing extern.

Change-Id: I2b4122f7939cec91a72284006748f99a23548324
Signed-off-by: Andreas Fritiofson <[email protected]>
Reviewed-on: http://openocd.zylin.com/5592
Tested-by: jenkins
Reviewed-by: Antonio Borneo <[email protected]>
Reviewed-by: Jonathan McDowell <[email protected]>
@matthewtraughber
Copy link
Author

af4a8ec can be removed from this PR, but it was required in order to get the RPi4B working as a debugger.

@hansemro
Copy link

Might be better to rebase with latest changes (like done here: mateuszradomski/re-masterkeys#1 (comment)). Had a better experience building on my RPi4B this way.

@matthewtraughber
Copy link
Author

Might be better to rebase with latest changes (like done here: mateuszradomski/re-masterkeys#1 (comment)). Had a better experience building on my RPi4B this way.

Thanks for the link.

I initially attempted a rebase, but ran into other issues. This patch was the minimum changes needed to get things working on my end.

If you were able to successfully rebase, then perhaps throw your changes up as a PR?

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.

3 participants