Skip to content

Slowdown i2c #19

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

Merged
merged 7 commits into from
Jun 15, 2021
Merged

Slowdown i2c #19

merged 7 commits into from
Jun 15, 2021

Conversation

ladyada
Copy link
Member

@ladyada ladyada commented Jun 15, 2021

clean example
slow down i2c default for finicky controllers
crc check register read

fixes #18
resolves #13 (even tho it doesnt matter)

ladyada added 3 commits June 15, 2021 14:33
slow down i2c to 1khz (works on ft232h now)
add inter-command delays (5ms)
@ladyada ladyada requested a review from caternuson June 15, 2021 18:38
@github-actions
Copy link

👋 Thanks for this pull request! Unfortunately, it looks like the automated continuous integration (CI) test(s) failed. These can be tricky to fix so we've written a guide on how to fix them locally. It has pages about running pre-commit locally and another about building the docs locally with sphinx. Thanks for contributing to CircuitPython! If you have more questions, feel free to join the Adafruit Discord and post in #circuitpython-dev.

@caternuson
Copy link
Contributor

Looks good. Tested with both MCP2221 and FT232H.

@ladyada ladyada merged commit 048b8da into main Jun 15, 2021
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Jun 16, 2021
Updating https://github.com/adafruit/Adafruit_CircuitPython_SCD30 to 2.2.0 from 2.1.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_SCD30#19 from adafruit/slowdown
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template
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.

NACKing with FT232H Datasheet mentions 3ms delays in i2c communication
2 participants