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

nuttx/mutex.h: bug fix, change MUTEX_INITIALIZER to NXMUTEX_INITIALIZER #321

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

CV-Bowen
Copy link
Contributor

Now nuttx use NXMUTEX_INITIALIZER to initialize the mutex, so updated.

@CV-Bowen
Copy link
Contributor Author

CV-Bowen commented Nov 4, 2024

@arnopo Hi, could you review this PR?

@arnopo arnopo requested review from edmooring, arnopo and tnmysh November 6, 2024 08:49
Copy link
Contributor

@arnopo arnopo left a comment

Choose a reason for hiding this comment

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

  • Please fix commit message subject format
    lib: system: nuttx: change MUTEX_INITIALIZER to NXMUTEX_INITIALIZER
  • fix uses typo

Now nuttx uses NXMUTEX_INITIALIZER to initialize the mutex, so updated.

Signed-off-by: Bowen Wang <wangbowen6@xiaomi.com>
@CV-Bowen
Copy link
Contributor Author

@arnopo Thanks, done, please review again.

Copy link
Contributor

@edmooring edmooring left a comment

Choose a reason for hiding this comment

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

Looks good to go.

@arnopo arnopo merged commit 465fcf0 into OpenAMP:main Nov 26, 2024
6 checks passed
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.

4 participants