ftjuh / i2cwrapper Goto Github PK

It worked (with small edits):

controller: adafruit metro express (m0)
target: adafruit itsybitsy m0
examples/Pin_control
edited firmware/firmware_modules.h to include PinI2C_firmware.h
dPinOut = 13
dPinIn = 12
aPinOut = 3
aPinIn = 14 // A0

Please see
https://github.com/awgrover/I2Cwrapper/pull/2
I did a PR in my repository so it wouldn't clutter up yours (yet). I'll move it to yours if/when you want it.

That PR only has the fixes to enable the samd to work (and only PinI2C). I can implement the accelstepper and servo.

Please also see
https://github.com/awgrover/I2Cwrapper/pull/2
That PR has changes I made to examples/Pin_control to enable a step-by-step process of testing. Specifically, I added a "loopback" idea for PinI2C. And, I would do the same if I implement accelstepper and servo.

These changes are more exploratory, and don't have to be accepted with the samd fixes, but it made my life easier.

If you have the old AccelStepperI2C installed with this new library, it won't pick up the right .h file (I think it does alphabetical order!).

How much of a problem is that? I think that everyone who upgrades to this new library will feel the pain. There does not seem to be a good deprecation/conflict mechanism.

There were some questions about deprecation/conflict:

• https://github.com/arduino/arduino-cli/issues/1001 "use deprecation/warnings in the old .h"
• https://forum.arduino.cc/t/issue-warning-for-deprecated-library-function/691200/2 how to deprecate

#include <PinI2C.h>
Causes

Multiple libraries were found for "PinI2C.h"
 Used: /home/awgrover/Arduino/libraries/AccelStepperI2C
 Not used: /home/awgrover/Arduino/libraries/I2Cwrapper
exit status 1

But, that is the last message. And, for other .h can cause a bunch of preceding errors that are confusing, like

someclass does not name a type