I2C readList() function error in device address and missing restart condition about adafruit_python_gpio HOT 4 CLOSED

I wish I'd found this earlier. I've just spent a long time finding this bug and came up with the exact same solution which works.
I wonder why the library hasn't been fixed yet ?
I'm going to give it a try.

from adafruit_python_gpio.

I had the same issue trying to use readList with a sensor. I can confirm that your edits solved my problem.

On a slightly related note, I noticed there is likely another issue related to bulk reads in the tests. Lines 70 & 71 of test_I2C.py read:

    def read_i2c_block_data(self, address, length):
        return [self._read_register(address+i) for i in range(length)] 

It appears as though we're incrementing the address instead of the register, meaning line 71 should be the following instead:

        return [self._read_register(address, register+i) for i in range(length)]

I noticed when I ran the tests that writeList is not being tested, which would explain how this bug got through.

from adafruit_python_gpio.

To add on to what lifetyper said the idle call is also not needed to correctly generate a repeated start condition.

from adafruit_python_gpio.

hiya thank you for the issue. we're going to be deprecating this library in favor of our python3 Blinka library which has support for almost 200 different drivers, and a wide variety of linux computers as well as ft232h

If this issue is still occuring when you use Blinka, please re-open it there!

from adafruit_python_gpio.