Re: ABI for these two registers?

2019-09-12 Thread Nicolin Chen
On Thu, Sep 12, 2019 at 03:45:29PM -0700, Nicolin Chen wrote: > On Thu, Sep 12, 2019 at 03:09:47PM -0700, Guenter Roeck wrote: > > On Thu, Sep 12, 2019 at 02:09:58PM -0700, Nicolin Chen wrote: > > > Hello Guenter, > > > > > > On Thu, Sep 12, 2019 at 11:32:18AM -0700, Guenter Roeck wrote: > > > >

Re: ABI for these two registers?

2019-09-12 Thread Nicolin Chen
On Thu, Sep 12, 2019 at 03:09:47PM -0700, Guenter Roeck wrote: > On Thu, Sep 12, 2019 at 02:09:58PM -0700, Nicolin Chen wrote: > > Hello Guenter, > > > > On Thu, Sep 12, 2019 at 11:32:18AM -0700, Guenter Roeck wrote: > > > On Wed, Sep 11, 2019 at 05:28:14PM -0700, Nicolin Chen wrote: > > > >

Re: ABI for these two registers?

2019-09-12 Thread Guenter Roeck
On Thu, Sep 12, 2019 at 02:09:58PM -0700, Nicolin Chen wrote: > Hello Guenter, > > On Thu, Sep 12, 2019 at 11:32:18AM -0700, Guenter Roeck wrote: > > On Wed, Sep 11, 2019 at 05:28:14PM -0700, Nicolin Chen wrote: > > > Hello Guenter, > > > > > > Datasheet:

Re: ABI for these two registers?

2019-09-12 Thread Nicolin Chen
Hello Guenter, On Thu, Sep 12, 2019 at 11:32:18AM -0700, Guenter Roeck wrote: > On Wed, Sep 11, 2019 at 05:28:14PM -0700, Nicolin Chen wrote: > > Hello Guenter, > > > > Datasheet: http://www.ti.com/lit/ds/symlink/ina3221.pdf > > (At page 32, chapter 8.6.2.14 and 8.6.2.15) > > > > I have two

Re: [v1,1/1] hwmon: (nct7904) Fix incorrect SMI status register setting of LTD temperature and fan.

2019-09-12 Thread Guenter Roeck
On Thu, Sep 12, 2019 at 11:33:00AM +, amy.s...@advantech.com.tw wrote: > From: "amy.shih" > > According to datasheet, the SMI status register setting of LTD > temperature is SMI_STS3, and the SMI status register setting > of fan is SMI_STS5 and SMI_STS6. > > Signed-off-by: amy.shih

Re: ABI for these two registers?

2019-09-12 Thread Guenter Roeck
On Wed, Sep 11, 2019 at 05:28:14PM -0700, Nicolin Chen wrote: > Hello Guenter, > > Datasheet: http://www.ti.com/lit/ds/symlink/ina3221.pdf > (At page 32, chapter 8.6.2.14 and 8.6.2.15) > > I have two registers that I need to expose to user space: > Shunt-Voltage-Sum and Shunt-Voltage-Limit