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.

Sign up for GitHub

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

Jump to bottom

i2c: sam0 driver does not prevent simultaneous transactions #42425

Closed
maxmclau opened this issue Feb 2, 2022 · 0 comments
Closed

i2c: sam0 driver does not prevent simultaneous transactions #42425

maxmclau opened this issue Feb 2, 2022 · 0 comments
Labels
Enhancement Changes/Updates/Additions to existing features

Comments

@maxmclau
Copy link
Contributor

maxmclau commented Feb 2, 2022

Is your enhancement proposal related to a problem? Please describe.
There are no protections against simultaneous i2c transactions colliding on the same bus. Identical to #29239.

Describe the solution you'd like
It seems like consensus from issues like #24511 is that a global i2c lock is the way to go, but I don't know what the status of that is.

Proposed change
Implement a locking semaphore for i2c_sam0_configure and i2c_sam0_transfer.
@maxmclau maxmclau added the Enhancement Changes/Updates/Additions to existing features label Feb 2, 2022
@maxmclau maxmclau mentioned this issue Feb 2, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

i2c: add locking sem to sam0 driver
1 participant
@maxmclau