Linux IIO development
 help / color / mirror / Atom feed
From: Marc Titinger <mtitinger@baylibre.com>
To: jic23@kernel.org, knaack.h@gmx.de, lars@metafoo.de,
	pmeerw@pmeerw.net, linux@roeck-us.net, jdelvare@suse.com
Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,
	lm-sensors@lm-sensors.org, Marc Titinger <mtitinger@baylibre.com>
Subject: [PATCH 0/2] IIO version of INA2xx (followup of related RFC)
Date: Wed, 25 Nov 2015 12:28:14 +0100	[thread overview]
Message-ID: <1448450896-24387-1-git-send-email-mtitinger@baylibre.com> (raw)

following RFC started in https://lkml.org/lkml/2015/11/10/370
and feed back from https://lkml.org/lkml/2015/11/18/395

 * squash to a single patch that provides DIRECT and SOFTWARE buffer mode.

 * implement INFO_INT_TIME abi for each adc in order to better match how
   the chip works.  This also allows to compute the actual sample freq
   for INFO_SAMP_FREQ, that results from both the averaging ratio and the
   possible integration times.

 * Add an INT_TIME setting for each voltage ADC (default values are compa
   -tible for previous implementations).

 * provide the averaging feature of the chip using the OVERSAMPLING_RATIO
   abi.

 * by default, only issue a new sample value in the buffer when the
   Conversion Ready Flag indicates that a new value is available. The
   capture thread polls slightly faster than the chip-internal sampling
   clock to prevent re-read or skipping of samples.

 * Since this check for CVRF has its cost (i2c xfer), allow for a relaxed
   mode for when re-read or skipping or one sample is not big deal, but
   a faster sampling rate is wanted.

 * remove the calibration INFO, since the driver sets a hardcoded value
   for 'Current_LSB', only RShunt is available as a parameter. No use to
   expose the register to the user.

Why two drivers (hwmon and IIO) for this device ?
------------------------------------------------

 * Hwmon and IIO do not address exactly the same use-cases, while this chip
   can (and is) being used either as a power monitoring feature of a host
   device or as sensor to measure power properties of a target DUT.

 * In the second use-case (probing a DUT) we wish to plot measurements
   over time, display transients, peak values, compute derived metrics
   (like energy). A buffer streaming scheme and remote capabilities
   with libiio seems beneficial.


Marc Titinger (1):
  iio: ina2xx: add support for TI INA2xx Power Monitors
  iio: ina2xx: provide a sysfs parameter to allow async readout of the
    ADCs

 drivers/iio/adc/Kconfig      |  10 +
 drivers/iio/adc/Makefile     |   1 +
 drivers/iio/adc/ina2xx-iio.c | 720 +++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 731 insertions(+)
 create mode 100644 drivers/iio/adc/ina2xx-iio.c

-- 
1.9.1


             reply	other threads:[~2015-11-25 11:29 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-25 11:28 Marc Titinger [this message]
2015-11-25 11:28 ` [PATCH 1/2] iio: ina2xx: add support for TI INA2xx Power Monitors Marc Titinger
2015-11-25 12:20   ` Peter Meerwald-Stadler
2015-11-26  9:00     ` Marc Titinger
2015-11-29 16:31   ` Jonathan Cameron
2015-11-29 17:38     ` Guenter Roeck
2015-12-01 10:01       ` Marc Titinger
2015-12-01 15:02         ` Guenter Roeck
2015-11-25 11:28 ` [PATCH 2/2] iio: ina2xx: provide a sysfs parameter to allow async readout of the ADCs Marc Titinger
2015-11-29 16:33   ` Jonathan Cameron

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1448450896-24387-1-git-send-email-mtitinger@baylibre.com \
    --to=mtitinger@baylibre.com \
    --cc=jdelvare@suse.com \
    --cc=jic23@kernel.org \
    --cc=knaack.h@gmx.de \
    --cc=lars@metafoo.de \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=lm-sensors@lm-sensors.org \
    --cc=pmeerw@pmeerw.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox