mbox series

[0/2] Add driver for LTP8800-1A, LTP8800-2 and LTP8800-4A

Message ID 20241029130137.31284-1-cedricjustine.encarnacion@analog.com
Headers show
Series Add driver for LTP8800-1A, LTP8800-2 and LTP8800-4A | expand

Message

Cedric Encarnacion Oct. 29, 2024, 1:01 p.m. UTC 
The LTP8800-1A/-2/-4A is a 150A/135A/200A step-down μModule regulator that
provides microprocessor core voltage from 54V power distribution
architecture. It features remote configurability and telemetry monitoring
of power management parameters over PMBus. This patch series adds driver
and bindings for these devices.

Cedric Encarnacion (2):
  dt-bindings: trivial-devices: add ltp8800
  hwmon: pmbus: add driver for ltp8800-1a, ltp8800-4a, and ltp8800-2

 .../devicetree/bindings/trivial-devices.yaml  |   6 +
 Documentation/hwmon/index.rst                 |   1 +
 Documentation/hwmon/ltp8800.rst               | 103 ++++++++++++++++++
 MAINTAINERS                                   |   7 ++
 drivers/hwmon/pmbus/Kconfig                   |  18 +++
 drivers/hwmon/pmbus/Makefile                  |   1 +
 drivers/hwmon/pmbus/ltp8800.c                 |  74 +++++++++++++
 7 files changed, 210 insertions(+)
 create mode 100644 Documentation/hwmon/ltp8800.rst
 create mode 100644 drivers/hwmon/pmbus/ltp8800.c


base-commit: 893acae4cdaf34fdde0e7285d5922af70775e5f7

Comments

Guenter Roeck Oct. 29, 2024, 2:04 p.m. UTC | #1 
On 10/29/24 06:01, Cedric Encarnacion wrote:
> Add Analog Devices LTP8800-1A, LTP8800-2, and LTP8800-4A DC/DC μModule
> regulator.
> 
> Signed-off-by: Cedric Encarnacion <cedricjustine.encarnacion@analog.com>
> ---
>   Documentation/devicetree/bindings/trivial-devices.yaml | 6 ++++++
>   MAINTAINERS                                            | 5 +++++
>   2 files changed, 11 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
> index bd784f6bb38b..efd959708c75 100644
> --- a/Documentation/devicetree/bindings/trivial-devices.yaml
> +++ b/Documentation/devicetree/bindings/trivial-devices.yaml
> @@ -43,6 +43,12 @@ properties:
>             - adi,adp5589
>               # Analog Devices LT7182S Dual Channel 6A, 20V PolyPhase Step-Down Silent Switcher
>             - adi,lt7182s
> +            # Analog Devices LTP8800-1A 150A, 54V DC/DC μModule regulator
> +          - adi,ltp8800-1a
> +            # Analog Devices LTP8800-2 135A, 54V DC/DC μModule regulator
> +          - adi,ltp8800-2
> +            # Analog Devices LTP8800-4A 200A, 54V DC/DC μModule regulator
> +          - adi,ltp8800-4a

I don't immediately see why this chip would warrant three entries instead of just
"adi,ltp8800". Programming for all chips is the same. The only difference is
the power rating.

Guenter

>               # AMS iAQ-Core VOC Sensor
>             - ams,iaq-core
>               # Temperature monitoring of Astera Labs PT5161L PCIe retimer
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ca7e827da33e..a6abf7243b94 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -13555,6 +13555,11 @@ S:	Maintained
>   F:	Documentation/devicetree/bindings/iio/light/liteon,ltr390.yaml
>   F:	drivers/iio/light/ltr390.c
>   
> +LTP8800 HARDWARE MONITOR DRIVER
> +M:	Cedric Encarnacion <cedricjustine.encarnacion@analog.com>
> +L:	linux-hwmon@vger.kernel.org
> +S:	Supported
> +
>   LYNX 28G SERDES PHY DRIVER
>   M:	Ioana Ciornei <ioana.ciornei@nxp.com>
>   L:	netdev@vger.kernel.org