commit 351be436af825391c83dc4194ead01c82a6c5347
author Ben Fennema <fennema@google.com> Wed Sep 21 13:29:30 2016 -0700
committer Ben Fennema <fennema@google.com> Wed Sep 21 15:00:44 2016 -0700
tree a6f1bcf9b30103b505fe4768c888439d4b92afbc
parent 06b098c883dca461a2b5ad4357f4ba5abce43eb3

synaptics_s3708: don't double enable or disable interrupts

Bug: 31653505
Change-Id: I0fa30f930009bc99d55af7427861147d4b6f8c02
Signed-off-by: Ben Fennema <fennema@google.com>

firmware/src/drivers/synaptics_s3708/synaptics_s3708.c

diff --git a/firmware/src/drivers/synaptics_s3708/synaptics_s3708.c b/firmware/src/drivers/synaptics_s3708/synaptics_s3708.c
index c858ada..1706a66 100644
--- a/firmware/src/drivers/synaptics_s3708/synaptics_s3708.c
+++ b/firmware/src/drivers/synaptics_s3708/synaptics_s3708.c
@@ -150,17 +150,19 @@
     enum ProxState proxState;
     bool on;
     bool gestureEnabled;
+    bool isrEnabled;
 } mTask;
 
 static inline void enableInterrupt(bool enable)
 {
-    if (enable) {
+    if (!mTask.isrEnabled && enable) {
         extiEnableIntGpio(mTask.pin, EXTI_TRIGGER_FALLING);
         extiChainIsr(TOUCH_IRQ, &mTask.isr);
-    } else {
+    } else if (mTask.isrEnabled && !enable) {
         extiUnchainIsr(TOUCH_IRQ, &mTask.isr);
         extiDisableIntGpio(mTask.pin);
     }
+    mTask.isrEnabled = enable;
 }
 
 static bool touchIsr(struct ChainedIsr *localIsr)