diff options
author | achaulk <achaulk@chromium.org> | 2015-05-25 09:15:59 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-05-25 16:17:32 +0000 |
commit | e214e82249824bd059d9a5a48387578bb678a16c (patch) | |
tree | ade3dddd8c4e22c011d93230b6404ad19b89fe98 /ui/ozone/platform/drm/gpu/hardware_display_plane_manager.cc | |
parent | 2f70d9902af612496792bbb1a8bb9a3a9d9cab49 (diff) | |
download | chromium_src-e214e82249824bd059d9a5a48387578bb678a16c.zip chromium_src-e214e82249824bd059d9a5a48387578bb678a16c.tar.gz chromium_src-e214e82249824bd059d9a5a48387578bb678a16c.tar.bz2 |
ozone: Remove committed bool from HDPM state
R=dnicoara@chromium.org
Review URL: https://codereview.chromium.org/1153743003
Cr-Commit-Position: refs/heads/master@{#331292}
Diffstat (limited to 'ui/ozone/platform/drm/gpu/hardware_display_plane_manager.cc')
-rw-r--r-- | ui/ozone/platform/drm/gpu/hardware_display_plane_manager.cc | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/ui/ozone/platform/drm/gpu/hardware_display_plane_manager.cc b/ui/ozone/platform/drm/gpu/hardware_display_plane_manager.cc index bd50d55..a790d62 100644 --- a/ui/ozone/platform/drm/gpu/hardware_display_plane_manager.cc +++ b/ui/ozone/platform/drm/gpu/hardware_display_plane_manager.cc @@ -24,7 +24,7 @@ const float kFixedPointScaleValue = 65536.0f; } // namespace -HardwareDisplayPlaneList::HardwareDisplayPlaneList() : committed(false) { +HardwareDisplayPlaneList::HardwareDisplayPlaneList() { #if defined(USE_DRM_ATOMIC) atomic_property_set.reset(drmModePropertySetAlloc()); #endif // defined(USE_DRM_ATOMIC) @@ -162,19 +162,18 @@ int HardwareDisplayPlaneManager::LookupCrtcIndex(uint32_t crtc_id) { return -1; } +void HardwareDisplayPlaneManager::BeginFrame( + HardwareDisplayPlaneList* plane_list) { + for (auto* plane : plane_list->old_plane_list) { + plane->set_in_use(false); + } +} + bool HardwareDisplayPlaneManager::AssignOverlayPlanes( HardwareDisplayPlaneList* plane_list, const OverlayPlaneList& overlay_list, uint32_t crtc_id, CrtcController* crtc) { - // If we had previously committed this set, mark all owned planes as free. - if (plane_list->committed) { - plane_list->committed = false; - for (auto* plane : plane_list->old_plane_list) { - plane->set_in_use(false); - } - } - int crtc_index = LookupCrtcIndex(crtc_id); if (crtc_index < 0) { LOG(ERROR) << "Cannot find crtc " << crtc_id; |