Re: [PATCH] hwmon: (nct6775) Flip Caseopen alarm as 1 is OK

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 7/15/23 09:43, Ahmad Khalifa wrote:
On 15/07/2023 17:33, Guenter Roeck wrote:
On 7/15/23 08:32, Ahmad Khalifa wrote:
+    /* nc6796d-s datasheet states: CASEOPEN0_STS. CaseOpen Status.
+     * 1: Caseopen0 is detected and latched.
+     * 0: Caseopen0 is not latched.
+     * so we flip intrusion alarms, 1 is OK
+     */
Ok, I am lost here. 1 means that case open is detected and latched. That
is no different to all other chips of the series. Why would you want
to return 0 (no alarm) in this case for nc6796d-s ?

I read it as 'latched' vs 'not latched', but if it's the same for all
of them, then I take this patch back.
My mistake.


The chips latch the case open signal, so once case open is detected
it has to be reset by writing into the attribute. The rest (latched
vs. not latched) is just wording; "not latched" just means that
case open was not detected since the bit was last reset.

Guenter




[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux