Adafruit_I2C does not parse the Pi Revision info properly

[wallywheely]

There is a static method in Adafruit_I2C class that reads the 'revision' info from /proc/cpuinfo.

No matter if the board is at rev 1 (first model with 256 MB) OR rev 2 (second model with 512 MB), the method will return the same value ('1').

This will cause a problem in the class initialization for board rev 2.

I suggest to change this method (getPiRevision in Adafruit_I2C) from:

with open('/proc/cpuinfo', 'r') as f:
for line in f:
if line.startswith('Revision'):
return 1 if line.rstrip()[-1] in ['1', '2'] else 2

to:

with open('/proc/cpuinfo', 'r') as f:
for line in f:
if line.startswith('Revision'):
return line.rstrip()[-1]

[tdicola]

Thanks for raising the issue and apologies it wasn't reviewed sooner. I took a look into the revisions, and based on this chart: http://elinux.org/RPi_HardwareHistory#Board_Revision_History it looks like revision 1 boards have a value that ends in 2 or 3, while revision 2 boards are higher. I updated the function and verified it works with revision 2 boards as expected. Submitted the fix with this pull http://elinux.org/RPi_HardwareHistory#Board_Revision_History Thanks again for raising the issue!