commit bd42692f6f55b3008bec86c23bbfcec5aecd6681
author Lajos Molnar <lajos@google.com> Thu Feb 05 10:57:24 2015 -0800
committer The Android Automerger <android-build@google.com> Thu Feb 05 15:03:32 2015 -0800
tree c880c4bd1270166b157e5fcb2132f834e16f9a1e
parent 9270c535ff88963a719e80325e88359a26acb47b

exynos: vdec: report width/height as stride/slice-height for BUFFER_COPY

This is a workaround for bytebuffer mode, as decoder actually uses width
and height as stride and slice height for the output buffer.  Changing
only at getParam as the internal value may be used elsewhere.

Bug: 19179288

Change-Id: I2a85f02bf626971ae3fa31bafe5af9ac761d3ca8

exynos_omx/openmax/exynos_omx/component/video/dec/Exynos_OMX_VdecControl.c

diff --git a/exynos_omx/openmax/exynos_omx/component/video/dec/Exynos_OMX_VdecControl.c b/exynos_omx/openmax/exynos_omx/component/video/dec/Exynos_OMX_VdecControl.c
index 80c3b63..7d4b56b 100644
--- a/exynos_omx/openmax/exynos_omx/component/video/dec/Exynos_OMX_VdecControl.c
+++ b/exynos_omx/openmax/exynos_omx/component/video/dec/Exynos_OMX_VdecControl.c
@@ -1123,6 +1123,15 @@
                 (OMX_COLOR_FORMATTYPE)Exynos_OSAL_OMX2HalPixelFormat(portDefinition->format.video.eColorFormat);
             Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "portDefinition->format.video.eColorFormat:0x%x", portDefinition->format.video.eColorFormat);
         }
+
+        // WORKAROUND provided by Google engineer
+        if (portIndex == OUTPUT_PORT_INDEX &&
+                (pExynosPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
+            // Decoder actually uses width and height as stride and slice height for buffer copy.
+            // Changing only at getParam as the internal value may be used elsewhere
+            portDefinition->format.video.nStride = portDefinition->format.video.nFrameWidth;
+            portDefinition->format.video.nSliceHeight = portDefinition->format.video.nFrameHeight;
+        }
     }
         break;
 #endif