On Mon, 2011-01-03 at 06:06 -0500, Jonathan Cameron wrote: > On 01/03/11 07:14, Urs Fleisch wrote: > > Hi Jonathan, > When posting an updated driver, use the form > [PATCH V2] hwmon: driver... > > That way it will be obvious to people that it isn't just a repost of the > previous code. > > > > Thanks for reviewing the patch. I have fixed the issues you reported and the > > style problems. > > > >> Sorry, but this isn't going to be acceptable. Classic case of magic numbers. > >> This needs to be broken up into several different attributes. > >> We have: > >> * control over the measurement resolution (which is somewhat fiddly > >> on this device) > >> * Battery voltage threshold > 2.25V > >> * Enable on chip heater > > > >> So this one is the only one I have problem with. Other two attributes are > >> standard (well humidity is pretty unusual but no one ever complained about > >> the sht15 afaik!) > > I have been thinking about this, and would like to get Jean's input. Humidity doesn't really sound like "hardware monitoring"; more like environmental monitoring. But on the other side, even though humidity doesn't really monitor the HW, it does monitor operational conditions, and its reported values may have impact on system operation (eg cause a shutdown if humidity is too high, to prevent HW damage). So question is if hwmon should include explicit environmental monitoring attributes or not, and if hwmon and environmental monitoring can and should be separated to start with (for example, what if any is the difference between environment temperature and hw monitoring temperature ?). Thoughts, anyone ? Guenter _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
