commit 753d107896e2129d255e1c7f5fdaf6f25251dd15
author Roman Stratiienko <roman.o.stratiienko@globallogic.com> Thu Dec 30 18:05:27 2021 +0200
committer Roman Stratiienko <roman.o.stratiienko@globallogic.com> Wed Jan 12 10:33:55 2022 +0200
tree f30bfab94deece49fb6b8e92167f995671551769
parent fc014f5792053a3a5712d1f6f65d55994dce24f4

drm_hwcomposer: Make Planner class fully static

clang-tidy suggested to make all methods of Planner static,
since they don't use any instance members.

This makes creation of Planner instances unnecessary, therefore
remove all such cases.

Signed-off-by: Roman Stratiienko <roman.o.stratiienko@globallogic.com>

DrmHwcTwo.cpp

diff --git a/DrmHwcTwo.cpp b/DrmHwcTwo.cpp
index fe4c2ad..ad4936b 100644
--- a/DrmHwcTwo.cpp
+++ b/DrmHwcTwo.cpp
@@ -240,12 +240,6 @@
 }
 
 HWC2::Error DrmHwcTwo::HwcDisplay::Init(std::vector<DrmPlane *> *planes) {
-  planner_ = Planner::CreateInstance(drm_);
-  if (!planner_) {
-    ALOGE("Failed to create planner instance for composition");
-    return HWC2::Error::NoResources;
-  }
-
   int display = static_cast<int>(handle_);
   int ret = compositor_.Init(resource_manager_, display);
   if (ret) {
@@ -737,8 +731,7 @@
     composition_layers.emplace_back(std::move(layer));
   }
 
-  auto composition = std::make_shared<DrmDisplayComposition>(crtc_,
-                                                             planner_.get());
+  auto composition = std::make_shared<DrmDisplayComposition>(crtc_);
 
   // TODO(nobody): Don't always assume geometry changed
   int ret = composition->SetLayers(composition_layers.data(),