On 09/01/2024 13:19, Nuno Sá wrote: > On Mon, 2023-12-18 at 17:29 +0100, Nuno Sa wrote: >> v1: >> * >> https://lore.kernel.org/linux-hwmon/20231110151905.1659873-1-nuno.sa@xxxxxxxxxx/ >> >> v2: >> * >> https://lore.kernel.org/linux-hwmon/20231124-ltc4282-support-v2-0-952bf926f83c@xxxxxxxxxx >> >> v3: >> * >> https://lore.kernel.org/r/20231205-ltc4282-support-v3-0-e0877b281bc2@xxxxxxxxxx >> >> 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 :) Merge window is open. Is it a critical fix? Does not look like, so don't ping, it's pointless and only adds unnecessary traffic. Best regards, Krzysztof
