Hi Hans, I have tried the driver with on kernel 2.6.20.1 for platform ARM (processor PXA255) to get fan speeds from the chip max6651. It seems that the driver works, because it exports the values in /sys file system. I have few small comments on the implementation: * use mutex instead of semaphore * define i2c driver ID in the include/i2c-id.h However I have noticed the following issue, which might be related to the driver of libsensors. Whenever I issue the command sensors to display values of the fans, I get an error: $ sensors max1617a-i2c-0-18 Adapter: pxa2xx-i2c Main PCB: +40 C (low = +0 C, high = +50 C) max1617a-i2c-0-29 Adapter: pxa2xx-i2c Main PCB: +38 C (low = +0 C, high = +50 C) max6650-i2c-0-48 Adapter: pxa2xx-i2c ERROR: Can't get FAN1 data! ERROR: Can't get FAN2 data! ERROR: Can't get FAN3 data! ERROR: Can't get FAN4 data! max6650-i2c-0-4b Adapter: pxa2xx-i2c ERROR: Can't get FAN1 data! ERROR: Can't get FAN2 data! ERROR: Can't get FAN3 data! ERROR: Can't get FAN4 data! $ sensors --version sensors version 2.10.2 with libsensors version 2.10.2 I tried to rename entries that the driver creates from fan0-fan3 to fan1-fan4 as used by the libsensors and sensors, but that didn't help. Did you have to modify the (lib)sensors in any way to display the values? Matej -- Matej Kenda, Senior Software Engineer HERMES SoftLab d.d. (www.hermes-softlab.com), Slovenia
