commit 81a519f59961617a03aae0656990b8fc3e8a83cc
author Nagesh Subba Reddy <nageshsreddy@codeaurora.org> Sat Aug 29 18:01:49 2015 -0700
committer Shuzhen Wang <shuzhenw@codeaurora.org> Wed Sep 09 13:04:50 2015 -0700
tree d3406faa3abbfa4aba49447b93fc45cb20686031
parent 330b21d64ec7e06ebaf6ad74d1374f8ccb9eb43f

gralloc: Default flexible YUV format to NV21_ZSL

There could be buffers allocated in flexible YUV format without any
camera usage flags. Default the format for such buffers to NV21.
Fixes CTS tests testYuvImageWriterReaderOperation, testAbandonedSurfaceExceptions

Change-Id: I72693b1d35d2803df16d3588b3428bcf1ab0b3eb

libgralloc/gpu.cpp

diff --git a/libgralloc/gpu.cpp b/libgralloc/gpu.cpp
index 6945daf..71c7bba 100644
--- a/libgralloc/gpu.cpp
+++ b/libgralloc/gpu.cpp
@@ -289,13 +289,18 @@
             grallocFormat = HAL_PIXEL_FORMAT_YCrCb_420_SP; //NV21
         else if(usage & GRALLOC_USAGE_HW_CAMERA_WRITE) {
            if (format == HAL_PIXEL_FORMAT_YCbCr_420_888) {
-               grallocFormat = HAL_PIXEL_FORMAT_YCrCb_420_SP; //NV21
+               grallocFormat = HAL_PIXEL_FORMAT_NV21_ZSL; //NV21
            } else {
                grallocFormat = HAL_PIXEL_FORMAT_YCbCr_420_SP_VENUS; //NV12 preview
            }
         } else if(usage & GRALLOC_USAGE_HW_COMPOSER)
             //XXX: If we still haven't set a format, default to RGBA8888
             grallocFormat = HAL_PIXEL_FORMAT_RGBA_8888;
+        else if(format == HAL_PIXEL_FORMAT_YCbCr_420_888) {
+            //If no other usage flags are detected, default the
+            //flexible YUV format to NV21_ZSL
+            grallocFormat = HAL_PIXEL_FORMAT_NV21_ZSL;
+        }
     }
 
     bool useFbMem = false;