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

M467: Replace PlatformMutex with rtos::Mutex #322

Merged
merged 1 commit into from
Aug 16, 2024

Conversation

ccli8
Copy link

@ccli8 ccli8 commented Aug 13, 2024

Summary of changes

This PR replaces PlatformMutex with rtos::Mutex for supporting both mbed-baremetal and mbed-os builds.


Pull request type

[X] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

PlatformMutex becomes unsupported. rtos::Mutex now replaces it,
supporting both mbed-baremetal and mbed-os.
@ccli8
Copy link
Author

ccli8 commented Aug 16, 2024

Hi, this is my first time to submit to mbed-ce. Would you take a look at this?

@JohnK1987 JohnK1987 merged commit 047bbc3 into mbed-ce:master Aug 16, 2024
10 checks passed
@JohnK1987
Copy link
Member

Hi, this is my first time to submit to mbed-ce. Would you take a look at this?

Hello,

thank you for the PR and welcome to the party.

BR, Jan

@ccli8 ccli8 deleted the nvt_m467_replace_platform_mutex branch August 16, 2024 05:09
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.

2 participants