dt-bindings: hwmon: pmbus: add bindings for isl68137
Status from this message: Applied.
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
Earlier versions · 4
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v4 | 2 patches | 0 replies | Superseded | |
| v3 | 2 patches | 3 replies | Superseded | |
| v2 | 2 patches | 5 replies | Superseded | |
| v1 | 2 patches | 2 replies | Superseded |
Changes in v7 from the cover letter
- Port existing device bindings from trivial-bindings.yaml
- Rename new bindings file to isil,isl68137.yaml to match pre-existing bindings
- Style fixes for issues reported by checkpatch --strict
Patches in v7 Applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/2 | hwmon: (pmbus/isl68137) add support for voltage divider on Voutdrivers/ | +205 | 1 reply | |
| 2/2 | dt-bindings: hwmon: isl68137: add bindings to support voltage dividersDocumentation/ Documentation/ | +148 | 2 replies |
Reviews trailers per patch
1 of 2 patches has a Reviewed-by or Acked-by.
| Patch | Rob Herring (Arm) |
|---|---|
| 1/2 | |
| 2/2 | R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v7 0/2] dt-bindings: hwmon: pmbus: add bindings for isl68137
Renesas digital multiphase voltage regulators are capable of regulating output voltages that exceed the range that their Vsense pins can detect. In such applications, users may place a voltage divider between Vout and the Vsense pin for a given rail. However, the driver currently has no way of knowing if a voltage divider is being used which results in erroneous telemetry being reported over hwmon.
This patch set defines a devicetree bindings schema for Renesas digital multiphase voltage regulators that are supported by the isl68137 driver to allow users to add voltage divider definitions for any rail powered by the device. This patch set also includes the required changes to the isl68137 driver to enable scaling Vout/Pout telemetry for rails with a defined voltage divider.
v7:
- Port existing device bindings from trivial-bindings.yaml
- Rename new bindings file to isil,isl68137.yaml to match pre-existing bindings
- Style fixes for issues reported by checkpatch --strict
v6:
- Amend patch commit messages to explain why the vout-voltage-divider property was copied from the max20730 instead of using the iio voltage-divider property
v5:
- Fix clang compilation errors related to C23 syntax
v4:
- Revert devicetree property name to "vout-voltage-divider" and refactor property description and driver implementation to match existing vout-voltage-divider implementation in max20730 as no suitable generic voltage divider schema exists.
- Minor fixes based on Guenter's review of v2.
- Initialize voltage dividers for all channels to defaults that simplify logic in PMBus word read/write functions.
v3:
- Report and return errors reading the vout-voltage-divider property from the devicetree when the property is defined
- Change u64 division/rounding operations to use explicit math64 macros
v2:
- Fix devicetree bindings schema errors
- Add "renesas," vendor prefix to "vout-voltage-divider" property
- Rebase patch series on v6.12-rc1
Grant Peltier (2):
hwmon: (pmbus/isl68137) add support for voltage divider on Vout
dt-bindings: hwmon: isl68137: add bindings to support voltage dividers
.../bindings/hwmon/pmbus/isil,isl68137.yaml | 148 ++++++++++++ .../devicetree/bindings/trivial-devices.yaml | 6 - drivers/hwmon/pmbus/isl68137.c | 210 +++++++++++++++++- 3 files changed, 353 insertions(+), 11 deletions(-) create mode 100644 Documentation/devicetree/bindings/hwmon/pmbus/isil,isl68137.yaml
Signature · 1 line
2.39.5