Re: [PATCH v4 0/3] Add support for LTC4282

["Nuno Sá" <noname.nuno@gmail.com>]

On Mon, 2023-12-18 at 17:29 +0100, Nuno Sa wrote:
> v1:
>  *
> https://lore.kernel.org/linux-hwmon/20231110151905.1659873-1-nuno.sa@analog.com/
> 
> v2:
>  *
> https://lore.kernel.org/linux-hwmon/20231124-ltc4282-support-v2-0-952bf926f83c@analog.com
> 
> v3:
>  *
> https://lore.kernel.org/r/20231205-ltc4282-support-v3-0-e0877b281bc2@analog.com
> 
> Changes in v4:
> - Patch 1:
>  * New patch. Support fault attributes in voltage channels.
> - Patch 2:
>  * Add default values for gpios and divider properties;
>  * Add adi,gpio3-monitor-enable property.
> - Patch 3:
>  - Docs:
>   * Document that fault logs are also cleared when writing in reset_history
>     attributes;
>   * Document debugfs entries;
>   * Add new in0_fault attributes and remove dropped ones.
>  - Driver:
>   * Add hwmon_in_fault attribute to report FET failures in VSOURCE;
>   * Clear fault logs in reset_history;
>   * Constify 'ltc4282_out_rates';
>   * Add missing error check in ltc4282_cache_history();
>   * Removed unused functions;
>   * Renamed clk provider name so it's unique per device;
>   * Support new adi,gpio3-monitor-enable property;
>   * Dropped power1_good, fet_bad_fault, fet_short_fault, fault_logs_reset
>     custom attributes. Note that only power1_good was really dropped.
>     The other ones are supported in standard ABI.
>   * Renamed debugfs directory for ltc4282-hwmonX;
>   * Added in0 prefix to FET fault logs so it's clear they affect VSOURCE;
>   * Fix in_range() condition (false means error);
>   * Fix reset_history attributes. We should not write 0 in the lowest
>     value. Write the theoretical max value in there. For vsource/vdd,
>     also do it during device setup (or we would end up with 0).
>   * Directly store the chip vdd instead of vin_mode in our device
>     structure. Easier to handle reset_history;
>   * Moved the vin_mode enum to reduce it's scope.
> 
> As mentioned in v3 discussion, clearing the power bad fault log has no
> effect but I'm still doing it for consistency and because we also allow
> to read it in debugfs (so better allow to clear it as well)
> 
> ---
> Nuno Sa (3):
>       dt-bindings: hwmon: Add LTC4282 bindings
>       hwmon: add fault attribute for voltage channels
>       hwmon: ltc4282: add support for the LTC4282 chip
> 
>  Documentation/ABI/testing/sysfs-class-hwmon        |    9 +
>  .../devicetree/bindings/hwmon/adi,ltc4282.yaml     |  159 ++
>  Documentation/hwmon/index.rst                      |    1 +
>  Documentation/hwmon/ltc4282.rst                    |  133 ++
>  MAINTAINERS                                        |    8 +
>  drivers/hwmon/Kconfig                              |   11 +
>  drivers/hwmon/Makefile                             |    1 +
>  drivers/hwmon/hwmon.c                              |    1 +
>  drivers/hwmon/ltc4282.c                            | 1784
> ++++++++++++++++++++
>  include/linux/hwmon.h                              |    2 +
>  10 files changed, 2109 insertions(+)
> 
> Thanks!
> - Nuno Sá
> 

Hi Guenter,

Just pinging this one. Not sure if you missed this one or just low priority in
your queue :)

- Nuno Sá