On Tue, 2011-06-21 at 05:52 -0400, Alexander Stein wrote: > An hwmon driver for the National Semiconductor LM95245 dual temperature > sensors chip. > > Signed-off-by: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxxxxxxxx> Overall pretty well written, only I am a bit at loss why you try to deal with the unsigned temperature registers. That adds a bit of complexity to the driver. Given that the signed temperature value range is up to 127 degrees C, and that the chip is only rated up to 125 degrees C, the added complexity doesn't seem to be worth it. Can you elaborate ? Other comments: For the interval attribute, idea would be to write the value into the conversion rate register. Your code does not match the interval with the rate programmed into the chip (which is the idea), nor does it update the rate if the interval is changed. Chip address 0x29 is missing. It would be nice if the driver would support limit, hysteresis, alarm, and fault attributes. Thanks, Guenter _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
