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

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

 



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.


--
Regards,
Ahmad Khalifa



[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