Skip to content

Commit 22bddfd

Browse files
committed
Don't swap bytes when fetching the two channel registers.
This was something I must have blindly copied from somewhere else. Chris Satterlee pointed out that it wasn't necessary, based on empirical testing. (The datasheet backs Chris up, too. It says that the register values are little-endian, and the I2C's readU16() method assumes little-endian data.)
1 parent 93ae2ae commit 22bddfd

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Adafruit_TSL2561/Adafruit_TSL2561.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -300,8 +300,8 @@ def _getData(self):
300300
if self.mode == TSL2561.MODE_SINGLE:
301301
self._poweron()
302302
_sleepms(self._integrationTime.time)
303-
chan0 = self._i2c.reverseByteOrder(self._i2c.readU16(TSL2561._REGISTER['CHAN0'] | TSL2561._REG_MOD['COMMAND'] | TSL2561._REG_MOD['WORD']))
304-
chan1 = self._i2c.reverseByteOrder(self._i2c.readU16(TSL2561._REGISTER['CHAN1'] | TSL2561._REG_MOD['COMMAND'] | TSL2561._REG_MOD['WORD']))
303+
chan0 = self._i2c.readU16(TSL2561._REGISTER['CHAN0'] | TSL2561._REG_MOD['COMMAND'] | TSL2561._REG_MOD['WORD'])
304+
chan1 = self._i2c.readU16(TSL2561._REGISTER['CHAN1'] | TSL2561._REG_MOD['COMMAND'] | TSL2561._REG_MOD['WORD'])
305305
if self.mode == TSL2561.MODE_SINGLE:
306306
self._poweroff()
307307

0 commit comments

Comments
 (0)