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

Mark Gpiox as Send and Sync #251

Merged
merged 1 commit into from
Jul 12, 2021

Conversation

russellmcc
Copy link
Contributor

This is appropriate because we only ever perform atomic operations on
these registers. This allows PXx to be Send + Sync.

Closes #249

@russellmcc russellmcc force-pushed the make-pxx-send-and-sync branch 2 times, most recently from 0e1c29d to 6ab7373 Compare July 6, 2021 00:16
Copy link
Member

@Sh3Rm4n Sh3Rm4n left a comment

Choose a reason for hiding this comment

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

Nice. Thanks for catching that Gpiox is not Send and Sync. I have some minor suggestions. And as this affects the API I would be happy about a changelog entry :)

src/gpio.rs Outdated Show resolved Hide resolved
@russellmcc russellmcc requested a review from Sh3Rm4n July 10, 2021 16:38
Copy link
Member

@Sh3Rm4n Sh3Rm4n left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM. There are unrelated CI failures which I'm about to fix in #252 so you can ignore these.

Either you rebase when it's merged or I will merge ignoring the failing CI :)

CHANGELOG.md Show resolved Hide resolved
@russellmcc
Copy link
Contributor Author

Thanks for the review! No preference on my end between you merging this with CI failures or rebasing after the CI fix. Let me know.

@Sh3Rm4n
Copy link
Member

Sh3Rm4n commented Jul 10, 2021

Let's rebase then :)

@russellmcc russellmcc force-pushed the make-pxx-send-and-sync branch 2 times, most recently from b5c607a to bc5c2ef Compare July 11, 2021 22:17
This is appropriate because we only ever perform atomic operations on
these registers.  This allows `PXx` to be Send + Sync.
@Sh3Rm4n Sh3Rm4n enabled auto-merge July 12, 2021 06:22
@Sh3Rm4n Sh3Rm4n merged commit 2fa1385 into stm32-rs:master Jul 12, 2021
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.

Should Gpiox be marked Send?
2 participants