lkml.org 
[lkml]   [2014]   [Oct]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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



\
 
 \ /
  Last update: 2014-10-02 04:01    [W:0.054 / U:0.656 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site