drm_hwcomposer: treewide: Handle bool properties in the Property class
To clean things up and unify the usage of properties.
Change-Id: I0e034ed309f5f8a0d4fc9be7a4a3edcb39050e7f
Signed-off-by: Roman Stratiienko <r.stratiienko@gmail.com>
diff --git a/drm/DrmDisplayPipeline.cpp b/drm/DrmDisplayPipeline.cpp
index 2d81578..7588ee2 100644
--- a/drm/DrmDisplayPipeline.cpp
+++ b/drm/DrmDisplayPipeline.cpp
@@ -158,22 +158,12 @@
return {};
}
-static bool ReadUseOverlayProperty() {
- char use_overlay_planes_prop[PROPERTY_VALUE_MAX];
- property_get("vendor.hwc.drm.use_overlay_planes", use_overlay_planes_prop,
- "1");
- constexpr int kStrtolBase = 10;
- return strtol(use_overlay_planes_prop, nullptr, kStrtolBase) != 0;
-}
-
auto DrmDisplayPipeline::GetUsablePlanes()
-> std::vector<std::shared_ptr<BindingOwner<DrmPlane>>> {
std::vector<std::shared_ptr<BindingOwner<DrmPlane>>> planes;
planes.emplace_back(primary_plane);
- const static bool kUseOverlayPlanes = ReadUseOverlayProperty();
-
- if (kUseOverlayPlanes) {
+ if (Properties::UseOverlayPlanes()) {
for (const auto &plane : device->GetPlanes()) {
if (plane->IsCrtcSupported(*crtc->Get())) {
if (plane->GetType() == DRM_PLANE_TYPE_OVERLAY) {