finani / ICM42688

Arduino library for communicating with the InvenSense ICM42688 nine-axis IMUs.
MIT License
47 stars 21 forks source link

Added SDA and SCL pin definitions to allow for more versatility #17

Closed jbuck2005 closed 2 months ago

jbuck2005 commented 2 months ago

I've added the requirement to specify the SDA and SCL pins because it adds a lot more flexibility to the library. The default for ESP32 (21,22) and Arduino (18,19) are specified in the examples.

In the ESP32C3 series, any pin can easily be used for I2C and so it is best -not- to limit people to the default(s), rather encourage the user to define the I2C pins as part of instantiation. This makes the library more accessible to inexperienced users.

finani commented 2 months ago

The default for this library is Arduino. So the default pin number will be changed. Thanks for your contribution!