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

Fix for issue #2589 - i2Csetup for ESP 32-C3 #2600

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

SimonGAndrews
Copy link

See comments in the issue #2589 regarding the details of the change and the testing completed.
Note

  • the ESP32-S3 build was not tested but hopefully and should be ok.
  • the element of this change, which reverses default pins for SDA & SCL for the ESP32 build (as discussed in the issue) will be a breaking change for anyone depending upon the I2C1.setup() function without specifying SDA and SCL in a config for the an ESP32 setup. However the ESP32 build will now be inline with common documentation.

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.

1 participant