Re: [PATCH 1/2 v3] hwmon: (aspeed-pwm-tacho) reduce fan_tach period

2017-06-26 Thread Joel Stanley
On Sat, Jun 24, 2017 at 11:23 PM, Guenter Roeck wrote: > On 06/23/2017 08:39 PM, Patrick Venture wrote: >> >> Reduce the fan_tach period such that the fan controller uses a shorter >> period to measure the rpm. >> > > This explains what you are doing, but not why. What is the problem you are > tr

Re: [PATCH 1/2 v3] hwmon: (aspeed-pwm-tacho) reduce fan_tach period

2017-06-26 Thread Joel Stanley
On Mon, Jun 26, 2017 at 10:38 PM, Joel Stanley wrote: > I also encourage his efforts as he has spent time looking at the > hardware in detail. And I see that there was a v4 that was merged. Thanks! Joel -- To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in the body of a me

Re: [PATCH 1/2 v3] hwmon: (aspeed-pwm-tacho) reduce fan_tach period

2017-06-26 Thread Guenter Roeck
On 06/26/2017 06:08 AM, Joel Stanley wrote: On Sat, Jun 24, 2017 at 11:23 PM, Guenter Roeck wrote: On 06/23/2017 08:39 PM, Patrick Venture wrote: Reduce the fan_tach period such that the fan controller uses a shorter period to measure the rpm. This explains what you are doing, but not wh

Re: [PATCH 1/2 v3] hwmon: (aspeed-pwm-tacho) reduce fan_tach period

2017-06-26 Thread Joel Stanley
On Mon, Jun 26, 2017 at 11:09 PM, Guenter Roeck wrote: > On 06/26/2017 06:08 AM, Joel Stanley wrote: >> >> On Sat, Jun 24, 2017 at 11:23 PM, Guenter Roeck >> wrote: >>> >>> On 06/23/2017 08:39 PM, Patrick Venture wrote: >> >> Reduce the fan_tach period such that the fan controller uses

userspace regression with hwmon

2017-06-26 Thread Sudip Mukherjee
Hi Guenter, Jean, The patch d72d19c26c41 ("hwmon: (coretemp) Convert to use devm_hwmon_device_register_with_groups") has changed the location of the sysfs nodes from /sys/devices/platform/coretemp.0/ to /sys/devices/platform/coretemp.0/hwmon/hwmon0/ and that has broken some of our usespace applica

[PATCH 0/2] DA9052 hardware monitoring improvements

2017-06-26 Thread Sebastian Reichel
Hi, GE Healthcare's PPD [0] uses DA9053's touchscreen pins for hardware monitoring purposes. This adds support for the feature and fixes a bug, which came up during stress-testing of the driver. [0] https://patchwork.kernel.org/patch/9809681/ -- Sebastian Sebastian Reichel (2): mfd: da9052: f

[PATCH 1/2] mfd: da9052: fix manual ADC read after timed out read

2017-06-26 Thread Sebastian Reichel
It is possible that under heavy system load, the counter in the completion struct, used for waiting for end of AD conversion, gets incremented twice. To make sure the driver recovers from this situation, the completion struct should be reinitialized. Signed-off-by: Sebastian Reichel --- drivers/

[PATCH 2/2] hwmon: da9052: add support for TSI channel

2017-06-26 Thread Sebastian Reichel
TSI channel has a 4 channel mux connected to it and is normally used for touchscreen support. The hardware may alternatively use it as general purpose adc. Signed-off-by: Sebastian Reichel --- .../devicetree/bindings/mfd/da9052-i2c.txt | 8 + drivers/hwmon/da9052-hwmon.c

Re: [PATCH 1/7] drivers/hwmon: Add On-Chip Controller (OCC) hwmon driver

2017-06-26 Thread Eddie James
On 06/24/2017 09:15 AM, Guenter Roeck wrote: On 06/22/2017 03:48 PM, Eddie James wrote: From: "Edward A. James" The OCC is a device embedded on a POWER processor that collects and aggregates sensor data from the processor and system. The OCC can provide the raw sensor data as well as perform

Re: [PATCH 2/7] drivers/hwmon/occ: Add command transport method for P8 and P9

2017-06-26 Thread Eddie James
On 06/24/2017 09:49 AM, Guenter Roeck wrote: On 06/22/2017 03:48 PM, Eddie James wrote: From: "Edward A. James" For the P8 OCC, add the procedure to send a command to the OCC over I2C bus. This involves writing the OCC command registers with serial communication operations (SCOMs) interprete

Re: [PATCH 1/7] drivers/hwmon: Add On-Chip Controller (OCC) hwmon driver

2017-06-26 Thread Guenter Roeck
On Mon, Jun 26, 2017 at 10:06:50AM -0500, Eddie James wrote: > > > On 06/24/2017 09:15 AM, Guenter Roeck wrote: > >On 06/22/2017 03:48 PM, Eddie James wrote: > >>From: "Edward A. James" > >> > >>The OCC is a device embedded on a POWER processor that collects and > >>aggregates sensor data from t

Re: [PATCH 2/7] drivers/hwmon/occ: Add command transport method for P8 and P9

2017-06-26 Thread Guenter Roeck
On Mon, Jun 26, 2017 at 10:30:28AM -0500, Eddie James wrote: > > > On 06/24/2017 09:49 AM, Guenter Roeck wrote: > >On 06/22/2017 03:48 PM, Eddie James wrote: > >>From: "Edward A. James" > >> > >>For the P8 OCC, add the procedure to send a command to the OCC over I2C > >>bus. This involves writin

Re: userspace regression with hwmon

2017-06-26 Thread Guenter Roeck
On Mon, Jun 26, 2017 at 03:33:20PM +0100, Sudip Mukherjee wrote: > Hi Guenter, Jean, > > The patch d72d19c26c41 ("hwmon: (coretemp) Convert to use > devm_hwmon_device_register_with_groups") has changed the location of the > sysfs nodes from /sys/devices/platform/coretemp.0/ to > /sys/devices/platf

Re: [PATCH 1/7] drivers/hwmon: Add On-Chip Controller (OCC) hwmon driver

2017-06-26 Thread Rob Herring
On Thu, Jun 22, 2017 at 05:48:30PM -0500, Eddie James wrote: > From: "Edward A. James" > > The OCC is a device embedded on a POWER processor that collects and > aggregates sensor data from the processor and system. The OCC can > provide the raw sensor data as well as perform thermal and power > m

Re: [PATCH 2/2] hwmon: da9052: add support for TSI channel

2017-06-26 Thread Guenter Roeck
On Mon, Jun 26, 2017 at 04:48:32PM +0200, Sebastian Reichel wrote: > TSI channel has a 4 channel mux connected to it and is normally > used for touchscreen support. The hardware may alternatively > use it as general purpose adc. > > Signed-off-by: Sebastian Reichel > --- > .../devicetree/binding