diff options
author | Dale Stimson <dale.b.stimson@intel.com> | 2014-12-07 20:15:09 +0800 |
---|---|---|
committer | Patrick Tjin <pattjin@google.com> | 2014-12-07 21:00:54 -0800 |
commit | 4c0269ac03de5644df56f2435ebc5ef37c32b7f0 (patch) | |
tree | 3692fdef182e1bec119ce8d4c66f0e1b20924374 | |
parent | 3d148c298841ece27d908dc514fbda2348342c1c (diff) | |
download | utils-4c0269ac03de5644df56f2435ebc5ef37c32b7f0.tar.gz |
[Workaround] Disable ISV for YV12 input
Disable ISV for YV12 input.
There is video corruption to playback VP9 with ISV.
ISV does not support YV12 1080P input well.
Author: Xigui Wang <xigui.wang@intel.com>
Bug: 18470801
BZ: 230271
Change-Id: I68fd6e8b74bc24e86127f860ca4ce9b2be085e92
Signed-off-by: Xigui Wang <xigui.wang@intel.com>
-rw-r--r-- | ISV/omx/isv_omxcomponent.cpp | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/ISV/omx/isv_omxcomponent.cpp b/ISV/omx/isv_omxcomponent.cpp index 1d49071..b6cdbdb 100644 --- a/ISV/omx/isv_omxcomponent.cpp +++ b/ISV/omx/isv_omxcomponent.cpp @@ -254,16 +254,18 @@ OMX_ERRORTYPE ISVComponent::ISV_GetParameter( if (nParamIndex == OMX_IndexParamPortDefinition && def->nPortIndex == kPortIndexOutput) { ALOGD_IF(ISV_COMPONENT_DEBUG, "%s: orignal bufferCountActual %d, bufferCountMin %d", __func__, def->nBufferCountActual, def->nBufferCountMin); - //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 OMX_VIDEO_PORTDEFINITIONTYPE *video_def = &def->format.video; if (video_def->eColorFormat == VA_FOURCC_YV12) { - mHackFormat = HAL_PIXEL_FORMAT_YV12; - video_def->eColorFormat = (OMX_COLOR_FORMATTYPE)HAL_PIXEL_FORMAT_NV12_VED; + //FIXME workaround Disable ISV for YV12 input + mVPPEnabled = false; + ALOGI("%s: Disable ISV for YV12 input. mVPPEnabled %d", __func__, mVPPEnabled); + } else { + //FIXME workaround avc low resolution playback + def->nBufferCountActual += mNumISVBuffers + 9; + def->nBufferCountMin += mNumISVBuffers + 9; } #endif } |