Hi Grant,
On Fri, Oct 25, 2024 at 7:16 PM Grant Peltier <grantpeltier93@xxxxxxxxx> wrote:
> 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.
>
> v5:
> - Fix clang compilation errors related to C23 syntax
Thanks for the update!
> 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.
Can you please elaborate (or point to the email that did so, in case
I missed it)?
In reply to v2, Günter wrote:
"I would prefer, in the order of preference,
1) an applicable generic property definition
2) a definition that is already used elsewhere
3) a new chips specific definition"
https://lore.kernel.org/all/3f460b62-4cd1-49dd-a98b-1fbcfdbd3af0@xxxxxxxxxxxx
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
