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

pkg/mynewt-core: fix semaphore #17771

Merged
merged 1 commit into from
Mar 9, 2022

Conversation

fjmolinas
Copy link
Contributor

Contribution description

The current implementation could not block forever, this did not really show since it would result in waiting for UINT32_MAX, bode makes for a more complete implementation.

Testing procedure

Green murdock should be enough.

@fjmolinas fjmolinas added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Mar 8, 2022
@github-actions github-actions bot added the Area: pkg Area: External package ports label Mar 8, 2022
@fjmolinas fjmolinas merged commit 5702ca0 into RIOT-OS:master Mar 9, 2022
@fjmolinas
Copy link
Contributor Author

Thanks!

@fjmolinas fjmolinas deleted the pr_mynewt_core_fix_sema branch March 9, 2022 11:36
@OlegHahm OlegHahm added this to the Release 2022.04 milestone Apr 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: pkg Area: External package ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants