Messages in this thread | | | Date | Wed, 01 Oct 2014 18:26:17 -0700 | From | Guenter Roeck <> | Subject | Re: [PATCH v4 4/4] pmbus: ltc2978: add regulator support |
| |
On 10/01/2014 03:20 PM, atull wrote: > On Wed, 1 Oct 2014, Guenter Roeck wrote: > >> On Wed, Oct 01, 2014 at 03:18:20PM -0500, atull@opensource.altera.com wrote: >>> From: Alan Tull <atull@opensource.altera.com> >>> >>> Add simple on/off regulator support for ltc2978 and >>> other pmbus parts supported by ltc2978.c >>> >>> Signed-off-by: Alan Tull <atull@opensource.altera.com> >>> >>> v2: Remove '#include <linux/regulator/machine.h>' >>> Only one regulator per pmbus device >>> Get regulator_init_data from pdata or device tree >>> >>> v3: Support multiple regulators for each chip >>> Move most code to pmbus_core.c >>> fixed values for on/off >>> >>> v4: fix a #endif comment >>> simplify probe code, remove added switch statement >>> remove BUG_ON(), add error message and fix num_regulators >>> --- >>> drivers/hwmon/pmbus/Kconfig | 7 +++++++ >>> drivers/hwmon/pmbus/ltc2978.c | 37 +++++++++++++++++++++++++++++++++++++ >>> 2 files changed, 44 insertions(+) >>> >>> diff --git a/drivers/hwmon/pmbus/Kconfig b/drivers/hwmon/pmbus/Kconfig >>> index 6e1e493..79117b7 100644 >>> --- a/drivers/hwmon/pmbus/Kconfig >>> +++ b/drivers/hwmon/pmbus/Kconfig >>> @@ -56,6 +56,13 @@ config SENSORS_LTC2978 >>> This driver can also be built as a module. If so, the module will >>> be called ltc2978. >>> >>> +config SENSORS_LTC2978_REGULATOR >>> + boolean "Regulator support for LTC2974, LTC2978, LTC3880, and LTC3883" >> >> We will need to update this and SENSORS_LTC2978 and add LTC2977 as well as >> LTM4646 to the list of supported chips. >> >> Guenter > > I will add the whole list to the 'help' sections of SENSORS_LTC2978 and > SENSORS_LTC2978_REGULATOR. > > For the tristate/boolean lines, I can't fit the whole list, so I will > leave them as they are. Or I could change to 'LTC2978 and compatibles' as > many others in this Kconfig have. > Yes, that makes sense.
Thanks, Guenter
| |