mbox series

[v7,0/2] hwmon: (isl28022) new driver for ISL28022 power monitor

Message ID 20240925031131.14645-1-yikai.tsai.wiwynn@gmail.com (mailing list archive)
Headers show
Series hwmon: (isl28022) new driver for ISL28022 power monitor | expand

Message

Yikai Tsai Sept. 25, 2024, 3:11 a.m. UTC 
Driver for Renesas ISL28022 power monitor chip.
Found e.g. on Ubiquiti Edgerouter ER-6P

v7: review comments and code refactoring
v6: shunt voltage in mV and revise code
v5: review comments incorporated
v4: property compatible fixed
v3: changelog added
v2: properties reworked
v2: calculations fixed
v2: shunt voltage input moved to debugfs
v2: documentation and devicetree schema reworked
v1: created

Yikai Tsai (2):
  dt-bindings: hwmon: add renesas,isl28022
  hwmon: (isl28022) new driver for ISL28022 power monitor

 .../bindings/hwmon/renesas,isl28022.yaml      |  64 +++
 Documentation/hwmon/index.rst                 |   1 +
 Documentation/hwmon/isl28022.rst              |  63 +++
 MAINTAINERS                                   |   8 +
 drivers/hwmon/Kconfig                         |  11 +
 drivers/hwmon/Makefile                        |   1 +
 drivers/hwmon/isl28022.c                      | 533 ++++++++++++++++++
 7 files changed, 681 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/hwmon/renesas,isl28022.yaml
 create mode 100644 Documentation/hwmon/isl28022.rst
 create mode 100644 drivers/hwmon/isl28022.c

Comments

Krzysztof Kozlowski Sept. 25, 2024, 7:35 a.m. UTC | #1 
On Wed, Sep 25, 2024 at 11:11:26AM +0800, Yikai Tsai wrote:
> Driver for Renesas ISL28022 power monitor chip.
> Found e.g. on Ubiquiti Edgerouter ER-6P
> 
> v7: review comments and code refactoring

What exactly happened? That's too vague.

Best regards,
Krzysztof
Yikai Tsai Oct. 2, 2024, 6:42 a.m. UTC | #2 
Hi Geert, Krzysztof,

Thanks for your feedback. I will fix them.


Best regards,
Yikai

Krzysztof Kozlowski <krzk@kernel.org> 於 2024年9月25日 週三 下午3:35寫道：
>
> On Wed, Sep 25, 2024 at 11:11:26AM +0800, Yikai Tsai wrote:
> > Driver for Renesas ISL28022 power monitor chip.
> > Found e.g. on Ubiquiti Edgerouter ER-6P
> >
> > v7: review comments and code refactoring
>
> What exactly happened? That's too vague.
>
> Best regards,
> Krzysztof
>