On Wed, 9 May 2012 08:15:49 -0700, Guenter Roeck wrote: > On Mon, 2012-05-07 at 11:33 -0400, R, Durgadoss wrote: > > Some time ago, you had submitted a patch to fix the usage of threshold > > attributes > > > > in coretemp driver. But we had some discussions and it did not get > > merged. > > > > I would really like to see that patch in mainline. Is there something > > that we > > > > can discuss/decide and I can work on it to submit it again, with > > changes needed, > > problem is that neither Jean nor me are happy with it. I think it is > something that should, by its nature, reside in the thermal subsystem. IIRC my main concern was that nobody was ever able to explain what semantics the thresholds were supposed to have, i.e. whether they were low or high limits, what was supposed to happen when they were crossed and who was responsible for acting upon alarm and clearing these alarms. As long as these questions are unanswered I don't feel safe applying your patch, because I suspect it could conflict with the BIOS, and I also don't see any value for the users. -- Jean Delvare _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
