Wire.usePullups() stopped working after version 2.6.1

[jvasileff]

As can be confirmed with a multimeter and the following setup code:

Wire.usePullups();
Wire.begin();

Wire.usePullups() stopped pulling PB0 and PB1 high with version 2.6.2 or 2.6.3. This was tested with an attiny1627.

[MX682X]

megaTinyCore/megaavr/libraries/Wire/src/twi_pins.c

Line 441 in f0e5976

#elif defined(MEGATINYCORE) && MEGATINYCORE_SERIES != 2 /* tinyAVR 0/1-series */

could you try the newest version but remove the right part of the #if conditional ( " && MEGATINYCORE_SERIES != 2 ")? I don't have a 2-Series to test

[jvasileff]

Thanks! Indeed, that does fix Wire.usePullups() on 2.6.10. I just did a quick test with a multimeter, I didn't try to actually use i2c with this change.

[jvasileff]

I've now tested actually using i2c with this change - everything works great.

[SpenceKonde]

There are CONSIDERABLY more serious issues here. Much more complicated than implied here.

[MX682X]

How so?
As far as I can see the use Pullups function isn't doing anything in the 2 series.
all we have to do is to #if def the OUTCLR, however I think it would be sensible to just keep it like that - I doubt anyone will end up using I2C pins on a PCB for anything but I2C