Hi Dirk, On Wed, Feb 09, 2011 at 04:51:34AM -0500, Dirk Eibach wrote: [ ... ] > +/* temp2_crit can only be set once which should be job of the bootloader */ > +static SENSOR_DEVICE_ATTR(temp2_crit, S_IRUGO, show_remote_temp8, > + NULL, 2); That only applies to lm63, though. It can be rewritten anytime on LM64. That would be a separate patch, though, to not mix improvements with bug fixes. Otherwise, looks good, so I'll apply it with an added remark that the restriction only applies to lm63. Thanks, Guenter _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors