Re: [PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-12 Thread Andrew F. Davis
On 04/12/2016 03:29 AM, ji...@jic23.retrosnub.co.uk wrote: > On 11.04.2016 16:48, Andrew F. Davis wrote: >> On 04/10/2016 06:57 AM, Jonathan Cameron wrote: >>> On 08/04/16 19:19, Andrew F. Davis wrote: The INA3221 is a three-channel, high-side current and bus voltage monitor with an

Re: [PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-12 Thread jic23
On 12.04.2016 09:34, ji...@jic23.retrosnub.co.uk wrote: On 11.04.2016 19:08, Guenter Roeck wrote: On Mon, Apr 11, 2016 at 11:47:44AM -0500, Andrew F. Davis wrote: On 04/11/2016 11:38 AM, Guenter Roeck wrote: > On Mon, Apr 11, 2016 at 10:48:27AM -0500, Andrew F. Davis wrote: >> On 04/10/2016 06:

Re: [PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-12 Thread jic23
On 11.04.2016 19:08, Guenter Roeck wrote: On Mon, Apr 11, 2016 at 11:47:44AM -0500, Andrew F. Davis wrote: On 04/11/2016 11:38 AM, Guenter Roeck wrote: > On Mon, Apr 11, 2016 at 10:48:27AM -0500, Andrew F. Davis wrote: >> On 04/10/2016 06:57 AM, Jonathan Cameron wrote: >>> On 08/04/16 19:19, And

Re: [PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-12 Thread jic23
On 11.04.2016 16:48, Andrew F. Davis wrote: On 04/10/2016 06:57 AM, Jonathan Cameron wrote: On 08/04/16 19:19, Andrew F. Davis wrote: The INA3221 is a three-channel, high-side current and bus voltage monitor with an I2C and SMBUS compatible interface. Signed-off-by: Andrew F. Davis Hi Andre

Re: [PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-11 Thread Guenter Roeck
On Mon, Apr 11, 2016 at 11:47:44AM -0500, Andrew F. Davis wrote: > On 04/11/2016 11:38 AM, Guenter Roeck wrote: > > On Mon, Apr 11, 2016 at 10:48:27AM -0500, Andrew F. Davis wrote: > >> On 04/10/2016 06:57 AM, Jonathan Cameron wrote: > >>> On 08/04/16 19:19, Andrew F. Davis wrote: > The INA322

Re: [PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-11 Thread Andrew F. Davis
On 04/11/2016 11:38 AM, Guenter Roeck wrote: > On Mon, Apr 11, 2016 at 10:48:27AM -0500, Andrew F. Davis wrote: >> On 04/10/2016 06:57 AM, Jonathan Cameron wrote: >>> On 08/04/16 19:19, Andrew F. Davis wrote: The INA3221 is a three-channel, high-side current and bus voltage monitor with a

Re: [PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-11 Thread Guenter Roeck
On Mon, Apr 11, 2016 at 10:48:27AM -0500, Andrew F. Davis wrote: > On 04/10/2016 06:57 AM, Jonathan Cameron wrote: > > On 08/04/16 19:19, Andrew F. Davis wrote: > >> The INA3221 is a three-channel, high-side current and bus voltage monitor > >> with an I2C and SMBUS compatible interface. > >> > >>

Re: [PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-11 Thread Guenter Roeck
On Mon, Apr 11, 2016 at 10:59:10AM -0500, Andrew F. Davis wrote: > On 04/10/2016 10:16 AM, Guenter Roeck wrote: > > On 04/10/2016 04:57 AM, Jonathan Cameron wrote: > >> On 08/04/16 19:19, Andrew F. Davis wrote: > >>> The INA3221 is a three-channel, high-side current and bus voltage > >>> monitor >

Re: [PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-11 Thread Andrew F. Davis
On 04/10/2016 10:16 AM, Guenter Roeck wrote: > On 04/10/2016 04:57 AM, Jonathan Cameron wrote: >> On 08/04/16 19:19, Andrew F. Davis wrote: >>> The INA3221 is a three-channel, high-side current and bus voltage >>> monitor >>> with an I2C and SMBUS compatible interface. >>> >>> Signed-off-by: Andrew

Re: [PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-11 Thread Andrew F. Davis
On 04/10/2016 06:57 AM, Jonathan Cameron wrote: > On 08/04/16 19:19, Andrew F. Davis wrote: >> The INA3221 is a three-channel, high-side current and bus voltage monitor >> with an I2C and SMBUS compatible interface. >> >> Signed-off-by: Andrew F. Davis > Hi Andrew, > > My immediate thought on thi

Re: [PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-10 Thread Guenter Roeck
On 04/10/2016 04:57 AM, Jonathan Cameron wrote: On 08/04/16 19:19, Andrew F. Davis wrote: The INA3221 is a three-channel, high-side current and bus voltage monitor with an I2C and SMBUS compatible interface. Signed-off-by: Andrew F. Davis Hi Andrew, My immediate thought on this one is whethe

Re: [PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-10 Thread Jonathan Cameron
On 08/04/16 19:19, Andrew F. Davis wrote: > The INA3221 is a three-channel, high-side current and bus voltage monitor > with an I2C and SMBUS compatible interface. > > Signed-off-by: Andrew F. Davis Hi Andrew, My immediate thought on this one is whether it would be better off in hwmon. I'm not c

[PATCH 1/1] iio: adc: Add driver for the TI INA3221 Triple Current/Voltage Monitor

2016-04-08 Thread Andrew F. Davis
The INA3221 is a three-channel, high-side current and bus voltage monitor with an I2C and SMBUS compatible interface. Signed-off-by: Andrew F. Davis --- .../ABI/testing/sysfs-bus-iio-adc-ina3221 | 23 ++ drivers/iio/adc/Kconfig| 7 + drivers/iio/adc/Makefi