-
Notifications
You must be signed in to change notification settings - Fork 94
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(PeriphDrivers): Fix Unusable MAX32690 I2C Pins #958
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like the wrong pins for I2C2C. I believe it is P0.13/P0.14
I was only able to test I2C0 due to the pin outs of the APARD board. I2C1 (P0.11/12) has pull downs and I2C2C (P1.7/8) are used as CTS/RTS for the NINA module. If someone has a generic 140WLP board to test those on, it would be helpful.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure about the MSDK version stuff reverting to the last release.
@sihyung-maxim need to merge #954 |
This reverts commit 7db1d20.
Pull Request Template
Description
Fixes #949
The ME18 assigns the same alternate function to both sets of I2C pins. This is logged as an errata (Rev A2 Errata 16), but in order to make the second set of pins usable our drivers will now configure both sets simultaneously.
Doxygen has been updated on the init function to highlight this change:
Checklist Before Requesting Review