From 6e817f9eb5d85d5f27b72c1abc16a85673ae3d01 Mon Sep 17 00:00:00 2001
From: Shane Smiskol <shane@smiskol.com>
Date: Tue, 11 Jan 2022 06:07:35 -0700
Subject: [PATCH] manager: default driverview to False (#23492)

---
 selfdrive/manager/process.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/selfdrive/manager/process.py b/selfdrive/manager/process.py
index a423b1ff84..2dcb42f339 100644
--- a/selfdrive/manager/process.py
+++ b/selfdrive/manager/process.py
@@ -67,6 +67,7 @@ class ManagerProcess(ABC):
   daemon = False
   sigkill = False
   persistent = False
+  driverview = False
   proc: Optional[Process] = None
   enabled = True
   name = ""
@@ -291,8 +292,7 @@ def ensure_running(procs: ValuesView[ManagerProcess], started: bool, driverview:
       p.stop(block=False)
     elif p.persistent:
       p.start()
-    elif getattr(p, 'driverview', False) and driverview:
-      # TODO: why is driverview an argument here? can this be done with the name?
+    elif p.driverview and driverview:
       p.start()
     elif started:
       p.start()