Re: [PATCH v5 3/4] pmbus (core): Add virtual page config bit

2017-11-19 Thread Guenter Roeck
On 11/17/2017 07:26 PM, Andrew Jeffery wrote: Some circumstances call for virtual pages, to expose multiple values packed into an extended PMBus register in a manner non-compliant with the PMBus standard. An example of this is the Maxim MAX31785 controller, which extends the READ_FAN_SPEED_1 PMBu

[PATCH] hwmon: Drop unnecessary 'default n' from Kconfig

2017-11-19 Thread Guenter Roeck
'default n' is default, so there is no need to specify it explicitly. Signed-off-by: Guenter Roeck --- drivers/hwmon/Kconfig | 16 1 file changed, 16 deletions(-) diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig index 530ff7c9234c..ef23553ff5cb 100644 --- a/drivers/hw

Re: [PATCH] hwmon: Drop unnecessary 'default n' from Kconfig

2017-11-19 Thread Jean Delvare
Hi Guenter, On Sun, 19 Nov 2017 09:16:02 -0800, Guenter Roeck wrote: > 'default n' is default, so there is no need to specify it explicitly. > > Signed-off-by: Guenter Roeck > --- > drivers/hwmon/Kconfig | 16 > 1 file changed, 16 deletions(-) > (...) Reviewed-by: Jean Delvare

Re: [PATCH v5 3/4] pmbus (core): Add virtual page config bit

2017-11-19 Thread Andrew Jeffery
On Sun, 2017-11-19 at 08:49 -0800, Guenter Roeck wrote: > On 11/17/2017 07:26 PM, Andrew Jeffery wrote: > > Some circumstances call for virtual pages, to expose multiple values > > packed into an extended PMBus register in a manner non-compliant with > > the PMBus standard. An example of this is th

[PATCH v6 0/4] pmbus: Expand fan support and add MAX31785 driver

2017-11-19 Thread Andrew Jeffery
Hello, This series introduces support for the MAX31785 intelligent fan controller, a PMBus device providing closed-loop fan control among a number of other features. Along the way the series adds support to control fans and create virtual pages to the PMBus core, the latter to support some of the

[PATCH v6 4/4] pmbus (max31785): Add dual tachometer support

2017-11-19 Thread Andrew Jeffery
The dual tachometer feature is implemented in hardware with a TACHSEL input to indicate the rotor under measurement, and exposed on the device by extending the READ_FAN_SPEED_1 word with two extra bytes*. The need to read the non-standard four-byte response leads to a cut-down implementation of i2c

[PATCH v6 1/4] pmbus (core): Add fan control support

2017-11-19 Thread Andrew Jeffery
Expose fanX_target, pwmX and pwmX_enable hwmon sysfs attributes. Fans in a PMBus device are driven by the configuration of two registers, FAN_CONFIG_x_y and FAN_COMMAND_x: FAN_CONFIG_x_y dictates how the fan and the tacho operate (if installed), while FAN_COMMAND_x sets the desired fan rate. The u

[PATCH v6 3/4] pmbus (core): Add virtual page config bit

2017-11-19 Thread Andrew Jeffery
Some circumstances call for virtual pages, to expose multiple values packed into an extended PMBus register in a manner non-compliant with the PMBus standard. An example of this is the Maxim MAX31785 controller, which extends the READ_FAN_SPEED_1 PMBus register from two to four bytes to support tac

[PATCH v6 2/4] pmbus (max31785): Add fan control

2017-11-19 Thread Andrew Jeffery
The implementation makes use of the new fan control virtual registers exposed by the pmbus core. It mixes use of the default implementations with some overrides via the read/write handlers to handle FAN_COMMAND_1 on the MAX31785, whose definition breaks the value range into various control bands de