Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1267088
| From | Daniel Baluta <daniel.baluta@gmail.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [RFC 2/4] iio: ina2xx: add SAMP_FREQ attribute. |
| Date | 2015-11-11 11:20 +0100 |
| Message-ID | <qtAFs-1sW-19@gated-at.bofh.it> (permalink) |
| References | <qtjEC-7hv-7@gated-at.bofh.it> <qtjEE-7hv-47@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Tue, Nov 10, 2015 at 6:07 PM, Marc Titinger <mtitinger@baylibre.com> wrote:
> Signed-off-by: Marc Titinger <mtitinger@baylibre.com>
I'm in favor of small incremental patches, anyhow since this is the
initial submission
I can't see why this patch isn't squashed in the previous one?
> ---
> drivers/iio/adc/ina2xx-iio.c | 51 ++++++++++++++++++++++++++++++++++++++++++--
> 1 file changed, 49 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/iio/adc/ina2xx-iio.c b/drivers/iio/adc/ina2xx-iio.c
> index 257d8d5..92169e1 100644
> --- a/drivers/iio/adc/ina2xx-iio.c
> +++ b/drivers/iio/adc/ina2xx-iio.c
> @@ -42,7 +42,8 @@
> /* settings - depend on use case */
> #define INA219_CONFIG_DEFAULT 0x399F /* PGA=8 */
> #define INA226_CONFIG_DEFAULT 0x4327
> -#define INA226_DEFAULT_AVG 4
> +#define INA226_DEFAULT_AVG 4
> +#define INA226_DEFAULT_FREQ 454
>
> #define INA2XX_RSHUNT_DEFAULT 10000
>
> @@ -51,6 +52,8 @@
> #define INA226_READ_AVG(reg) (((reg) & INA226_AVG_RD_MASK) >> 9)
> #define INA226_SHIFT_AVG(val) ((val) << 9)
>
> +#define INA226_SFREQ_RD_MASK 0x01f8
> +
> static struct regmap_config ina2xx_regmap_config = {
> .reg_bits = 8,
> .val_bits = 16,
> @@ -172,6 +175,10 @@ static int ina2xx_read_raw(struct iio_dev *indio_dev,
> return ina2xx_get_value(chip, INA2XX_CALIBRATION, regval,
> val, val2);
>
> + case IIO_CHAN_INFO_SAMP_FREQ:
> + *val = chip->freq;
> + return IIO_VAL_INT;
> +
> default:
> return -EINVAL;
> }
> @@ -216,6 +223,38 @@ static unsigned int ina226_set_average(struct ina2xx_chip_info *chip,
> return 0;
> }
>
> +/*
> + * Conversion times in uS
> + */
> +static const int ina226_conv_time_tab[] = { 140, 204, 332, 588, 1100,
> + 2116, 4156, 8244};
> +
> +static unsigned int ina226_set_frequency(struct ina2xx_chip_info *chip,
> + unsigned int val,
> + unsigned int *config)
> +{
> + int bits;
> +
> + if (val > 3550 || val < 50)
> + return -EINVAL;
> +
> + /* integration time in uS, for both voltage channels */
> + val = DIV_ROUND_CLOSEST(1000000, 2 * val);
> +
> + bits = find_closest(val, ina226_conv_time_tab,
> + ARRAY_SIZE(ina226_conv_time_tab));
> +
> + chip->period_us = 2 * ina226_conv_time_tab[bits];
> +
> + chip->freq = DIV_ROUND_CLOSEST(1000000, chip->period_us);
> +
> + *config &= ~INA226_SFREQ_RD_MASK;
> + *config |= (bits << 3) | (bits << 6);
> +
> + return 0;
> +}
> +
> +
> static int ina2xx_write_raw(struct iio_dev *indio_dev,
> struct iio_chan_spec const *chan,
> int val, int val2, long mask)
> @@ -238,6 +277,11 @@ static int ina2xx_write_raw(struct iio_dev *indio_dev,
> ret = ina226_set_average(chip, val, &tmp);
> break;
>
> + case IIO_CHAN_INFO_SAMP_FREQ:
> +
> + ret = ina226_set_frequency(chip, val, &tmp);
> + break;
> +
> default:
> ret = -EINVAL;
> }
> @@ -257,6 +301,7 @@ _err:
> .channel = (_index), \
> .info_mask_separate = BIT(IIO_CHAN_INFO_RAW), \
> .info_mask_shared_by_dir = BIT(IIO_CHAN_INFO_AVERAGE_RAW) | \
> + BIT(IIO_CHAN_INFO_SAMP_FREQ) | \
> BIT(IIO_CHAN_INFO_CALIBSCALE), \
> .scan_index = (_index), \
> .scan_type = { \
> @@ -355,8 +400,10 @@ static int ina2xx_probe(struct i2c_client *client,
>
> /* Patch the current config register with default. */
> val = chip->config->config_default;
> - if (id->driver_data == ina226)
> + if (id->driver_data == ina226) {
> ina226_set_average(chip, INA226_DEFAULT_AVG, &val);
> + ina226_set_frequency(chip, INA226_DEFAULT_FREQ, &val);
> + }
>
> ret = ina2xx_init(chip, val);
> if (ret < 0) {
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread
[RFC 0/4] IIO: add support for INA2xx power monitor Marc Titinger <mtitinger@baylibre.com> - 2015-11-10 17:10 +0100
[RFC 1/4] iio: ina2xx: add direct IO support for TI INA2xx Power Monitors Marc Titinger <mtitinger@baylibre.com> - 2015-11-10 17:10 +0100
Re: [RFC 1/4] iio: ina2xx: add direct IO support for TI INA2xx Power Monitors Daniel Baluta <daniel.baluta@gmail.com> - 2015-11-11 11:20 +0100
Re: [RFC 1/4] iio: ina2xx: add direct IO support for TI INA2xx Power Monitors Marc Titinger <mtitinger@baylibre.com> - 2015-11-12 10:30 +0100
Re: [RFC 1/4] iio: ina2xx: add direct IO support for TI INA2xx Power Monitors Daniel Baluta <daniel.baluta@gmail.com> - 2015-11-11 13:20 +0100
Re: [RFC 1/4] iio: ina2xx: add direct IO support for TI INA2xx Power Monitors Marc Titinger <mtitinger@baylibre.com> - 2015-11-12 10:40 +0100
[RFC v2 1/2] iio: ina2xx: add direct IO support for TI INA2xx Power Monitors Marc Titinger <mtitinger@baylibre.com> - 2015-11-12 14:00 +0100
[RFC v2 2/2] iio: ina2xx: add SOFTWARE buffer mode using an iio kfifo. Marc Titinger <mtitinger@baylibre.com> - 2015-11-12 14:00 +0100
Re: [RFC v2 1/2] iio: ina2xx: add direct IO support for TI INA2xx Power Monitors Jonathan Cameron <jic23@kernel.org> - 2015-11-14 20:10 +0100
Re: [RFC v2 1/2] iio: ina2xx: add direct IO support for TI INA2xx Power Monitors Marc Titinger <mtitinger@baylibre.com> - 2015-11-16 10:40 +0100
[RFC 4/4] iio: ina2xx: add SOFTWARE buffer mode using an iio kfifo. Marc Titinger <mtitinger@baylibre.com> - 2015-11-10 17:10 +0100
Re: [RFC 4/4] iio: ina2xx: add SOFTWARE buffer mode using an iio kfifo. Lars-Peter Clausen <lars@metafoo.de> - 2015-11-10 19:30 +0100
Re: [RFC 4/4] iio: ina2xx: add SOFTWARE buffer mode using an iio kfifo. Marc Titinger <mtitinger@baylibre.com> - 2015-11-12 11:20 +0100
Re: [RFC 4/4] iio: ina2xx: add SOFTWARE buffer mode using an iio kfifo. Lars-Peter Clausen <lars@metafoo.de> - 2015-11-12 11:30 +0100
Re: [RFC 4/4] iio: ina2xx: add SOFTWARE buffer mode using an iio kfifo. Jonathan Cameron <jic23@kernel.org> - 2015-11-14 19:50 +0100
Re: [RFC 4/4] iio: ina2xx: add SOFTWARE buffer mode using an iio kfifo. Marc Titinger <mtitinger@baylibre.com> - 2015-11-16 10:40 +0100
[RFC 2/4] iio: ina2xx: add SAMP_FREQ attribute. Marc Titinger <mtitinger@baylibre.com> - 2015-11-10 17:10 +0100
Re: [RFC 2/4] iio: ina2xx: add SAMP_FREQ attribute. Daniel Baluta <daniel.baluta@gmail.com> - 2015-11-11 11:20 +0100
csiph-web