mbox series

[v2,0/2] Add MPS MP5920 Host-Swap controller

Message ID 20240627090113.391730-1-xzeol@yahoo.com
Headers show
Series Add MPS MP5920 Host-Swap controller | expand

Message

Alex Vdovydchenko June 27, 2024, 9:01 a.m. UTC 
This series of patches adds the MP5920 Host-swap controller, which is used
as a protection and control IC for devices that are being inserted into a live
backplane. MP5920 acts as a voltage regulator (MP5911 etc) supervisor. IC
utilizes pmbus and provides monitoring, statistics and limits to electrical and
thermal characteristics such as:
- input and output voltage
- output current
- output power
- IC temperature

One must take into account the nonlinear character of readings, so there will be
a statistical error in the range 5–10 percents, depending on current passing
through. In order to use the IC, make sure to specify a valid I2C address
(consult to datasheet and dts-bindings)
MP5920 datasheet: https://www.monolithicpower.com/en/mp5920.html

Changes in v2:
  -  fixed typos

Alex Vdovydchenko (2):
  dt-bindings: hwmon: Add MPS mp5920
  hwmon: add MP5920 driver

 .../devicetree/bindings/trivial-devices.yaml  |  2 +
 Documentation/hwmon/index.rst                 |  1 +
 Documentation/hwmon/mp5920.rst                | 91 ++++++++++++++++++
 drivers/hwmon/pmbus/Kconfig                   |  9 ++
 drivers/hwmon/pmbus/Makefile                  |  1 +
 drivers/hwmon/pmbus/mp5920.c                  | 93 +++++++++++++++++++
 6 files changed, 197 insertions(+)
 create mode 100644 Documentation/hwmon/mp5920.rst
 create mode 100644 drivers/hwmon/pmbus/mp5920.c

Comments

Guenter Roeck June 27, 2024, 2:30 p.m. UTC | #1 
On 6/27/24 02:01, Alex Vdovydchenko wrote:
> This series of patches adds the MP5920 Host-swap controller, which is used
> as a protection and control IC for devices that are being inserted into a live
> backplane. MP5920 acts as a voltage regulator (MP5911 etc) supervisor. IC
> utilizes pmbus and provides monitoring, statistics and limits to electrical and
> thermal characteristics such as:
> - input and output voltage
> - output current
> - output power
> - IC temperature
> 
> One must take into account the nonlinear character of readings, so there will be
> a statistical error in the range 5–10 percents, depending on current passing
> through. In order to use the IC, make sure to specify a valid I2C address
> (consult to datasheet and dts-bindings)
> MP5920 datasheet: https://www.monolithicpower.com/en/mp5920.html
> 
> Changes in v2:
>    -  fixed typos

... and ignored all other feedback without explaining the reasons.

I am not even going to review this version.

Guenter