commit 4719abbcf36ca7a584d592da017586d07033fd8b
author Roman Stratiienko <r.stratiienko@gmail.com> Wed Dec 28 18:51:37 2022 +0200
committer Roman Stratiienko <r.stratiienko@gmail.com> Wed Dec 28 18:56:03 2022 +0200
tree 24f63796d6dfa000b3ca8be04f920f6061e40f46
parent 3295719124162e5091168173064fedf1cd28f5d3

drm_hwcomposer: Rework UEventListener to work without utils/worker

utils/worker just complicates the logic without providing any benefit.

Change-Id: I0e6427492b8298c6c327c1edb660c8a9789f706f
Signed-off-by: Roman Stratiienko <r.stratiienko@gmail.com>

drm/ResourceManager.cpp

diff --git a/drm/ResourceManager.cpp b/drm/ResourceManager.cpp
index fb5c32b..f64cae4 100644
--- a/drm/ResourceManager.cpp
+++ b/drm/ResourceManager.cpp
@@ -36,13 +36,7 @@
 ResourceManager::ResourceManager(
     PipelineToFrontendBindingInterface *p2f_bind_interface)
     : frontend_interface_(p2f_bind_interface) {
-  if (uevent_listener_.Init() != 0) {
-    ALOGE("Can't initialize event listener");
-  }
-}
-
-ResourceManager::~ResourceManager() {
-  uevent_listener_.Exit();
+  uevent_listener_ = UEventListener::CreateInstance();
 }
 
 void ResourceManager::Init() {
@@ -87,7 +81,7 @@
     return;
   }
 
-  uevent_listener_.RegisterHotplugHandler([this] {
+  uevent_listener_->RegisterHotplugHandler([this] {
     const std::lock_guard<std::mutex> lock(GetMainLock());
     UpdateFrontendDisplays();
   });
@@ -103,7 +97,7 @@
     return;
   }
 
-  uevent_listener_.RegisterHotplugHandler([] {});
+  uevent_listener_->RegisterHotplugHandler({});
 
   DetachAllFrontendDisplays();
   drms_.clear();