Skip to content

For atmel-samd release SERCOM on I2C deinit #10402

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

eightycc
Copy link
Collaborator

@eightycc eightycc commented Jun 4, 2025

This PR corrects an issue where creating a new board.I2C object after a VM reset would throw a ValueError: Invalid pins.

This PR also fixes a minor typo in the https address of the current ARM Cortex toolchain.

@eightycc
Copy link
Collaborator Author

eightycc commented Jun 4, 2025

@dhalbert This should fix the issue Ross is seeing with I2C on his new board. I have not tested it myself, but it should be safe.

@eightycc eightycc requested a review from dhalbert June 4, 2025 20:46
@dhalbert
Copy link
Collaborator

dhalbert commented Jun 4, 2025

@eightycc I understand the fix, but were you able to replicate the problem? Doing something like
code.py:

import board
i = board.I2C()
while True:
    pass

and running it again and again does not seem to cause the problem.

@eightycc
Copy link
Collaborator Author

eightycc commented Jun 4, 2025

@dhalbert I didn't try to replicate the problem. The fix is based purely on the reported symptoms and my examination of the code.

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