hwc: Check for >1 overlay used in idle timeout

It was observed during ADTF runs than we get repeated redraws when
there are no layers to render.

This is because the check in prepare() will reset force_sgx to 0
when there are 1 or less layers. The side effect of this is to
cause a redraw every idle period.

Instead of doing this, account for the number of overlays used
for the primary display before calling the invalidate() api. This
logic should be satisfactory most current usecases.

Change-Id: I4b9cb44ebe250a6a46a6da27a1db94e88d6f26f3
Signed-off-by: Tony Lofthouse <a0741364@ti.com>

1 files changed, 1 insertions, 5 deletions

diff --git a/hwc/hwc.c b/hwc/hwc.c
index c98cacc..7e2e544 100644
--- a/hwc/hwc.c
+++ b/hwc/hwc.c
@@ -1701,10 +1701,6 @@ static int omap4_hwc_prepare(struct hwc_composer_device *dev, hwc_layer_list_t*
 
     decide_supported_cloning(hwc_dev, &num);
 
-    /* Disable the forced SGX rendering if there is only one layer */
-    if (hwc_dev->force_sgx && num.composited_layers <= 1)
-        hwc_dev->force_sgx = 0;
-
     /* phase 3 logic */
     if (can_dss_render_all(hwc_dev, &num)) {
         /* All layers can be handled by the DSS -- don't use SGX for composition */
@@ -2545,7 +2541,7 @@ static void *omap4_hwc_hdmi_thread(void *data)
             if (hwc_dev->idle) {
                 if (hwc_dev->procs && hwc_dev->procs->invalidate) {
                     pthread_mutex_lock(&hwc_dev->lock);
-                    invalidate = !hwc_dev->force_sgx;
+                    invalidate = hwc_dev->last_int_ovls > 1 && !hwc_dev->force_sgx;
                     if (invalidate) {
                         hwc_dev->force_sgx = 2;
                     }