Re: [PATCH 3/3] hwmon: (pmbus/ltc2978): add support for more parts.

From: Rob Herring
Date: Thu Feb 06 2020 - 15:45:39 EST


On Tue, Jan 28, 2020 at 11:00:00AM -0700, Mike Jones wrote:
> LTC2972, LTC2979, LTC3884, LTC3889, LTC7880, LTM4664, LTM4677,
> LTM4678, LTM4680, LTM4700.
>
> Signed-off-by: Mike Jones <michael-a1.jones@xxxxxxxxxx>
> ---
> .../devicetree/bindings/hwmon/ltc2978.txt | 22 ++-

DT bindings should be a separate patch.

> Documentation/hwmon/ltc2978.rst | 164 ++++++++++++++++-----
> drivers/hwmon/pmbus/Kconfig | 11 +-
> drivers/hwmon/pmbus/ltc2978.c | 92 +++++++++++-
> 4 files changed, 238 insertions(+), 51 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/hwmon/ltc2978.txt b/Documentation/devicetree/bindings/hwmon/ltc2978.txt
> index b428a70..4e7f621 100644
> --- a/Documentation/devicetree/bindings/hwmon/ltc2978.txt
> +++ b/Documentation/devicetree/bindings/hwmon/ltc2978.txt
> @@ -2,20 +2,30 @@ ltc2978
>
> Required properties:
> - compatible: should contain one of:
> + * "lltc,ltc2972"
> * "lltc,ltc2974"
> * "lltc,ltc2975"
> * "lltc,ltc2977"
> * "lltc,ltc2978"
> + * "lltc,ltc2979"
> * "lltc,ltc2980"
> * "lltc,ltc3880"
> * "lltc,ltc3882"
> * "lltc,ltc3883"
> + * "lltc,ltc3884"
> * "lltc,ltc3886"
> * "lltc,ltc3887"
> + * "lltc,ltc3889"
> + * "lltc,ltc7880"
> * "lltc,ltm2987"
> + * "lltc,ltm4664"
> * "lltc,ltm4675"
> * "lltc,ltm4676"
> + * "lltc,ltm4677"
> + * "lltc,ltm4678"
> + * "lltc,ltm4680"
> * "lltc,ltm4686"
> + * "lltc,ltm4700"
> - reg: I2C slave address
>
> Optional properties:
> @@ -25,13 +35,17 @@ Optional properties:
> standard binding for regulators; see regulator.txt.
>
> Valid names of regulators depend on number of supplies supported per device:
> + * ltc2972 vout0 - vout1
> * ltc2974, ltc2975 : vout0 - vout3
> - * ltc2977, ltc2980, ltm2987 : vout0 - vout7
> + * ltc2977, ltc2979, ltc2980, ltm2987 : vout0 - vout7
> * ltc2978 : vout0 - vout7
> - * ltc3880, ltc3882, ltc3886 : vout0 - vout1
> + * ltc3880, ltc3882, ltc3884, ltc3886, ltc3887, ltc3889 : vout0 - vout1
> + * ltc7880 : vout0 - vout1
> * ltc3883 : vout0
> - * ltm4676 : vout0 - vout1
> - * ltm4686 : vout0 - vout1
> + * ltm4664 : vout0 - vout1
> + * ltm4675, ltm4676, ltm4677, ltm4678 : vout0 - vout1
> + * ltm4680, ltm4686 : vout0 - vout1
> + * ltm4700 : vout0 - vout1
>
> Example:
> ltc2978@5e {