From 9f356f88cce50b730382e8775e7a39e557b09aea Mon Sep 17 00:00:00 2001
From: Jeremy Soller <jeremy@system76.com>
Date: Tue, 19 Nov 2019 11:38:59 -0700
Subject: [PATCH] Send battery charger config repeatedly

---
 src/board/system76/galp3-c/include/board/battery.h |  4 ++--
 src/board/system76/galp3-c/main.c                  | 10 ++++++++--
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/board/system76/galp3-c/include/board/battery.h b/src/board/system76/galp3-c/include/board/battery.h
index 3d83562..8e57bd4 100644
--- a/src/board/system76/galp3-c/include/board/battery.h
+++ b/src/board/system76/galp3-c/include/board/battery.h
@@ -8,8 +8,8 @@ extern uint16_t battery_voltage;
 extern uint16_t battery_current;
 extern uint16_t battery_charge;
 
-void battery_charger_disable(void);
-void battery_charger_enable(void);
+int battery_charger_disable(void);
+int battery_charger_enable(void);
 void battery_event(void);
 void battery_debug(void);
 
diff --git a/src/board/system76/galp3-c/main.c b/src/board/system76/galp3-c/main.c
index eaf0bea..b76e05a 100644
--- a/src/board/system76/galp3-c/main.c
+++ b/src/board/system76/galp3-c/main.c
@@ -66,15 +66,21 @@ void ac_adapter() {
     // Set ACIN LED
     gpio_set(&LED_ACIN, !new);
 
+    // Battery charger disables by default after timeout, make sure it knows
+    // we are still alive by setting charge values repeatedly
+    if (new) {
+        battery_charger_disable();
+    } else {
+        battery_charger_enable();
+    }
+
     // If there has been a change, print
     if (new != last) {
         DEBUG("Power adapter ");
         if (new) {
             DEBUG("unplugged\n");
-            battery_charger_disable();
         } else {
             DEBUG("plugged in\n");
-            battery_charger_enable();
         }
         battery_debug();
     }