On Thu, Oct 14, 2010 at 05:46:26AM -0400, Jean Delvare wrote: [ ... ] > + if ((reg_config1 & 0x2A) == 0x00 > + && (reg_config2 & 0xF8) == 0x00) { > + if ((chip_id & 0xFE) == 0x10 /* W83L771AWG/ASG */ > + && reg_convrate <= 0x08) { > + name = "w83l771"; > + } else > + if (chip_id == 0x01 /* W83L771W/G */ > + && reg_convrate <= 0x09) { > + name = "w83l771"; > + } If the chips have a different maximum conversion rate, that should probably be reflected in .max_convrate as well. Unless you want to ignore it on purpose for simplicity ... which is fine with me. Acked-by: Guenter Roeck <guenter.roeck@xxxxxxxxxxxx> Guenter _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors