On Wed, 2023-12-13 at 06:35 -0800, Guenter Roeck wrote: > On 12/13/23 02:06, Nuno Sá wrote: > [ ... ] > > > > > > We could also use in[01]_reset_history. While that is originally intended to > > > reset > > > min/max voltage history, I think it makes sense to extend the meaning to > > > include > > > fault history (even more so if that history includes over/undervoltage events). > > > Plus, there are other _reset_history attributes which could be used to reset > > > power/current/temperature history separately if that is supported by the chip. > > > > > > > Well, I'm already supporting _reset_history in the voltage/power/current channels > > so > > I can easily extend that for clearing fault history if that is fine with you. I > > just > > need to document it because it's a bit of an "hidden" thing. The question would > > also > > be, should I just document this for this chip docs or in the general hwmon docs? > > > > I'd say chip specific for now. We can document it in the general documentation > if/when we get more chips with the same characteristics. > Agreed... Alright, I think I have now all I need to work on v4. Thanks! - Nuno Sá
