Add setPins to Wire instead of overloading begin() in a breaking manner

[ladyada]

hi folks - we get about 1 request every week or so for us to change our libraries so people can pass I2C pin numbers into Wire.begin()

We can't merge these PRs because official Arduino code style has Wire.begin() in sensor/device begin():
https://github.com/arduino-libraries/Arduino_LSM6DS3/blob/master/src/LSM6DS3.cpp#L76

please add a Wire.setPins() or similar function that can be called at the beginning of the sketch, instead of breaking compatibility by changing the prototype of Wire.begin() - it will make both library developers and users much happier!

see:

• Addional I2C overloads to begin adafruit/Adafruit_BME280_Library#62
• Pull request #21 can cause problems adafruit/Adafruit_MCP9808_Library#23
• Allow user to call TwoWire::begin themselves adafruit/Adafruit_BMP280_Library#38
• (and more..)

[stickbreaker]

@ladyada A simple answer for this problem, call Wire.begin(sdapin,sclpin,busspeed) in setup() and your libraries use the standard Wie.begin() in the library. (I assume your libraries execute a Wire.begin() as part of the library initialization code, multiple invocations of Wire.begin() are supported.) The Wire() code is designed to reuse the previously assigned pins and bus speed. Since the pin allocation is necessary for ESP32, it should be set. And Setup() is the logical place.

Chuck.

[ladyada]

@stickbreaker hi, thanks for the response. that is the exact thing we will not do because official Arduino code style has Wire.begin() in sensor/device begin() :)

https://www.arduino.cc/en/Reference/WireBegin

ESP changed the Wire prototype so that it is different from every other Arduino platform. We are asking that this not be done and an alternative way to set pins is defined

[stickbreaker]

@ladyada So, you want code like this?

void setup(){
#ifdef ARDUINO_ARCH_ESP32
   Wire.setpins(sda,scl);
#endif
   Wire.begin();
}

Chuck.

[ladyada]

yeah that would best! also, capitalize P in setPins() to match arduino style :)

[stickbreaker]

If you can use the #ifdef this example is already supported.

void setup(){
#ifdef ARDUINO_ARCH_ESP32
   Wire.begin(sda,scl);
#endif
   Wire.begin(); //redundant, but won't cause any problems.
}

Chuck.

[ladyada]

hi, not if the code is inside existing written libraries. they need to be able to call setPins before the sensor begin

APDS9960 sensor = APDS9960();

void setup() {
   Wire.setPins(4, 5);
   sensor.begin(); // calls Wire.begin
}

[Andrewiski]

@ladyada Is this why the OLED feather displays garbage on ESP32 but same code works on a Wiced Feather?

[ladyada]

@Andrewiski probably not - if you need tech support please post in the adafruit forums :)

[ladyada]

@stickbreaker hi hi, checkin' in - do you have any other questions on this issue? :)

[stale]

[STALE_SET] This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.

[ladyada]

@me-no-dev can y'all assign someone else who is interested in this issue? @stickbreaker has not replied in ~2 months.

[stale]

[STALE_CLR] This issue has been removed from the stale queue. Please ensure activity to keep it openin the future.