linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
  • * Re: [PATCH 0/2] iio: Heart Rate Monitors
           [not found] <1446309089-21094-1-git-send-email-afd@ti.com>
           [not found] ` <1446309089-21094-2-git-send-email-afd@ti.com>
    @ 2015-11-01 18:35 ` Jonathan Cameron
      2015-11-02 16:31   ` Andrew F. Davis
           [not found] ` <1446309089-21094-3-git-send-email-afd@ti.com>
      2 siblings, 1 reply; 14+ messages in thread
    From: Jonathan Cameron @ 2015-11-01 18:35 UTC (permalink / raw)
      To: Andrew F. Davis, Rob Herring, Pawel Moll, Mark Rutland,
    	Ian Campbell, Kumar Gala, Hartmut Knaack, Lars-Peter Clausen,
    	Peter Meerwald
      Cc: devicetree, linux-iio, linux-api, linux-kernel
    
    On 31/10/15 16:31, Andrew F. Davis wrote:
    > Hello all,
    > 
    > This series adds the TI AFE4404 "Ultra-small, Integrated AFE for
    > Wearable, Optical Heart Rate Monitoring and Bio-Sensing".
    > 
    > This work is based on previous work by Dan Murphy [0] who is working
    > on other tasks at the moment, so I will be helping to continue
    > upstreaming this driver. This is more of a re-write than a continuation
    > and there are many changes so I am submitting this as a v1.
    > 
    > This device is very similar to the AFE4403 and I was originally planning
    > on pushing the two drivers together with common core functions in a
    > third file. The AFE4403 driver is still being tested so I merged common
    > code back into this driver, this is why this driver may seem a bit
    > unnecessarily modular. I will probably split this stuff back out when
    > I push the AFE4403.
    > 
    > I also had some issues with sysfs naming for the channels; this device
    > has three input channels from three LED stages and two ambient
    > channels based on the LED stages. This might have been be a good place
    > for using IIO modifiers[1], but we also have two differential channels
    > based on the ambient channels, and channels cannot have both modifiers
    > and be differential (the modifier is stored in the differential channel's
    > ID field?).
    True enough. Didn't expect to run into this particular problem, but I guess
    someone will always make hardware breaking any assumptions we make from the
    software side of things.
     So I used sysfs names that would be close to what they
    > would be if IIO supported these things.
    Fair enough as a starting point though we probably want to figure out how
    to do this 'right'.   Adding an extra field to the channel descriptor will
    be easy enough - it'll be event codes that are nasty to handle.
    
    Jonathan
    > 
    > [0] http://www.spinics.net/lists/linux-iio/msg18413.html
    > [1] IIO_MOD_TEMP_AMBIENT could be renamed IIO_MOD_AMBIENT as it can
    > also apply to LIGHT, PRESSURE, HUMIDITY, etc..
    No problem with this change so please send a patch.
    > 
    > Thanks,
    > Andrew
    > 
    > Andrew F. Davis (2):
    >   Documentation: afe4404: Add DT bindings for the AFE4404 heart monitor
    >   iio: health: Add driver for the TI AFE4404 heart monitor
    > 
    >  .../ABI/testing/sysfs-bus-iio-health-afe4404       |  70 +++
    >  .../devicetree/bindings/iio/health/afe4404.txt     |  27 ++
    >  drivers/iio/Kconfig                                |   1 +
    >  drivers/iio/Makefile                               |   1 +
    >  drivers/iio/health/Kconfig                         |  24 +
    >  drivers/iio/health/Makefile                        |   6 +
    >  drivers/iio/health/afe4404.c                       | 526 +++++++++++++++++++++
    >  drivers/iio/health/afe440x.h                       | 159 +++++++
    >  8 files changed, 814 insertions(+)
    >  create mode 100644 Documentation/ABI/testing/sysfs-bus-iio-health-afe4404
    >  create mode 100644 Documentation/devicetree/bindings/iio/health/afe4404.txt
    >  create mode 100644 drivers/iio/health/Kconfig
    >  create mode 100644 drivers/iio/health/Makefile
    >  create mode 100644 drivers/iio/health/afe4404.c
    >  create mode 100644 drivers/iio/health/afe440x.h
    > 
    
    
    ^ permalink raw reply	[flat|nested] 14+ messages in thread
  • [parent not found: <1446309089-21094-3-git-send-email-afd@ti.com>]

  • end of thread, other threads:[~2015-11-23 20:53 UTC | newest]
    
    Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
    -- links below jump to the message on this page --
         [not found] <1446309089-21094-1-git-send-email-afd@ti.com>
         [not found] ` <1446309089-21094-2-git-send-email-afd@ti.com>
    2015-10-31 18:44   ` [PATCH 1/2] Documentation: afe4404: Add DT bindings for the AFE4404 heart monitor Rob Herring
    2015-11-02 16:08     ` Andrew F. Davis
    2015-11-01 18:35 ` [PATCH 0/2] iio: Heart Rate Monitors Jonathan Cameron
    2015-11-02 16:31   ` Andrew F. Davis
    2015-11-04 18:57     ` Jonathan Cameron
         [not found] ` <1446309089-21094-3-git-send-email-afd@ti.com>
    2015-11-01 20:52   ` [PATCH 2/2] iio: health: Add driver for the TI AFE4404 heart monitor Jonathan Cameron
    2015-11-02 20:37     ` Andrew F. Davis
    2015-11-04 19:40       ` Jonathan Cameron
    2015-11-04 21:17         ` Andrew F. Davis
    2015-11-05 19:09           ` Jonathan Cameron
    2015-11-10 19:19             ` Andrew F. Davis
    2015-11-15 12:07               ` Jonathan Cameron
    2015-11-17 17:07                 ` Andrew F. Davis
    2015-11-23 20:53                   ` Andrew F. Davis
    

    This is a public inbox, see mirroring instructions
    for how to clone and mirror all data and code used for this inbox;
    as well as URLs for NNTP newsgroup(s).