Re: [PATCH v5 1/1] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-06-24 Thread Guenter Roeck
On Fri, Jun 24, 2016 at 12:30:56PM -0500, Andrew F. Davis wrote: > On 06/24/2016 11:46 AM, Guenter Roeck wrote: > > On Fri, Jun 24, 2016 at 10:02:51AM -0500, Andrew F. Davis wrote: > >> On 06/18/2016 10:16 AM, Guenter Roeck wrote: > >>> > >>> The chip registers are 16 bit. Can you repeat the

Re: [PATCH v5 1/1] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-06-24 Thread Andrew F. Davis
On 06/24/2016 11:46 AM, Guenter Roeck wrote: > On Fri, Jun 24, 2016 at 10:02:51AM -0500, Andrew F. Davis wrote: >> On 06/18/2016 10:16 AM, Guenter Roeck wrote: >>> >>> The chip registers are 16 bit. Can you repeat the command using the "w" >>> option ? >>> >> >> # i2cdump -y 2 0x40 w >> 0,8

Re: [PATCH v5 1/1] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-06-24 Thread Guenter Roeck
On Fri, Jun 24, 2016 at 10:02:51AM -0500, Andrew F. Davis wrote: > On 06/18/2016 10:16 AM, Guenter Roeck wrote: > > > > The chip registers are 16 bit. Can you repeat the command using the "w" > > option ? > > > > # i2cdump -y 2 0x40 w > 0,8 1,9 2,a 3,b 4,c 5,d 6,e 7,f > 00: 2771

Re: [PATCH v5 1/1] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-06-18 Thread Guenter Roeck
Hi Andrew, On 06/17/2016 04:21 PM, Andrew F. Davis wrote: On 06/10/2016 11:44 AM, Guenter Roeck wrote: On Fri, Jun 10, 2016 at 10:32:33AM -0500, Andrew F. Davis wrote: Add support for the the INA3221 26v capable, Triple channel, Bi-Directional, Zero-Drift, Low-/High-Side, Current/Voltage

Re: [PATCH v5 1/1] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-06-17 Thread Andrew F. Davis
On 06/10/2016 11:44 AM, Guenter Roeck wrote: > On Fri, Jun 10, 2016 at 10:32:33AM -0500, Andrew F. Davis wrote: >> Add support for the the INA3221 26v capable, Triple channel, >> Bi-Directional, Zero-Drift, Low-/High-Side, Current/Voltage Monitor >> with I2C interface. >> >> Signed-off-by: Andrew

Re: [PATCH v5 1/1] hwmon: Add support for INA3221 Triple Current/Voltage Monitors

2016-06-10 Thread Guenter Roeck
On Fri, Jun 10, 2016 at 10:32:33AM -0500, Andrew F. Davis wrote: > Add support for the the INA3221 26v capable, Triple channel, > Bi-Directional, Zero-Drift, Low-/High-Side, Current/Voltage Monitor > with I2C interface. > > Signed-off-by: Andrew F. Davis Applied to -next. Can you