commit d159048c0a3b9c0ff225e3cf8831b49773b8589d
author Dileep Kumar Reddi <dkumarre@codeaurora.org> Thu May 14 15:16:05 2015 +0530
committer Dileep Kumar Reddi <dkumarre@codeaurora.org> Thu May 14 15:32:20 2015 +0530
tree f22e1830b3455a2e94a317ea01f2b78f282fedaf
parent be311da157a5c7e78a21f984aaeca0f213fe86f1

hwc: Enable swapRect for multiple updating layers use cases

In current code swapRect optimization is enabled for the use cases,
where only one layer is updating. this change will enable swapRect for
use cases where more than one layer is updating.

Change-Id: I902ecaa4ab53eac1115d5ea40c9cd582f639bfd8

libqdutils/cb_utils.cpp

diff --git a/libqdutils/cb_utils.cpp b/libqdutils/cb_utils.cpp
index 350d7fd..b85b901 100644
--- a/libqdutils/cb_utils.cpp
+++ b/libqdutils/cb_utils.cpp
@@ -56,7 +56,7 @@
 namespace qdutils {
 
 int CBUtils::uiClearRegion(hwc_display_contents_1_t* list,
-        int version, LayerProp *layerProp,  int dirtyIndex,
+        int version, LayerProp *layerProp,  hwc_rect_t dirtyRect,
             copybit_device_t *copybit, private_handle_t *renderBuffer) {
 
     size_t last = list->numHwLayers - 1;
@@ -64,25 +64,11 @@
     Rect fbFrameRect(fbFrame.left,fbFrame.top,fbFrame.right,fbFrame.bottom);
     Region wormholeRegion(fbFrameRect);
 
-   if (dirtyIndex != -1) {
+   if ((dirtyRect.right - dirtyRect.left > 0) &&
+                               (dirtyRect.bottom - dirtyRect.top > 0)) {
 #ifdef QCOM_BSP
-      /*
-       * 1. Map dirty rect of updating layer to its display frame.
-       * 2. Use this display frame as wormholeRegion instead of full Frame
-       * */
-      hwc_rect_t dirtyRect = list->hwLayers[dirtyIndex].dirtyRect;
-      hwc_rect_t displayFrame = list->hwLayers[dirtyIndex].displayFrame;
-      hwc_frect_t sCropF = list->hwLayers[dirtyIndex].sourceCropf;
-      hwc_rect_t srcRect = {int(ceilf(sCropF.left)), int(ceilf(sCropF.top)),
-                           int(ceilf(sCropF.right)), int(ceilf(sCropF.bottom))};
-
-      displayFrame.left += dirtyRect.left - srcRect.left;
-      displayFrame.top += dirtyRect.top - srcRect.top;
-      displayFrame.right -= srcRect.right - dirtyRect.right;
-      displayFrame.bottom -= srcRect.bottom - dirtyRect.bottom;
-
-      Rect tmpRect(displayFrame.left,displayFrame.top,displayFrame.right,
-            displayFrame.bottom);
+      Rect tmpRect(dirtyRect.left,dirtyRect.top,dirtyRect.right,
+            dirtyRect.bottom);
       Region tmpRegion(tmpRect);
       wormholeRegion = wormholeRegion.intersect(tmpRegion);
 #endif
@@ -116,7 +102,7 @@
      }
    }
    if(wormholeRegion.isEmpty()){
-        return 0;
+        return 1;
    }
    //TO DO :- 1. remove union and call clear for each rect.
    Region::const_iterator it = wormholeRegion.begin();

libqdutils/cb_utils.h

diff --git a/libqdutils/cb_utils.h b/libqdutils/cb_utils.h
index a383fb0..55225b6 100644
--- a/libqdutils/cb_utils.h
+++ b/libqdutils/cb_utils.h
@@ -38,7 +38,7 @@
 class CBUtils {
 public:
     static int uiClearRegion(hwc_display_contents_1_t* list,
-            int version, LayerProp *layerProp, int dirtyIndex,
+            int version, LayerProp *layerProp, hwc_rect_t dirtyIndex,
             copybit_device_t *copybit, private_handle_t *renderBuffer);
 };
 }//namespace qdutils