Wire should have ability to not enable internal pull-ups [imported]

[cmaglie]

This is Issue 506 moved from a Google Code project.
Added by 2011-03-19T17:41:49.000Z by fabio.va...@gmail.com.
Please review that bug for more context and additional comments, but update this bug.

Original labels: Type-Defect, Priority-Medium, Component-Core

Original description

Hi there,

currently Wire (through twi_init()) enables by default the internal pullups of the uC.

As all 16MHz runs at 5V this means that with pullups enabled signals will have a 5 volt on logic level. Unfortunately, as many I2C devices runs at 3 Volts and aren't 5Volts tolerant, sending 5V signals to them is clearly a very bad idea.

I would propose making the pullpus enabling optional and off by default. In my experience, people overlook the fact that Wire does enable the pullups causing possible problems on the I2C devices used.

Moreover, tests (http://www.dsscircuits.com/articles/effects-of-varying-i2c-pull-up-resistors.html) have shown that internal pullups performs extremely bad in terms of signal quality and bus speed. So, they shouldn't been used anyway.

In case this is accepted, I'll gladly submit a patch for this.

Thanks,

Fabio Varesano

[sandeepmistry]

This is similar to arduino/Arduino#2687.

[cmaglie]

In effect arduino/Arduino#2687 is a duplicate of this one, good catch.
This one also have some more useful discussion on the original googlecode issue tracker:

http://code.google.com/p/arduino/issues/detail?id=506

so let's keep this one open.

[pwillard]

Can we please have the ability to not have internal pullups on wire. With so many vendors like Adafruit including external pull ups already (many to 3.3v) its also very redundant.

[Kyrunix]

A function to enable/disable the pullups would be a nice addition.
If you will write some arduino slaves and use a rpi or an esp as master, in that configuration the resistor value is realy low.

Editing the library every time is really annoying...