Re: [PATCH] hwmon (ina3221) Add single-shot mode support

2018-11-19 Thread Nicolin Chen
On Mon, Nov 19, 2018 at 09:45:59AM -0800, Guenter Roeck wrote: > > In short, other than exposing it via a generic ABI to the user > > space, how about defining some policy to maintaining it within > > the driver? > I think that would be a bad idea. It changes timing for everyone > curently using

Re: [PATCH v2 1/2] dt-bindings: hwmon: add binding documentation for adt7475

2018-11-19 Thread Chris Packham
On 20/11/18 10:40 AM, Rob Herring wrote: > On Sun, Nov 18, 2018 at 4:56 PM Guenter Roeck wrote: >> >> On 11/17/18 7:29 AM, Rob Herring wrote: >>> On Fri, Nov 09, 2018 at 11:56:06AM +1300, Chris Packham wrote: With the addition of the invert-pwm property the adt7475 needs its own binding

Re: [PATCH v2 1/2] dt-bindings: hwmon: add binding documentation for adt7475

2018-11-19 Thread Rob Herring
On Sun, Nov 18, 2018 at 4:56 PM Guenter Roeck wrote: > > On 11/17/18 7:29 AM, Rob Herring wrote: > > On Fri, Nov 09, 2018 at 11:56:06AM +1300, Chris Packham wrote: > >> With the addition of the invert-pwm property the adt7475 needs its own > >> binding documentation rather being captured under

Re: [PATCH 2/2] hwmon: (ntc_thermistor): add support for B57891S0103 from Epcos

2018-11-19 Thread Guenter Roeck
On Mon, Nov 19, 2018 at 09:16:12PM +, Peter Rosin wrote: > > > > Guess I deserve the non-alphabetic order as penalty for not enforcing it > > earlier. I'll accept the patch after DT approval and submit another one > > myself afterwards to restore alphabetic order. > > Right, I'm thinking

Re: [PATCH 2/2] hwmon: (ntc_thermistor): add support for B57891S0103 from Epcos

2018-11-19 Thread Peter Rosin
On 2018-11-19 18:07, Guenter Roeck wrote: > On Sat, Nov 17, 2018 at 12:13:00PM +, Peter Rosin wrote: >> More of the same... >> >> Signed-off-by: Peter Rosin >> --- >> drivers/hwmon/Kconfig| 3 +- >> drivers/hwmon/ntc_thermistor.c | 57 >>

[PATCH] Documentation: hwmon: Add descriptions for ina2xx sysfs entries

2018-11-19 Thread Nicolin Chen
There are a few sysfs entries being exposed to user space by the ina2xx hwmon driver while not getting explicitly documented. So this patch just adds a description section for them. Signed-off-by: Nicolin Chen --- Documentation/hwmon/ina2xx | 15 +++ 1 file changed, 15 insertions(+)

Re: [PATCH] hwmon (ina3221) Add single-shot mode support

2018-11-19 Thread Guenter Roeck
On Fri, Nov 16, 2018 at 05:51:34PM -0800, Nicolin Chen wrote: > Hello Guenter, > > On Wed, Nov 14, 2018 at 09:23:30AM -0800, Guenter Roeck wrote: > > > An alternative way (without the sysfs node), after looking at > > > other hwmon code, could be to have a timed polling thread and > > > read data

Re: [PATCH 2/2] hwmon: (ntc_thermistor): add support for B57891S0103 from Epcos

2018-11-19 Thread Guenter Roeck
On Sat, Nov 17, 2018 at 12:13:00PM +, Peter Rosin wrote: > More of the same... > > Signed-off-by: Peter Rosin > --- > drivers/hwmon/Kconfig| 3 +- > drivers/hwmon/ntc_thermistor.c | 57 > +++- >

Re: [PATCH hwmon-next v1 1/1] hwmon: (mlxreg-fan) Fix macros for tacho fault reading

2018-11-19 Thread Guenter Roeck
On Mon, Nov 19, 2018 at 06:07:40AM +, Vadim Pasternak wrote: > > > > -Original Message- > > From: Guenter Roeck On Behalf Of Guenter Roeck > > Sent: Friday, November 16, 2018 6:15 PM > > To: Vadim Pasternak > > Cc: linux-hwmon@vger.kernel.org; Michael Shych > > Subject: Re: [PATCH