lkml.org 
[lkml]   [2011]   [Jan]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] hwmon: driver for Sensirion SHT21 humidity and temperature sensor
    On 01/03/11 18:09, Guenter Roeck wrote:
    > 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).
    That was my original argument when I proposed the sht15 driver. No idea
    if anyone actually does this though!
    >
    > 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 ?
    I wondered exactly this when we added the sht15 driver. Never really
    came to a firm conclusion though. If you and Jean decide these shouldn't
    be in hwmon, I'm happy to take them both in IIO. Given our scope is much
    broader and we already have things like light sensors (also sometimes environment
    monitoring devices), that would work for me. There are a few sht15 users
    out there I know of beyond the imote2 sensor boards (which is how I encountered
    them). The imote2 users should be fine as I maintain the platform and most of
    the tools anyway, the others maybe have more issues with a move.
    Uses I know of are indeed more general environment monitoring.
    Not seen one of these in conventional hardware monitoring but could be wrong.
    I guess, Urs will have a better idea of where these tend to be used?

    Obviously there is an argument for an 'environment' monitoring subsystem, but
    my guess in Linus won't pull based on exactly the same issue he had with ALS
    when we proposed that. Linus won't take small subsystems for sensors given
    he wants them all in the same one. Actually he mostly wanted them to be in input
    but Dmitry quite rightly pushed back hard against that for exactly the reason
    you are wondering whether they ought to be in hwmon, avoidance of scope spread.

    >
    > Guenter
    >
    >
    >



    \
     
     \ /
      Last update: 2011-01-04 13:15    [W:0.030 / U:31.116 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site