From 3ec5362a3632c43ac005f566d95268eb120faa21 Mon Sep 17 00:00:00 2001
From: kegman <8837066+kegman@users.noreply.github.com>
Date: Tue, 24 Dec 2019 23:38:28 -0500
Subject: [PATCH] Revert "GM fix 'Controls Failed' on resuming after cycling
 main cruise switch (#255)"

This reverts commit 78b629a97c8a80efcf199fc01b0d8146979a1d28.
---
 cereal/car.capnp                        | 1 -
 selfdrive/car/gm/interface.py           | 9 +--------
 selfdrive/controls/controlsd.py         | 4 ----
 selfdrive/controls/lib/drive_helpers.py | 1 -
 4 files changed, 1 insertion(+), 14 deletions(-)

diff --git a/cereal/car.capnp b/cereal/car.capnp
index 4a47ec8a0c4d8f..b3953ac2b50086 100644
--- a/cereal/car.capnp
+++ b/cereal/car.capnp
@@ -19,7 +19,6 @@ struct CarEvent @0x9b1657f34caf3ad3 {
   immediateDisable @6 :Bool;
   preEnable @7 :Bool;
   permanent @8 :Bool;
-  resetVCruise @9 :Bool;
 
   enum EventName @0xbaa8c5d505f727de {
     # TODO: copy from error list
diff --git a/selfdrive/car/gm/interface.py b/selfdrive/car/gm/interface.py
index a227b367bde037..13dd2a4e38f719 100755
--- a/selfdrive/car/gm/interface.py
+++ b/selfdrive/car/gm/interface.py
@@ -377,18 +377,11 @@ def update(self, c, can_strings):
       # handle button presses
       for b in ret.buttonEvents:
         # do enable on both accel and decel buttons
-        # The ECM will fault if resume triggers an enable while speed is set to 0
-        if b.type == ButtonType.accelCruise and c.hudControl.setSpeed > 0 and c.hudControl.setSpeed < 70 and not b.pressed:
-          events.append(create_event('buttonEnable', [ET.ENABLE]))
-        if b.type == ButtonType.decelCruise and not b.pressed:
+        if b.type in [ButtonType.accelCruise, ButtonType.decelCruise] and not b.pressed:
           events.append(create_event('buttonEnable', [ET.ENABLE]))
         # do disable on button down
         if b.type == ButtonType.cancel and b.pressed:
           events.append(create_event('buttonCancel', [ET.USER_DISABLE]))
-        # The ECM independently tracks a ‘speed is set’ state that is reset on main off.
-        # To keep controlsd in sync with the ECM state, generate a RESET_V_CRUISE event on main cruise presses.
-        if b.type == ButtonType.altButton3 and b.pressed:
-          events.append(create_event('buttonCancel', [ET.RESET_V_CRUISE, ET.USER_DISABLE]))
 
     ret.events = events
 
diff --git a/selfdrive/controls/controlsd.py b/selfdrive/controls/controlsd.py
index 321389d7b16211..f6a27902bd5f20 100755
--- a/selfdrive/controls/controlsd.py
+++ b/selfdrive/controls/controlsd.py
@@ -165,10 +165,6 @@ def state_transition(frame, CS, CP, state, events, soft_disable_timer, v_cruise_
   # decrease the soft disable timer at every step, as it's reset on
   # entrance in SOFT_DISABLING state
   soft_disable_timer = max(0, soft_disable_timer - 1)
-  
-  # Reset v_cruise_kph to 0
-  if get_events(events, [ET.RESET_V_CRUISE]):
-    v_cruise_kph = 0
 
   # DISABLED
   if state == State.disabled:
diff --git a/selfdrive/controls/lib/drive_helpers.py b/selfdrive/controls/lib/drive_helpers.py
index f8d432e186435f..a9e809a873b84e 100644
--- a/selfdrive/controls/lib/drive_helpers.py
+++ b/selfdrive/controls/lib/drive_helpers.py
@@ -35,7 +35,6 @@ class EventTypes:
   SOFT_DISABLE = 'softDisable'
   IMMEDIATE_DISABLE = 'immediateDisable'
   PERMANENT = 'permanent'
-  RESET_V_CRUISE = 'resetVCruise'
 
 
 def create_event(name, types):