commit c97e7bc08a0a4b4b54837b0c7ec5b569f65c6f4f
author Saurabh Shah <saurshah@codeaurora.org> Thu Aug 15 10:30:25 2013 -0700
committer Saurabh Shah <saurshah@codeaurora.org> Fri Sep 06 09:55:44 2013 -0700
tree 02cf5669c6b4c418560407ea875236ca0b97186e
parent 885e144dbdcac4200d433c974701d929f8b8450e

overlay: writeback: Add API to set output format, get writeback FB fd

Add support for setting writeback output format.
Modify the getter to query the driver for output format
if not already done.
Add API to retrieve writeback fb node's fd.

Change-Id: I4a3681b6501c41a094e4cd840591b707b4d7ed66

liboverlay/overlayWriteback.cpp

diff --git a/liboverlay/overlayWriteback.cpp b/liboverlay/overlayWriteback.cpp
index 1d58ece..f1f0eb5 100644
--- a/liboverlay/overlayWriteback.cpp
+++ b/liboverlay/overlayWriteback.cpp
@@ -84,7 +84,6 @@
         ALOGE("%s failed to init %s", __func__, Res::fbPath);
         return;
     }
-    queryOutputFormat();
     startSession();
 }
 
@@ -190,15 +189,33 @@
     return writeSync(mWbMem.getDstFd(), mWbMem.getOffset());
 }
 
-void Writeback::queryOutputFormat() {
-    struct msmfb_metadata metadata;
-    memset(&metadata, 0 , sizeof(metadata));
-    metadata.op = metadata_op_wb_format;
-    if (ioctl(mFd.getFD(), MSMFB_METADATA_GET, &metadata) < 0) {
-        ALOGE("Error retrieving MDP Writeback format");
-        return;
+bool Writeback::setOutputFormat(int mdpFormat) {
+    if(mdpFormat != mOpFmt) {
+        struct msmfb_metadata metadata;
+        memset(&metadata, 0 , sizeof(metadata));
+        metadata.op = metadata_op_wb_format;
+        metadata.data.mixer_cfg.writeback_format = mdpFormat;
+        if (ioctl(mFd.getFD(), MSMFB_METADATA_SET, &metadata) < 0) {
+            ALOGE("Error setting MDP Writeback format");
+            return false;
+        }
+        mOpFmt = mdpFormat;
     }
-    mOpFmt = metadata.data.mixer_cfg.writeback_format;
+    return true;
+}
+
+int Writeback::getOutputFormat() {
+    if(mOpFmt < 0) {
+        struct msmfb_metadata metadata;
+        memset(&metadata, 0 , sizeof(metadata));
+        metadata.op = metadata_op_wb_format;
+        if (ioctl(mFd.getFD(), MSMFB_METADATA_GET, &metadata) < 0) {
+            ALOGE("Error retrieving MDP Writeback format");
+            return -1;
+        }
+        mOpFmt =  metadata.data.mixer_cfg.writeback_format;
+    }
+    return mOpFmt;
 }
 
 //static