On Wed, Jul 15, 2009 at 10:28:24PM +0200, Jean Delvare wrote: > Make the I/O faster, mainly by using combined SMBus transactions when > possible. While the TSL2550 datasheet doesn't say the device supports > them, they seem to work just fine in practice, and a combined > transaction is faster than two simple transactions in many cases and > always more reliable. > > A side effect is to suppress the delays between SMBus writes and > reads. The datasheet doesn't say they are needed and things work just > fine for me without them. > > I also couldn't see any reason for the delay between reading the two > channels. Nor for the loop to get a reading in the first place. The > 400 ms delay between samples only matters at chip power-up, after that > the chip always hold the previously sampled value so we never get to > wait. > > All these changes make reading the lux value much faster and cheaper. > > Signed-off-by: Jean Delvare <khali@xxxxxxxxxxxx> > Cc: Michele De Candia <michele.decandia@xxxxxxxxxxxxx> > Cc: Rodolfo Giometti <giometti@xxxxxxxx> > --- > Michele, Rodolfo, can you please test and confirm the driver still > works OK for you? Thanks. I'm sorry but I cannot do it right now (and in the near future) since my hardware is out of duty, but if Michele and you have tested this new implementation then the modification it's ok for me! :) Acked-by: Rodolfo Giometti <giometti@xxxxxxxx> Ciao, Rodolfo -- GNU/Linux Solutions e-mail: giometti@xxxxxxxxxxxx Linux Device Driver giometti@xxxxxxxx Embedded Systems phone: +39 349 2432127 UNIX programming skype: rodolfo.giometti -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
