commit 3d148c298841ece27d908dc514fbda2348342c1c
author Xigui Wang <xigui.wang@intel.com> Thu Nov 27 11:01:45 2014 +0800
committer Pat Tjin <pattjin@google.com> Tue Dec 02 21:49:54 2014 +0000
tree 0d8f8b8ecf771a0282ef9b012b71549f70867527
parent c214ebefec0a4f5ce329854bebcc50e1028283ab

Workaround AVC low resolution playback

Add 9 buffer to workaround AVC low resolution playback

BZ: 229811
Bug: 17383204


Change-Id: I6b90f9e9eceaf52a8406bf12d979e677254a5f42
Signed-off-by: Xigui Wang <xigui.wang@intel.com>

ISV/omx/isv_omxcomponent.cpp

diff --git a/ISV/omx/isv_omxcomponent.cpp b/ISV/omx/isv_omxcomponent.cpp
index 5a23fb1..1d49071 100644
--- a/ISV/omx/isv_omxcomponent.cpp
+++ b/ISV/omx/isv_omxcomponent.cpp
@@ -254,8 +254,9 @@
         if (nParamIndex == OMX_IndexParamPortDefinition
                 && def->nPortIndex == kPortIndexOutput) {
             ALOGD_IF(ISV_COMPONENT_DEBUG, "%s: orignal bufferCountActual %d, bufferCountMin %d",  __func__, def->nBufferCountActual, def->nBufferCountMin);
-            def->nBufferCountActual += mNumISVBuffers;
-            def->nBufferCountMin += mNumISVBuffers;
+            //FIXME workaround avc low resolution playback
+            def->nBufferCountActual += mNumISVBuffers + 9;
+            def->nBufferCountMin += mNumISVBuffers + 9;
 #ifndef TARGET_VPP_USE_GEN
             //FIXME: THIS IS A HACK!! Request NV12 buffer for YV12 format
             //because VSP only support NV12 output

ISV/profile/isv_profile.cpp

diff --git a/ISV/profile/isv_profile.cpp b/ISV/profile/isv_profile.cpp
index 52752b9..86fdd02 100644
--- a/ISV/profile/isv_profile.cpp
+++ b/ISV/profile/isv_profile.cpp
@@ -355,7 +355,7 @@
     FILE *setting_handle, *config_handle;
 
     snprintf(path, 80, "/data/user/%d/com.intel.vpp/shared_prefs/vpp_settings.xml", userId);
-    ALOGI("%s: %s",__func__, path);
+    ALOGV("%s: %s",__func__, path);
     setting_handle = fopen(path, "r");
     if(setting_handle == NULL) {
         ALOGE("%s: failed to open file %s\n", __func__, path);
@@ -381,7 +381,7 @@
         if (strstr(xml_buf, "name=\"DefaultFRCStatus\" value=\"1\"") != NULL)
             status |= VPP_FRC_ON;
 
-        ALOGI("%s: using the default status: VPP=%d, FRC=%d\n", __func__,
+        ALOGV("%s: using the default status: VPP=%d, FRC=%d\n", __func__,
             ((status & VPP_COMMON_ON) == 0) ? 0 : 1,
             ((status & VPP_FRC_ON) == 0) ? 0: 1);
 
@@ -433,27 +433,27 @@
         "FRC_RATE_4X",
     };
 
-    ALOGI("========== VPP filter configs:==========\n");
+    ALOGV("========== VPP filter configs:==========\n");
     for (i = 1; i < ProcFilterCount; i++) {
-        ALOGI("name=%s, enabled=%s, minResolution=%d, maxResolution=%d, isOn=%s\n",
+        ALOGV("name=%s, enabled=%s, minResolution=%d, maxResolution=%d, isOn=%s\n",
             filterNames[i],
             (mConfigs[i].enabled == true) ? "true" : "false",
             mConfigs[i].minResolution,
             mConfigs[i].maxResolution,
             ((mStatus & (1 << i)) == 0) ? "false" : "true");
         if (mConfigs[i].paraSize) {
-            ALOGI("\t\t parameters: ");
+            ALOGV("\t\t parameters: ");
             for(j = 0; j < mConfigs[i].paraSize; j++)
                 ALOGE("%s=%f,", mConfigs[i].paraTables[j].name, mConfigs[i].paraTables[j].value);
-            ALOGI("\n");
+            ALOGV("\n");
         }
     }
 
-    ALOGI("========== FRC rate configs:===========\n");
+    ALOGV("========== FRC rate configs:===========\n");
     for (i = 0; i < MAX_TAB_SIZE; i++) {
         if (mFrcRates[i].input_fps == 0)
             break;
-        ALOGI("input_fps=%d, rate=%s\n", mFrcRates[i].input_fps, rateNames[mFrcRates[i].rate]);
+        ALOGV("input_fps=%d, rate=%s\n", mFrcRates[i].input_fps, rateNames[mFrcRates[i].rate]);
     }
 
     ALOGI("========== common parameter configs:===========\n");