lkml.org 
[lkml]   [2011]   [Jan]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH] hwmon: driver for Sensirion SHT21 humidity and temperature sensor
    From
    Date
    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




    \
     
     \ /
      Last update: 2011-01-03 19:13    [W:2.152 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site