Re: [PATCH 5/7] hwmon: Add driver for Marvel AlphaServer environmental sensors

[Guenter Roeck]

On 11/14/26 18:50, Matt Turner wrote:
...
> > > +``in[0-*]_alarm``           Sensor is in the warning or fault state
> >
> > Which one is it ? Is it distinct ? A "fault" means
> > that the hardware is broken/defective. An alarm means that
> > the voltage is too high or too low. A faulty channel
> > should not be reported as alarm but as fault.
>
> It is always the voltage being out of range: Warning is the reading
> outside the warning limits (5% from nominal), Failure outside the
> failure limits (10%). Both are reported as in_alarm in v2. The limits
> are not in the console's packet, so they are not exported.

This could be reported as inX_max_alarm and inX_crit_alarm.

Thanks,
Guenter