exynos_omx: multi_thread: Code clean-up.
authorSeungBeom Kim <sbcrux.kim@samsung.com>
Tue, 3 Jul 2012 11:03:59 +0000 (20:03 +0900)
committerDima Zavin <dima@android.com>
Tue, 24 Jul 2012 22:12:41 +0000 (15:12 -0700)
- remove tab & line.
- remove log message.

Change-Id: I0e21ff7da9b1d4e3533e340be276e98e82f6f6e7
Signed-off-by: SeungBeom Kim <sbcrux.kim@samsung.com>
exynos_omx/openmax/exynos_omx/component/video/dec/Exynos_OMX_Vdec.c
exynos_omx/openmax/exynos_omx/component/video/dec/Exynos_OMX_Vdec.h
exynos_omx/openmax/exynos_omx/component/video/dec/Exynos_OMX_VdecControl.c
exynos_omx/openmax/exynos_omx/component/video/dec/h264/Exynos_OMX_H264dec.c
exynos_omx/openmax/exynos_omx/component/video/dec/mpeg4/Exynos_OMX_Mpeg4dec.c
exynos_omx/openmax/exynos_omx/component/video/dec/vp8/Exynos_OMX_Vp8dec.c
exynos_omx/openmax/exynos_omx/component/video/enc/Exynos_OMX_Venc.c
exynos_omx/openmax/exynos_omx/component/video/enc/Exynos_OMX_Venc.h
exynos_omx/openmax/exynos_omx/component/video/enc/Exynos_OMX_VencControl.c
exynos_omx/openmax/exynos_omx/component/video/enc/h264/Exynos_OMX_H264enc.c
exynos_omx/openmax/exynos_omx/component/video/enc/mpeg4/Exynos_OMX_Mpeg4enc.c

index 0e02d0a8cf59721733262370eb50ea5adf48edb1..010448fd66e3d9e81d4ee4b78ce0450c2d22d39b 100644 (file)
@@ -108,7 +108,7 @@ inline void Exynos_UpdateFrameSize(OMX_COMPONENTTYPE *pOMXComponent)
         }
     }
 
-  return;
+    return;
 }
 
 OMX_BOOL Exynos_Check_BufferProcess_State(EXYNOS_OMX_BASECOMPONENT *pExynosComponent, OMX_U32 nPortIndex)
@@ -157,13 +157,10 @@ OMX_ERRORTYPE Exynos_Output_CodecBufferToData(EXYNOS_OMX_BASECOMPONENT *pExynosC
     OMX_PTR pSrcBuf[MAX_BUFFER_PLANE];
     OMX_U32 allocSize[MAX_BUFFER_PLANE];
 
-    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%s: buffer=%p\n", __func__, codecBuffer);
-
     pVideoDec->exynos_codec_getCodecOutputPrivateData(codecBuffer, pSrcBuf, allocSize);
     pData->buffer.multiPlaneBuffer.dataBuffer[0] = pSrcBuf[0];
     pData->buffer.multiPlaneBuffer.dataBuffer[1] = pSrcBuf[1];
     pData->buffer.multiPlaneBuffer.dataBuffer[2] = pSrcBuf[2];
-
     pData->allocSize     = allocSize[0] + allocSize[1] + allocSize[2];
     pData->dataLen       = 0;
     pData->usedDataLen   = 0;
@@ -281,11 +278,7 @@ OMX_BOOL Exynos_Preprocessor_InputData(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_
                 flagEOF = OMX_FALSE;
             }
 
-//            if ((inputUseBuffer->remainDataLen == 0) ||
-//                (CHECK_PORT_BEING_FLUSHED(exynosInputPort)))
-                Exynos_InputBufferReturn(pOMXComponent);
-//            else
-//                inputUseBuffer->dataValid = OMX_TRUE;
+            Exynos_InputBufferReturn(pOMXComponent);
         }
 
         if ((srcInputData->nFlags & OMX_BUFFERFLAG_EOS) == OMX_BUFFERFLAG_EOS) {
@@ -620,8 +613,6 @@ OMX_ERRORTYPE Exynos_OMX_SrcOutputBufferProcess(OMX_HANDLETYPE hComponent)
     while (!pVideoDec->bExitBufferProcessThread) {
         Exynos_OSAL_SleepMillisec(0);
 
-  //      Exynos_Wait_ProcessPause(pExynosComponent, INPUT_PORT_INDEX);
-
         while (!pVideoDec->bExitBufferProcessThread) {
             if ((exynosInputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
                 if (Exynos_Check_BufferProcess_State(pExynosComponent, INPUT_PORT_INDEX) == OMX_FALSE)
@@ -631,10 +622,7 @@ OMX_ERRORTYPE Exynos_OMX_SrcOutputBufferProcess(OMX_HANDLETYPE hComponent)
 
             if (CHECK_PORT_BEING_FLUSHED(exynosInputPort))
                 break;
-/*
-            if (exynosInputPort->portState != OMX_StateIdle)
-                break;
-*/
+
             Exynos_OSAL_MutexLock(srcOutputUseBuffer->bufferMutex);
             ret = pVideoDec->exynos_codec_srcOutputProcess(pOMXComponent, &srcOutputData);
 
@@ -648,13 +636,6 @@ OMX_ERRORTYPE Exynos_OMX_SrcOutputBufferProcess(OMX_HANDLETYPE hComponent)
                 if (exynosInputPort->bufferProcessType == BUFFER_SHARE) {
                     Exynos_Shared_DataToBuffer(&srcOutputData, srcOutputUseBuffer);
                     Exynos_InputBufferReturn(pOMXComponent);
-/*
-                    if ((pVideoDec->exynos_codec_getCodecBufferNumber != NULL) &&
-                        (CHECK_PORT_BEING_FLUSHED(exynosInputPort)))
-                        if (pVideoDec->exynos_codec_getCodecBufferNumber(pOMXComponent, INPUT_PORT_INDEX) == 0) {
-                            Exynos_OSAL_SignalSet(exynosInputPort->hAllCodecBufferReturnEvent);
-                        }
-*/
                 }
                 Exynos_ResetCodecData(&srcOutputData);
             }
@@ -684,8 +665,6 @@ OMX_ERRORTYPE Exynos_OMX_DstInputBufferProcess(OMX_HANDLETYPE hComponent)
     while (!pVideoDec->bExitBufferProcessThread) {
         Exynos_OSAL_SleepMillisec(0);
 
-//        Exynos_Wait_ProcessPause(pExynosComponent, OUTPUT_PORT_INDEX);
-
         while ((Exynos_Check_BufferProcess_State(pExynosComponent, OUTPUT_PORT_INDEX)) &&
                (!pVideoDec->bExitBufferProcessThread)) {
             Exynos_OSAL_SleepMillisec(0);
@@ -773,10 +752,7 @@ OMX_ERRORTYPE Exynos_OMX_DstOutputBufferProcess(OMX_HANDLETYPE hComponent)
 
             if (CHECK_PORT_BEING_FLUSHED(exynosOutputPort))
                 break;
-/*
-            if (exynosOutputPort->portState != OMX_StateIdle)
-                break;
-*/
+
             Exynos_OSAL_MutexLock(dstOutputUseBuffer->bufferMutex);
             if ((exynosOutputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
                 if ((dstOutputUseBuffer->dataValid != OMX_TRUE) &&
@@ -983,7 +959,6 @@ OMX_ERRORTYPE Exynos_OMX_BufferProcess_Terminate(OMX_HANDLETYPE hComponent)
 
     pVideoDec->bExitBufferProcessThread = OMX_TRUE;
 
-    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%s:%d", __FUNCTION__, __LINE__);
     Exynos_OSAL_Get_SemaphoreCount(pExynosComponent->pExynosPort[INPUT_PORT_INDEX].bufferSemID, &countValue);
     if (countValue == 0)
         Exynos_OSAL_SemaphorePost(pExynosComponent->pExynosPort[INPUT_PORT_INDEX].bufferSemID);
@@ -994,7 +969,6 @@ OMX_ERRORTYPE Exynos_OMX_BufferProcess_Terminate(OMX_HANDLETYPE hComponent)
     Exynos_OSAL_ThreadTerminate(pVideoDec->hSrcInputThread);
     pVideoDec->hSrcInputThread = NULL;
 
-    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%s:%d", __FUNCTION__, __LINE__);
     Exynos_OSAL_Get_SemaphoreCount(pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX].bufferSemID, &countValue);
     if (countValue == 0)
         Exynos_OSAL_SemaphorePost(pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX].bufferSemID);
@@ -1005,22 +979,18 @@ OMX_ERRORTYPE Exynos_OMX_BufferProcess_Terminate(OMX_HANDLETYPE hComponent)
     Exynos_OSAL_ThreadTerminate(pVideoDec->hDstInputThread);
     pVideoDec->hDstInputThread = NULL;
 
-    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%s:%d", __FUNCTION__, __LINE__);
     pVideoDec->exynos_codec_stop(pOMXComponent, INPUT_PORT_INDEX);
     pVideoDec->exynos_codec_bufferProcessRun(pOMXComponent, INPUT_PORT_INDEX);
     Exynos_OSAL_SignalSet(pExynosComponent->pExynosPort[INPUT_PORT_INDEX].pauseEvent);
     Exynos_OSAL_ThreadTerminate(pVideoDec->hSrcOutputThread);
     pVideoDec->hSrcOutputThread = NULL;
 
-    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%s:%d", __FUNCTION__, __LINE__);
     pVideoDec->exynos_codec_stop(pOMXComponent, OUTPUT_PORT_INDEX);
     pVideoDec->exynos_codec_bufferProcessRun(pOMXComponent, INPUT_PORT_INDEX);
     Exynos_OSAL_SignalSet(pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX].pauseEvent);
     Exynos_OSAL_ThreadTerminate(pVideoDec->hDstOutputThread);
     pVideoDec->hDstOutputThread = NULL;
 
-    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%s:%d", __FUNCTION__, __LINE__);
-
 EXIT:
     FunctionOut();
 
index 6fcc054c5ccf1407030ca822f233de1caf0c3ff8..6f6188fde35157743ad76e66f4a9d43c4a51e525 100644 (file)
@@ -49,7 +49,6 @@
 #define DEFAULT_MFC_INPUT_BUFFER_SIZE       1920 * 1080 * 3 / 2
 
 #define MFC_OUTPUT_BUFFER_NUM_MAX           16 * 2
-#define MFC_OUTPUT_BUFFER_PLANE             2
 #define DEFAULT_MFC_OUTPUT_YBUFFER_SIZE     1920 * 1080
 #define DEFAULT_MFC_OUTPUT_CBUFFER_SIZE     1920 * 1080 / 2
 
@@ -58,6 +57,9 @@
 
 #define EXTRA_DPB_NUM                       5
 
+#define MFC_INPUT_BUFFER_PLANE              1
+#define MFC_OUTPUT_BUFFER_PLANE             2
+
 typedef struct
 {
     void *pAddrY;
index d696900309bb0619b9ba0649f2ff756d75d93c64..c95f2a4babe0b36b119d27b52de23584cc679ebe 100644 (file)
@@ -68,8 +68,6 @@ OMX_ERRORTYPE Exynos_OMX_UseBuffer(
 
     FunctionIn();
 
-    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%s: idx=%d sz=%d pBuffer=%p\n", __func__, nPortIndex, nSizeBytes, pBuffer);
-
     if (hComponent == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -589,7 +587,6 @@ EXIT:
     return ret;
 }
 
-
 OMX_ERRORTYPE Exynos_InputBufferReturn(OMX_COMPONENTTYPE *pOMXComponent)
 {
     OMX_ERRORTYPE          ret = OMX_ErrorNone;
@@ -961,7 +958,7 @@ OMX_ERRORTYPE Exynos_CodecBufferReset(EXYNOS_OMX_BASECOMPONENT *pExynosComponent
     }
     while (1) {
         int cnt = 0;
-        Exynos_OSAL_Get_SemaphoreCount(pExynosPort->codecSemID, &cnt); 
+        Exynos_OSAL_Get_SemaphoreCount(pExynosPort->codecSemID, &cnt);
         if (cnt > 0)
             Exynos_OSAL_SemaphoreWait(pExynosPort->codecSemID);
         else
@@ -975,7 +972,6 @@ EXIT:
     return ret;
 }
 
-
 OMX_ERRORTYPE Exynos_OMX_VideoDecodeGetParameter(
     OMX_IN OMX_HANDLETYPE hComponent,
     OMX_IN OMX_INDEXTYPE  nParamIndex,
@@ -1326,13 +1322,11 @@ OMX_ERRORTYPE Exynos_OMX_VideoDecodeGetConfig(
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     if (pOMXComponent->pComponentPrivate == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
-
     if (pComponentConfigStructure == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -1374,13 +1368,11 @@ OMX_ERRORTYPE Exynos_OMX_VideoDecodeSetConfig(
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     if (pOMXComponent->pComponentPrivate == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
-
     if (pComponentConfigStructure == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
index b8b538c35ddaab5f10899f5b6ba1c7323d96c9da..b5b9358ecdcfcc1c51393d9b1f2c0d5bd3e231ac 100644 (file)
@@ -250,8 +250,7 @@ static OMX_BOOL Check_H264_StartCode(
 
 OMX_ERRORTYPE H264CodecOpen(EXYNOS_H264DEC_HANDLE *pH264Dec)
 {
-    OMX_ERRORTYPE           ret = OMX_ErrorNone;
-
+    OMX_ERRORTYPE            ret        = OMX_ErrorNone;
     ExynosVideoDecOps       *pDecOps    = NULL;
     ExynosVideoDecBufferOps *pInbufOps  = NULL;
     ExynosVideoDecBufferOps *pOutbufOps = NULL;
@@ -264,7 +263,6 @@ OMX_ERRORTYPE H264CodecOpen(EXYNOS_H264DEC_HANDLE *pH264Dec)
         goto EXIT;
     }
 
-    /* MFC Open */
     /* alloc ops structure */
     pDecOps    = (ExynosVideoDecOps *)Exynos_OSAL_Malloc(sizeof(ExynosVideoDecOps));
     pInbufOps  = (ExynosVideoDecBufferOps *)Exynos_OSAL_Malloc(sizeof(ExynosVideoDecBufferOps));
@@ -286,6 +284,7 @@ OMX_ERRORTYPE H264CodecOpen(EXYNOS_H264DEC_HANDLE *pH264Dec)
     pOutbufOps->nSize = sizeof(ExynosVideoDecBufferOps);
 
     Exynos_Video_Register_Decoder(pDecOps, pInbufOps, pOutbufOps);
+
     /* check mandatory functions for decoder ops */
     if ((pDecOps->Init == NULL) || (pDecOps->Finalize == NULL) ||
         (pDecOps->Get_ActualBufferCount == NULL) || (pDecOps->Set_FrameTag == NULL) ||
@@ -330,6 +329,8 @@ OMX_ERRORTYPE H264CodecClose(EXYNOS_H264DEC_HANDLE *pH264Dec)
     ExynosVideoDecBufferOps *pInbufOps  = NULL;
     ExynosVideoDecBufferOps *pOutbufOps = NULL;
 
+    FunctionIn();
+
     if (pH264Dec == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -342,19 +343,19 @@ OMX_ERRORTYPE H264CodecClose(EXYNOS_H264DEC_HANDLE *pH264Dec)
 
     if (hMFCHandle != NULL) {
         pDecOps->Finalize(hMFCHandle);
-        hMFCHandle = pH264Dec->hMFCH264Handle.hMFCHandle = NULL;
+        pH264Dec->hMFCH264Handle.hMFCHandle = NULL;
     }
     if (pOutbufOps != NULL) {
         Exynos_OSAL_Free(pOutbufOps);
-        pOutbufOps = pH264Dec->hMFCH264Handle.pOutbufOps = NULL;
+        pH264Dec->hMFCH264Handle.pOutbufOps = NULL;
     }
     if (pInbufOps != NULL) {
         Exynos_OSAL_Free(pInbufOps);
-        pInbufOps = pH264Dec->hMFCH264Handle.pInbufOps = NULL;
+        pH264Dec->hMFCH264Handle.pInbufOps = NULL;
     }
     if (pDecOps != NULL) {
         Exynos_OSAL_Free(pDecOps);
-        pDecOps = pH264Dec->hMFCH264Handle.pDecOps = NULL;
+        pH264Dec->hMFCH264Handle.pDecOps = NULL;
     }
 
     ret = OMX_ErrorNone;
@@ -375,6 +376,8 @@ OMX_ERRORTYPE H264CodecStart(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortInde
     EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = NULL;
     EXYNOS_H264DEC_HANDLE   *pH264Dec = NULL;
 
+    FunctionIn();
+
     if (pOMXComponent == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -385,6 +388,7 @@ OMX_ERRORTYPE H264CodecStart(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortInde
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
+
     pH264Dec = (EXYNOS_H264DEC_HANDLE *)pVideoDec->hCodecHandle;
     if (pH264Dec == NULL) {
         ret = OMX_ErrorBadParameter;
@@ -401,7 +405,6 @@ OMX_ERRORTYPE H264CodecStart(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortInde
     else if (nPortIndex == OUTPUT_PORT_INDEX)
         pOutbufOps->Run(hMFCHandle);
 
-
     ret = OMX_ErrorNone;
 
 EXIT:
@@ -420,6 +423,8 @@ OMX_ERRORTYPE H264CodecStop(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortIndex
     EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = NULL;
     EXYNOS_H264DEC_HANDLE   *pH264Dec = NULL;
 
+    FunctionIn();
+
     if (pOMXComponent == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -581,7 +586,6 @@ OMX_ERRORTYPE H264CodecSrcSetup(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_DAT
     OMX_U32                  inputBufferNumber = 0;
     int i;
 
-
     FunctionIn();
 
     if ((oneFrameSize <= 0) && (pSrcInputData->nFlags & OMX_BUFFERFLAG_EOS)) {
@@ -679,6 +683,7 @@ OMX_ERRORTYPE H264CodecSrcSetup(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_DAT
     else
         pH264Dec->hMFCH264Handle.MFCOutputColorType = bufferConf.eColorFormat = VIDEO_COLORFORMAT_NV12_TILED;
 #endif
+
     if (pOutbufOps->Set_Geometry(hMFCHandle, &bufferConf) != VIDEO_ERROR_NONE) {
         Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "Failed to set geometry for output buffer");
         ret = OMX_ErrorInsufficientResources;
@@ -1247,13 +1252,11 @@ OMX_ERRORTYPE Exynos_H264Dec_GetConfig(
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     if (pOMXComponent->pComponentPrivate == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
-
     if (pComponentConfigStructure == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -1325,13 +1328,11 @@ OMX_ERRORTYPE Exynos_H264Dec_SetConfig(
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     if (pOMXComponent->pComponentPrivate == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
-
     if (pComponentConfigStructure == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -1389,9 +1390,7 @@ OMX_ERRORTYPE Exynos_H264Dec_GetExtensionIndex(
 
     if (Exynos_OSAL_Strcmp(cParameterName, EXYNOS_INDEX_PARAM_ENABLE_THUMBNAIL) == 0) {
         EXYNOS_H264DEC_HANDLE *pH264Dec = (EXYNOS_H264DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
-
         *pIndexType = OMX_IndexVendorThumbnailMode;
-
         ret = OMX_ErrorNone;
     } else {
         ret = Exynos_OMX_VideoDecodeGetExtensionIndex(hComponent, cParameterName, pIndexType);
@@ -1439,7 +1438,7 @@ OMX_ERRORTYPE Exynos_H264Dec_Init(OMX_COMPONENTTYPE *pOMXComponent)
     EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec         = (EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle;
     EXYNOS_OMX_BASEPORT           *pExynosInputPort  = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
     EXYNOS_OMX_BASEPORT           *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
-    EXYNOS_H264DEC_HANDLE         *pH264Dec          = (EXYNOS_H264DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
+    EXYNOS_H264DEC_HANDLE         *pH264Dec          = (EXYNOS_H264DEC_HANDLE *)pVideoDec->hCodecHandle;
     OMX_PTR                        hMFCHandle        = pH264Dec->hMFCH264Handle.hMFCHandle;
 
     ExynosVideoDecOps       *pDecOps    = NULL;
@@ -1470,8 +1469,7 @@ OMX_ERRORTYPE Exynos_H264Dec_Init(OMX_COMPONENTTYPE *pOMXComponent)
         Exynos_OSAL_SemaphoreCreate(&pExynosInputPort->codecSemID);
         Exynos_OSAL_QueueCreate(&pExynosInputPort->codecBufferQ, MAX_QUEUE_ELEMENTS);
 
-        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++) {
             /*************/
             /*    TBD    */
             /*************/
@@ -1569,8 +1567,7 @@ OMX_ERRORTYPE Exynos_H264Dec_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
     pH264Dec->bSourceStart = OMX_FALSE;
 
     if ((pExynosOutputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
-        for (i = 0; i < MFC_OUTPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_OUTPUT_BUFFER_NUM_MAX; i++) {
             if (pVideoDec->pMFCDecOutputBuffer[i] != NULL) {
                 if (pVideoDec->pMFCDecOutputBuffer[i]->VirAddr[0] != NULL)
                     Exynos_OSAL_SharedMemory_Free(pVideoDec->hSharedMemory, pVideoDec->pMFCDecOutputBuffer[i]->VirAddr[0]);
@@ -1584,19 +1581,14 @@ OMX_ERRORTYPE Exynos_H264Dec_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
         Exynos_OSAL_QueueTerminate(&pExynosOutputPort->codecBufferQ);
         Exynos_OSAL_SemaphoreTerminate(pExynosOutputPort->codecSemID);
     } else if (pExynosOutputPort->bufferProcessType == BUFFER_SHARE) {
-
-
         /*************/
         /*    TBD    */
         /*************/
         /* Does not require any actions. */
-
-
     }
 
     if ((pExynosInputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
-        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++) {
             if (pVideoDec->pMFCDecInputBuffer[i] != NULL) {
                 if (pVideoDec->pMFCDecInputBuffer[i]->VirAddr != NULL)
                     Exynos_OSAL_SharedMemory_Free(pVideoDec->hSharedMemory, pVideoDec->pMFCDecInputBuffer[i]->VirAddr);
@@ -1608,14 +1600,10 @@ OMX_ERRORTYPE Exynos_H264Dec_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
         Exynos_OSAL_QueueTerminate(&pExynosInputPort->codecBufferQ);
         Exynos_OSAL_SemaphoreTerminate(pExynosInputPort->codecSemID);
     } else if (pExynosInputPort->bufferProcessType == BUFFER_SHARE) {
-
-
         /*************/
         /*    TBD    */
         /*************/
         /* Does not require any actions. */
-
-
     }
     H264CodecClose(pH264Dec);
 
@@ -1684,7 +1672,6 @@ OMX_ERRORTYPE Exynos_H264Dec_SrcIn(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_
     ret = OMX_ErrorNone;
 
 EXIT:
-
     FunctionOut();
 
     return ret;
@@ -1726,13 +1713,12 @@ OMX_ERRORTYPE Exynos_H264Dec_SrcOut(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX
             int i = 0;
             while (pSrcOutputData->buffer.singlePlaneBuffer.dataBuffer != pVideoDec->pMFCDecInputBuffer[i]->VirAddr) {
                 if (i >= MFC_INPUT_BUFFER_NUM_MAX) {
-                    ret = (OMX_ERRORTYPE)OMX_ErrorCodecDecode;
                     Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "Can not find buffer");
+                    ret = (OMX_ERRORTYPE)OMX_ErrorCodecDecode;
                     goto EXIT;
                 }
                 i++;
             }
-
             pVideoDec->pMFCDecInputBuffer[i]->dataSize = 0;
             pSrcOutputData->pPrivate = pVideoDec->pMFCDecInputBuffer[i];
         }
@@ -1772,8 +1758,8 @@ OMX_ERRORTYPE Exynos_H264Dec_DstIn(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_
     Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%s : %d => ADDR: 0x%x", __FUNCTION__, __LINE__, pDstInputData->buffer.multiPlaneBuffer.dataBuffer[0]);
     codecReturn = pOutbufOps->Enqueue(hMFCHandle, (unsigned char **)pDstInputData->buffer.multiPlaneBuffer.dataBuffer, (unsigned int *)&dataLen, 2, pDstInputData->bufferHeader);
     if (codecReturn != VIDEO_ERROR_NONE) {
-        ret = (OMX_ERRORTYPE)OMX_ErrorCodecDecode;
         Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "%s : %d", __FUNCTION__, __LINE__);
+        ret = (OMX_ERRORTYPE)OMX_ErrorCodecDecode;
         goto EXIT;
     }
     H264CodecStart(pOMXComponent, OUTPUT_PORT_INDEX);
@@ -1898,7 +1884,7 @@ OMX_ERRORTYPE Exynos_H264Dec_DstOut(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX
 
     if ((displayStatus == VIDEO_FRAME_STATUS_CHANGE_RESOL) ||
         ((pDstOutputData->nFlags & OMX_BUFFERFLAG_EOS) == OMX_BUFFERFLAG_EOS)) {
-        Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%x displayStatus:%d, nFlags0x%x", pExynosComponent, displayStatus, pDstOutputData->nFlags);
+        Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "displayStatus:%d, nFlags0x%x", displayStatus, pDstOutputData->nFlags);
         pDstOutputData->remainDataLen = 0;
     } else {
         pDstOutputData->remainDataLen = bufferGeometry->nFrameWidth * bufferGeometry->nFrameHeight * 3 / 2;
@@ -2061,13 +2047,13 @@ EXIT:
 
 OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(OMX_HANDLETYPE hComponent, OMX_STRING componentName)
 {
-    OMX_ERRORTYPE                  ret = OMX_ErrorNone;
-    OMX_COMPONENTTYPE             *pOMXComponent = NULL;
+    OMX_ERRORTYPE                  ret              = OMX_ErrorNone;
+    OMX_COMPONENTTYPE             *pOMXComponent    = NULL;
     EXYNOS_OMX_BASECOMPONENT      *pExynosComponent = NULL;
-    EXYNOS_OMX_BASEPORT           *pExynosPort = NULL;
-    EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = NULL;
-    EXYNOS_H264DEC_HANDLE         *pH264Dec = NULL;
-    OMX_BOOL                       bDRMPlayerMode = OMX_FALSE;
+    EXYNOS_OMX_BASEPORT           *pExynosPort      = NULL;
+    EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec        = NULL;
+    EXYNOS_H264DEC_HANDLE         *pH264Dec         = NULL;
+    OMX_BOOL                       bDRMPlayerMode   = OMX_FALSE;
     int i = 0;
 
     FunctionIn();
@@ -2193,8 +2179,8 @@ OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(OMX_HANDLETYPE hComponent
     pVideoDec->exynos_codec_dstInputProcess  = &Exynos_H264Dec_dstInputBufferProcess;
     pVideoDec->exynos_codec_dstOutputProcess = &Exynos_H264Dec_dstOutputBufferProcess;
 
-    pVideoDec->exynos_codec_start         = &H264CodecStart;
-    pVideoDec->exynos_codec_stop          = &H264CodecStop;
+    pVideoDec->exynos_codec_start            = &H264CodecStart;
+    pVideoDec->exynos_codec_stop             = &H264CodecStop;
     pVideoDec->exynos_codec_bufferProcessRun = &H264CodecOutputBufferProcessRun;
     pVideoDec->exynos_codec_enqueueAllBuffer = &H264CodecEnQueueAllBuffer;
 
@@ -2204,6 +2190,7 @@ OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(OMX_HANDLETYPE hComponent
 
     pVideoDec->hSharedMemory = Exynos_OSAL_SharedMemory_Open();
     if (pVideoDec->hSharedMemory == NULL) {
+        Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "OMX_ErrorInsufficientResources, Line:%d", __LINE__);
         Exynos_OSAL_Free(pH264Dec);
         pH264Dec = ((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle = NULL;
         Exynos_OMX_VideoDecodeComponentDeinit(pOMXComponent);
index 8474463f479ac4d8542748041fa79f35ea488ee1..47b0a2e83dfe0af0209d7ea0ef93dac5d8f7d44d 100644 (file)
@@ -354,8 +354,7 @@ static OMX_BOOL Check_Stream_StartCode(
 
 OMX_ERRORTYPE Mpeg4CodecOpen(EXYNOS_MPEG4DEC_HANDLE *pMpeg4Dec)
 {
-    OMX_ERRORTYPE           ret = OMX_ErrorNone;
-
+    OMX_ERRORTYPE            ret        = OMX_ErrorNone;
     ExynosVideoDecOps       *pDecOps    = NULL;
     ExynosVideoDecBufferOps *pInbufOps  = NULL;
     ExynosVideoDecBufferOps *pOutbufOps = NULL;
@@ -368,7 +367,6 @@ OMX_ERRORTYPE Mpeg4CodecOpen(EXYNOS_MPEG4DEC_HANDLE *pMpeg4Dec)
         goto EXIT;
     }
 
-    /* MFC Open */
     /* alloc ops structure */
     pDecOps    = (ExynosVideoDecOps *)Exynos_OSAL_Malloc(sizeof(ExynosVideoDecOps));
     pInbufOps  = (ExynosVideoDecBufferOps *)Exynos_OSAL_Malloc(sizeof(ExynosVideoDecBufferOps));
@@ -390,6 +388,7 @@ OMX_ERRORTYPE Mpeg4CodecOpen(EXYNOS_MPEG4DEC_HANDLE *pMpeg4Dec)
     pOutbufOps->nSize = sizeof(ExynosVideoDecBufferOps);
 
     Exynos_Video_Register_Decoder(pDecOps, pInbufOps, pOutbufOps);
+
     /* check mandatory functions for decoder ops */
     if ((pDecOps->Init == NULL) || (pDecOps->Finalize == NULL) ||
         (pDecOps->Get_ActualBufferCount == NULL) || (pDecOps->Set_FrameTag == NULL) ||
@@ -428,12 +427,14 @@ EXIT:
 
 OMX_ERRORTYPE Mpeg4CodecClose(EXYNOS_MPEG4DEC_HANDLE *pMpeg4Dec)
 {
-    OMX_ERRORTYPE            ret = OMX_ErrorNone;
+    OMX_ERRORTYPE            ret        = OMX_ErrorNone;
     void                    *hMFCHandle = NULL;
     ExynosVideoDecOps       *pDecOps    = NULL;
     ExynosVideoDecBufferOps *pInbufOps  = NULL;
     ExynosVideoDecBufferOps *pOutbufOps = NULL;
 
+    FunctionIn();
+
     if (pMpeg4Dec == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -446,19 +447,19 @@ OMX_ERRORTYPE Mpeg4CodecClose(EXYNOS_MPEG4DEC_HANDLE *pMpeg4Dec)
 
     if (hMFCHandle != NULL) {
         pDecOps->Finalize(hMFCHandle);
-        hMFCHandle = pMpeg4Dec->hMFCMpeg4Handle.hMFCHandle = NULL;
+        pMpeg4Dec->hMFCMpeg4Handle.hMFCHandle = NULL;
     }
     if (pOutbufOps != NULL) {
         Exynos_OSAL_Free(pOutbufOps);
-        pOutbufOps = pMpeg4Dec->hMFCMpeg4Handle.pOutbufOps = NULL;
+        pMpeg4Dec->hMFCMpeg4Handle.pOutbufOps = NULL;
     }
     if (pInbufOps != NULL) {
         Exynos_OSAL_Free(pInbufOps);
-        pInbufOps = pMpeg4Dec->hMFCMpeg4Handle.pInbufOps = NULL;
+        pMpeg4Dec->hMFCMpeg4Handle.pInbufOps = NULL;
     }
     if (pDecOps != NULL) {
         Exynos_OSAL_Free(pDecOps);
-        pDecOps = pMpeg4Dec->hMFCMpeg4Handle.pDecOps = NULL;
+        pMpeg4Dec->hMFCMpeg4Handle.pDecOps = NULL;
     }
 
     ret = OMX_ErrorNone;
@@ -479,6 +480,8 @@ OMX_ERRORTYPE Mpeg4CodecStart(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortInd
     EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = NULL;
     EXYNOS_MPEG4DEC_HANDLE   *pMpeg4Dec = NULL;
 
+    FunctionIn();
+
     if (pOMXComponent == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -489,6 +492,7 @@ OMX_ERRORTYPE Mpeg4CodecStart(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortInd
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
+
     pMpeg4Dec = (EXYNOS_MPEG4DEC_HANDLE *)pVideoDec->hCodecHandle;
     if (pMpeg4Dec == NULL) {
         ret = OMX_ErrorBadParameter;
@@ -505,7 +509,6 @@ OMX_ERRORTYPE Mpeg4CodecStart(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortInd
     else if (nPortIndex == OUTPUT_PORT_INDEX)
         pOutbufOps->Run(hMFCHandle);
 
-
     ret = OMX_ErrorNone;
 
 EXIT:
@@ -524,6 +527,8 @@ OMX_ERRORTYPE Mpeg4CodecStop(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortInde
     EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = NULL;
     EXYNOS_MPEG4DEC_HANDLE   *pMpeg4Dec = NULL;
 
+    FunctionIn();
+
     if (pOMXComponent == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -540,7 +545,6 @@ OMX_ERRORTYPE Mpeg4CodecStop(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortInde
         goto EXIT;
     }
 
-
     hMFCHandle = pMpeg4Dec->hMFCMpeg4Handle.hMFCHandle;
     pDecOps    = pMpeg4Dec->hMFCMpeg4Handle.pDecOps;
     pInbufOps  = pMpeg4Dec->hMFCMpeg4Handle.pInbufOps;
@@ -551,7 +555,6 @@ OMX_ERRORTYPE Mpeg4CodecStop(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortInde
     else if (nPortIndex == OUTPUT_PORT_INDEX)
         pOutbufOps->Stop(hMFCHandle);
 
-
     ret = OMX_ErrorNone;
 
 EXIT:
@@ -570,6 +573,8 @@ OMX_ERRORTYPE Mpeg4CodecOutputBufferProcessRun(OMX_COMPONENTTYPE *pOMXComponent,
     EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = NULL;
     EXYNOS_MPEG4DEC_HANDLE   *pMpeg4Dec = NULL;
 
+    FunctionIn();
+
     if (pOMXComponent == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -618,7 +623,7 @@ OMX_ERRORTYPE Mpeg4CodecEnQueueAllBuffer(OMX_COMPONENTTYPE *pOMXComponent, OMX_U
     OMX_ERRORTYPE                  ret = OMX_ErrorNone;
     EXYNOS_OMX_BASECOMPONENT      *pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = (EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle;
-    EXYNOS_MPEG4DEC_HANDLE         *pMpeg4Dec = (EXYNOS_MPEG4DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
+    EXYNOS_MPEG4DEC_HANDLE         *pMpeg4Dec = (EXYNOS_MPEG4DEC_HANDLE *)pVideoDec->hCodecHandle;
     void                          *hMFCHandle = pMpeg4Dec->hMFCMpeg4Handle.hMFCHandle;
     EXYNOS_OMX_BASEPORT           *pExynosInputPort = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
     EXYNOS_OMX_BASEPORT           *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
@@ -628,13 +633,13 @@ OMX_ERRORTYPE Mpeg4CodecEnQueueAllBuffer(OMX_COMPONENTTYPE *pOMXComponent, OMX_U
     ExynosVideoDecBufferOps *pInbufOps  = pMpeg4Dec->hMFCMpeg4Handle.pInbufOps;
     ExynosVideoDecBufferOps *pOutbufOps = pMpeg4Dec->hMFCMpeg4Handle.pOutbufOps;
 
+    FunctionIn();
 
     if ((nPortIndex == INPUT_PORT_INDEX) &&
         (pMpeg4Dec->bSourceStart == OMX_TRUE)) {
         Exynos_CodecBufferReset(pExynosComponent, INPUT_PORT_INDEX);
 
-        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++) {
             Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "pVideoDec->pMFCDecInputBuffer[%d]: 0x%x", i, pVideoDec->pMFCDecInputBuffer[i]);
             Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "pVideoDec->pMFCDecInputBuffer[%d]->VirAddr: 0x%x", i, pVideoDec->pMFCDecInputBuffer[i]->VirAddr);
 
@@ -685,7 +690,6 @@ OMX_ERRORTYPE Mpeg4CodecSrcSetup(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_DA
     OMX_U32                  inputBufferNumber = 0;
     int i;
 
-
     FunctionIn();
 
     if ((oneFrameSize <= 0) && (pSrcInputData->nFlags & OMX_BUFFERFLAG_EOS)) {
@@ -786,6 +790,7 @@ OMX_ERRORTYPE Mpeg4CodecSrcSetup(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_DA
     else
         pMpeg4Dec->hMFCMpeg4Handle.MFCOutputColorType = bufferConf.eColorFormat = VIDEO_COLORFORMAT_NV12_TILED;
 #endif
+
     if (pOutbufOps->Set_Geometry(hMFCHandle, &bufferConf) != VIDEO_ERROR_NONE) {
         Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "Failed to set geometry for output buffer");
         ret = OMX_ErrorInsufficientResources;
@@ -895,7 +900,6 @@ OMX_ERRORTYPE Mpeg4CodecDstSetup(OMX_COMPONENTTYPE *pOMXComponent)
 
     int i, nOutbufs;
 
-
     FunctionIn();
 
     /* get dpb count */
@@ -996,6 +1000,7 @@ OMX_ERRORTYPE Mpeg4CodecDstSetup(OMX_COMPONENTTYPE *pOMXComponent)
 //            Exynos_OSAL_UnlockANB(pExynosOutputPort->extendBufferHeader[i].OMXBufferHeader->pBuffer);
         }
     }
+
     if (pOutbufOps->Run(hMFCHandle) != VIDEO_ERROR_NONE) {
         Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "Failed to run output buffer");
         ret = OMX_ErrorInsufficientResources;
@@ -1003,7 +1008,7 @@ OMX_ERRORTYPE Mpeg4CodecDstSetup(OMX_COMPONENTTYPE *pOMXComponent)
     }
 
     if (pExynosOutputPort->bufferProcessType == BUFFER_SHARE) {
-        Mpeg4CodecStop (pOMXComponent, OUTPUT_PORT_INDEX);
+        Mpeg4CodecStop(pOMXComponent, OUTPUT_PORT_INDEX);
     }
     pMpeg4Dec->hMFCMpeg4Handle.bConfiguredMFCDst = OMX_TRUE;
 
@@ -1051,7 +1056,7 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_GetParameter(
     {
         OMX_VIDEO_PARAM_MPEG4TYPE *pDstMpeg4Param = (OMX_VIDEO_PARAM_MPEG4TYPE *)pComponentParameterStructure;
         OMX_VIDEO_PARAM_MPEG4TYPE *pSrcMpeg4Param = NULL;
-        EXYNOS_MPEG4DEC_HANDLE       *pMpeg4Dec      = NULL;
+        EXYNOS_MPEG4DEC_HANDLE    *pMpeg4Dec      = NULL;
 
         ret = Exynos_OMX_Check_SizeVersion(pDstMpeg4Param, sizeof(OMX_VIDEO_PARAM_MPEG4TYPE));
         if (ret != OMX_ErrorNone) {
@@ -1071,9 +1076,9 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_GetParameter(
         break;
     case OMX_IndexParamVideoH263:
     {
-        OMX_VIDEO_PARAM_H263TYPE  *pDstH263Param = (OMX_VIDEO_PARAM_H263TYPE *)pComponentParameterStructure;
-        OMX_VIDEO_PARAM_H263TYPE  *pSrcH263Param = NULL;
-        EXYNOS_MPEG4DEC_HANDLE       *pMpeg4Dec     = NULL;
+        OMX_VIDEO_PARAM_H263TYPE *pDstH263Param = (OMX_VIDEO_PARAM_H263TYPE *)pComponentParameterStructure;
+        OMX_VIDEO_PARAM_H263TYPE *pSrcH263Param = NULL;
+        EXYNOS_MPEG4DEC_HANDLE   *pMpeg4Dec     = NULL;
 
         ret = Exynos_OMX_Check_SizeVersion(pDstH263Param, sizeof(OMX_VIDEO_PARAM_H263TYPE));
         if (ret != OMX_ErrorNone) {
@@ -1149,7 +1154,7 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_GetParameter(
         OMX_VIDEO_PARAM_PROFILELEVELTYPE *pDstProfileLevel = (OMX_VIDEO_PARAM_PROFILELEVELTYPE *)pComponentParameterStructure;
         OMX_VIDEO_PARAM_MPEG4TYPE        *pSrcMpeg4Param   = NULL;
         OMX_VIDEO_PARAM_H263TYPE         *pSrcH263Param    = NULL;
-        EXYNOS_MPEG4DEC_HANDLE              *pMpeg4Dec        = NULL;
+        EXYNOS_MPEG4DEC_HANDLE           *pMpeg4Dec        = NULL;
         OMX_S32                           codecType;
 
         ret = Exynos_OMX_Check_SizeVersion(pDstProfileLevel, sizeof(OMX_VIDEO_PARAM_PROFILELEVELTYPE));
@@ -1179,7 +1184,7 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_GetParameter(
     {
         OMX_VIDEO_PARAM_ERRORCORRECTIONTYPE *pDstErrorCorrectionType = (OMX_VIDEO_PARAM_ERRORCORRECTIONTYPE *)pComponentParameterStructure;
         OMX_VIDEO_PARAM_ERRORCORRECTIONTYPE *pSrcErrorCorrectionType = NULL;
-        EXYNOS_MPEG4DEC_HANDLE                 *pMpeg4Dec               = NULL;
+        EXYNOS_MPEG4DEC_HANDLE              *pMpeg4Dec               = NULL;
 
         ret = Exynos_OMX_Check_SizeVersion(pDstErrorCorrectionType, sizeof(OMX_VIDEO_PARAM_ERRORCORRECTIONTYPE));
         if (ret != OMX_ErrorNone) {
@@ -1320,9 +1325,8 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_SetParameter(
         OMX_S32                           codecType;
 
         ret = Exynos_OMX_Check_SizeVersion(pSrcProfileLevel, sizeof(OMX_VIDEO_PARAM_PROFILELEVELTYPE));
-        if (ret != OMX_ErrorNone) {
+        if (ret != OMX_ErrorNone)
             goto EXIT;
-        }
 
         if (pSrcProfileLevel->nPortIndex >= ALL_PORT_NUM) {
             ret = OMX_ErrorBadPortIndex;
@@ -1410,7 +1414,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_GetConfig(
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     if (pExynosComponent->currentState == OMX_StateInvalid) {
         ret = OMX_ErrorInvalidState;
@@ -1453,7 +1456,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_SetConfig(
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     if (pExynosComponent->currentState == OMX_StateInvalid) {
         ret = OMX_ErrorInvalidState;
@@ -1492,13 +1494,11 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_GetExtensionIndex(
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     if (pOMXComponent->pComponentPrivate == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
-
     if ((cParameterName == NULL) || (pIndexType == NULL)) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -1510,9 +1510,7 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_GetExtensionIndex(
 
     if (Exynos_OSAL_Strcmp(cParameterName, EXYNOS_INDEX_PARAM_ENABLE_THUMBNAIL) == 0) {
         EXYNOS_MPEG4DEC_HANDLE *pMpeg4Dec = (EXYNOS_MPEG4DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
-
         *pIndexType = OMX_IndexVendorThumbnailMode;
-
         ret = OMX_ErrorNone;
     } else {
         ret = Exynos_OMX_VideoDecodeGetExtensionIndex(hComponent, cParameterName, pIndexType);
@@ -1574,22 +1572,23 @@ EXIT:
 /* MFC Init */
 OMX_ERRORTYPE Exynos_Mpeg4Dec_Init(OMX_COMPONENTTYPE *pOMXComponent)
 {
-    OMX_ERRORTYPE          ret = OMX_ErrorNone;
-    EXYNOS_OMX_BASECOMPONENT *pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
-    EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = (EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle;
-    EXYNOS_OMX_BASEPORT      *pExynosInputPort = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
-    EXYNOS_OMX_BASEPORT      *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
-    EXYNOS_MPEG4DEC_HANDLE    *pMpeg4Dec = (EXYNOS_MPEG4DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
-    OMX_PTR                hMFCHandle = pMpeg4Dec->hMFCMpeg4Handle.hMFCHandle;
+    OMX_ERRORTYPE                  ret               = OMX_ErrorNone;
+    EXYNOS_OMX_BASECOMPONENT      *pExynosComponent  = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
+    EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec         = (EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle;
+    EXYNOS_OMX_BASEPORT           *pExynosInputPort  = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
+    EXYNOS_OMX_BASEPORT           *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
+    EXYNOS_MPEG4DEC_HANDLE        *pMpeg4Dec         = (EXYNOS_MPEG4DEC_HANDLE *)pVideoDec->hCodecHandle;
+    OMX_PTR                        hMFCHandle        = pMpeg4Dec->hMFCMpeg4Handle.hMFCHandle;
 
     ExynosVideoDecOps       *pDecOps    = NULL;
     ExynosVideoDecBufferOps *pInbufOps  = NULL;
     ExynosVideoDecBufferOps *pOutbufOps = NULL;
 
     CSC_METHOD csc_method = CSC_METHOD_SW;
-
     int i = 0;
 
+    FunctionIn();
+
     pMpeg4Dec->hMFCMpeg4Handle.bConfiguredMFCSrc = OMX_FALSE;
     pMpeg4Dec->hMFCMpeg4Handle.bConfiguredMFCDst = OMX_FALSE;
     pExynosComponent->bUseFlagEOF = OMX_TRUE;
@@ -1609,8 +1608,7 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_Init(OMX_COMPONENTTYPE *pOMXComponent)
         Exynos_OSAL_SemaphoreCreate(&pExynosInputPort->codecSemID);
         Exynos_OSAL_QueueCreate(&pExynosInputPort->codecBufferQ, MAX_QUEUE_ELEMENTS);
 
-        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++) {
             /*************/
             /*    TBD    */
             /*************/
@@ -1632,13 +1630,10 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_Init(OMX_COMPONENTTYPE *pOMXComponent)
             Exynos_CodecBufferEnQueue(pExynosComponent, INPUT_PORT_INDEX, pVideoDec->pMFCDecInputBuffer[i]);
         }
     } else if (pExynosInputPort->bufferProcessType == BUFFER_SHARE) {
-
-
         /*************/
         /*    TBD    */
         /*************/
         /* Does not require any actions. */
-
     }
 
     if ((pExynosOutputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
@@ -1703,9 +1698,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
         pVideoDec->csc_handle = NULL;
     }
 
-//    pInbufOps->Stop(hMFCHandle);
-//    pOutbufOps->Stop(hMFCHandle);
-
     Exynos_OSAL_SignalTerminate(pMpeg4Dec->hDestinationStartEvent);
     pMpeg4Dec->hDestinationStartEvent = NULL;
     pMpeg4Dec->bDestinationStart = OMX_FALSE;
@@ -1714,8 +1706,7 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
     pMpeg4Dec->bSourceStart = OMX_FALSE;
 
     if ((pExynosOutputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
-        for (i = 0; i < MFC_OUTPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_OUTPUT_BUFFER_NUM_MAX; i++) {
             if (pVideoDec->pMFCDecOutputBuffer[i] != NULL) {
                 if (pVideoDec->pMFCDecOutputBuffer[i]->VirAddr[0] != NULL)
                     Exynos_OSAL_SharedMemory_Free(pVideoDec->hSharedMemory, pVideoDec->pMFCDecOutputBuffer[i]->VirAddr[0]);
@@ -1729,38 +1720,29 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
         Exynos_OSAL_QueueTerminate(&pExynosOutputPort->codecBufferQ);
         Exynos_OSAL_SemaphoreTerminate(pExynosOutputPort->codecSemID);
     } else if (pExynosOutputPort->bufferProcessType == BUFFER_SHARE) {
-
-
         /*************/
         /*    TBD    */
         /*************/
         /* Does not require any actions. */
-
-
     }
 
     if ((pExynosInputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
-        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++) {
             if (pVideoDec->pMFCDecInputBuffer[i] != NULL) {
                 if (pVideoDec->pMFCDecInputBuffer[i]->VirAddr != NULL)
-            Exynos_OSAL_SharedMemory_Free(pVideoDec->hSharedMemory, pVideoDec->pMFCDecInputBuffer[i]->VirAddr);
-            Exynos_OSAL_Free(pVideoDec->pMFCDecInputBuffer[i]);
-            pVideoDec->pMFCDecInputBuffer[i] = NULL;
-        }
+                    Exynos_OSAL_SharedMemory_Free(pVideoDec->hSharedMemory, pVideoDec->pMFCDecInputBuffer[i]->VirAddr);
+                Exynos_OSAL_Free(pVideoDec->pMFCDecInputBuffer[i]);
+                pVideoDec->pMFCDecInputBuffer[i] = NULL;
+            }
         }
 
         Exynos_OSAL_QueueTerminate(&pExynosInputPort->codecBufferQ);
         Exynos_OSAL_SemaphoreTerminate(pExynosInputPort->codecSemID);
     } else if (pExynosInputPort->bufferProcessType == BUFFER_SHARE) {
-
-
         /*************/
         /*    TBD    */
         /*************/
         /* Does not require any actions. */
-
-
     }
     Mpeg4CodecClose(pMpeg4Dec);
 
@@ -1797,7 +1779,7 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_SrcIn(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX
     }
 
     if ((Check_Stream_StartCode(pSrcInputData->buffer.singlePlaneBuffer.dataBuffer, oneFrameSize, pMpeg4Dec->hMFCMpeg4Handle.codecType) == OMX_TRUE) ||
-        ((pSrcInputData->nFlags & OMX_BUFFERFLAG_EOS) == OMX_BUFFERFLAG_EOS)){
+        ((pSrcInputData->nFlags & OMX_BUFFERFLAG_EOS) == OMX_BUFFERFLAG_EOS)) {
         pExynosComponent->timeStamp[pMpeg4Dec->hMFCMpeg4Handle.indexTimestamp] = pSrcInputData->timeStamp;
         pExynosComponent->nFlags[pMpeg4Dec->hMFCMpeg4Handle.indexTimestamp] = pSrcInputData->nFlags;
         Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "input timestamp %lld us (%.2f secs), Tag: %d, nFlags: 0x%x", pSrcInputData->timeStamp, pSrcInputData->timeStamp / 1E6, pMpeg4Dec->hMFCMpeg4Handle.indexTimestamp, pSrcInputData->nFlags);
@@ -1813,7 +1795,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_SrcIn(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX
             Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "%s : %d", __FUNCTION__, __LINE__);
             goto EXIT;
         }
-
         Mpeg4CodecStart(pOMXComponent, INPUT_PORT_INDEX);
         if (pMpeg4Dec->bSourceStart == OMX_FALSE) {
             pMpeg4Dec->bSourceStart = OMX_TRUE;
@@ -1830,7 +1811,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_SrcIn(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX
     ret = OMX_ErrorNone;
 
 EXIT:
-
     FunctionOut();
 
     return ret;
@@ -1873,17 +1853,14 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_SrcOut(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OM
             int i = 0;
             while (pSrcOutputData->buffer.singlePlaneBuffer.dataBuffer != pVideoDec->pMFCDecInputBuffer[i]->VirAddr) {
                 if (i >= MFC_INPUT_BUFFER_NUM_MAX) {
-                    ret = (OMX_ERRORTYPE)OMX_ErrorCodecDecode;
                     Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "Can not find buffer");
+                    ret = (OMX_ERRORTYPE)OMX_ErrorCodecDecode;
                     goto EXIT;
                 }
                 i++;
             }
-
             pVideoDec->pMFCDecInputBuffer[i]->dataSize = 0;
             pSrcOutputData->pPrivate = pVideoDec->pMFCDecInputBuffer[i];
-
-            Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "i: %d, out dataBuffer: 0x%x", i, pSrcOutputData->buffer.singlePlaneBuffer.dataBuffer);
         }
 
         /* For Share Buffer */
@@ -1920,8 +1897,8 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_DstIn(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX
 
     codecReturn = pOutbufOps->Enqueue(hMFCHandle, (unsigned char **)pDstInputData->buffer.multiPlaneBuffer.dataBuffer, (unsigned int *)&dataLen, 2, pDstInputData->bufferHeader);
     if (codecReturn != VIDEO_ERROR_NONE) {
-        ret = (OMX_ERRORTYPE)OMX_ErrorCodecDecode;
         Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "%s : %d", __FUNCTION__, __LINE__);
+        ret = (OMX_ERRORTYPE)OMX_ErrorCodecDecode;
         goto EXIT;
     }
     Mpeg4CodecStart(pOMXComponent, OUTPUT_PORT_INDEX);
@@ -1978,22 +1955,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_DstOut(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OM
                 break;
             }
         }
-/*
-        if ((CHECK_PORT_BEING_FLUSHED(pExynosOutputPort)) &&
-            (pVideoBuffer != NULL)) {
-            ret = OMX_ErrorNone;
-            break;
-        }
-*/
-#if 0
-        else if (displayStatus == VIDEO_FRAME_STATUS_DECODING_ONLY) {
-            if (pVideoBuffer != NULL) {
-                /* Just Flush Case */
-                ret = OMX_ErrorNone;
-                break;
-            }
-        }
-#endif
     }
 
     if (ret != OMX_ErrorNone)
@@ -2030,13 +1991,13 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_DstOut(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OM
     }
 
     indexTimestamp = pDecOps->Get_FrameTag(hMFCHandle);
-    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%x indexTimestamp: %d", pExynosComponent, indexTimestamp);
+    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "out indexTimestamp: %d", indexTimestamp);
     if ((indexTimestamp < 0) || (indexTimestamp >= MAX_TIMESTAMP)) {
         if ((pExynosComponent->checkTimeStamp.needSetStartTimeStamp != OMX_TRUE) &&
             (pExynosComponent->checkTimeStamp.needCheckStartTimeStamp != OMX_TRUE)) {
             pDstOutputData->timeStamp = pExynosComponent->timeStamp[pMpeg4Dec->hMFCMpeg4Handle.outputIndexTimestamp];
             pDstOutputData->nFlags = pExynosComponent->nFlags[pMpeg4Dec->hMFCMpeg4Handle.outputIndexTimestamp];
-            Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "indexTimestamp: %d", indexTimestamp);
+            Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "missing out indexTimestamp: %d", indexTimestamp);
         } else {
             pDstOutputData->timeStamp = 0x00;
             pDstOutputData->nFlags = 0x00;
@@ -2082,7 +2043,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_srcInputBufferProcess(OMX_COMPONENTTYPE *pOMXCompo
     EXYNOS_OMX_BASECOMPONENT *pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     EXYNOS_MPEG4DEC_HANDLE    *pMpeg4Dec = (EXYNOS_MPEG4DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
     EXYNOS_OMX_BASEPORT      *pExynosInputPort = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
-//    EXYNOS_OMX_BASEPORT      *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
 
     FunctionIn();
 
@@ -2114,7 +2074,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_srcOutputBufferProcess(OMX_COMPONENTTYPE *pOMXComp
     EXYNOS_OMX_BASECOMPONENT *pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     EXYNOS_MPEG4DEC_HANDLE   *pMpeg4Dec = (EXYNOS_MPEG4DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
     EXYNOS_OMX_BASEPORT     *pExynosInputPort = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
-//    EXYNOS_OMX_BASEPORT      *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
 
     FunctionIn();
 
@@ -2154,7 +2113,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_dstInputBufferProcess(OMX_COMPONENTTYPE *pOMXCompo
     OMX_ERRORTYPE             ret = OMX_ErrorNone;
     EXYNOS_OMX_BASECOMPONENT *pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     EXYNOS_MPEG4DEC_HANDLE    *pMpeg4Dec = (EXYNOS_MPEG4DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
-//    EXYNOS_OMX_BASEPORT      *pExynosInputPort = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
     EXYNOS_OMX_BASEPORT      *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
 
     FunctionIn();
@@ -2167,7 +2125,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_dstInputBufferProcess(OMX_COMPONENTTYPE *pOMXCompo
         ret = OMX_ErrorNone;
         goto EXIT;
     }
-
     if (pExynosOutputPort->bufferProcessType == BUFFER_SHARE) {
         if ((pMpeg4Dec->bDestinationStart == OMX_FALSE) &&
            (!CHECK_PORT_BEING_FLUSHED(pExynosOutputPort))) {
@@ -2195,7 +2152,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_dstOutputBufferProcess(OMX_COMPONENTTYPE *pOMXComp
     OMX_ERRORTYPE             ret = OMX_ErrorNone;
     EXYNOS_OMX_BASECOMPONENT *pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     EXYNOS_MPEG4DEC_HANDLE   *pMpeg4Dec = (EXYNOS_MPEG4DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
-//    EXYNOS_OMX_BASEPORT      *pExynosInputPort = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
     EXYNOS_OMX_BASEPORT     *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
 
     FunctionIn();
@@ -2216,7 +2172,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Dec_dstOutputBufferProcess(OMX_COMPONENTTYPE *pOMXComp
             Exynos_OSAL_SignalReset(pMpeg4Dec->hDestinationStartEvent);
         }
     }
-
     ret = Exynos_Mpeg4Dec_DstOut(pOMXComponent, pDstOutputData);
     if ((ret != OMX_ErrorNone) &&
         (pExynosComponent->currentState == OMX_StateExecuting)) {
@@ -2233,12 +2188,12 @@ EXIT:
 
 OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(OMX_HANDLETYPE hComponent, OMX_STRING componentName)
 {
-    OMX_ERRORTYPE                  ret = OMX_ErrorNone;
-    OMX_COMPONENTTYPE             *pOMXComponent = NULL;
+    OMX_ERRORTYPE                  ret              = OMX_ErrorNone;
+    OMX_COMPONENTTYPE             *pOMXComponent    = NULL;
     EXYNOS_OMX_BASECOMPONENT      *pExynosComponent = NULL;
-    EXYNOS_OMX_BASEPORT           *pExynosPort = NULL;
-    EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = NULL;
-    EXYNOS_MPEG4DEC_HANDLE         *pMpeg4Dec = NULL;
+    EXYNOS_OMX_BASEPORT           *pExynosPort      = NULL;
+    EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec        = NULL;
+    EXYNOS_MPEG4DEC_HANDLE        *pMpeg4Dec        = NULL;
     int i = 0;
     OMX_S32 codecType = -1;
 
@@ -2377,8 +2332,8 @@ OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(OMX_HANDLETYPE hComponent
     pVideoDec->exynos_codec_dstInputProcess  = &Exynos_Mpeg4Dec_dstInputBufferProcess;
     pVideoDec->exynos_codec_dstOutputProcess = &Exynos_Mpeg4Dec_dstOutputBufferProcess;
 
-    pVideoDec->exynos_codec_start         = &Mpeg4CodecStart;
-    pVideoDec->exynos_codec_stop          = &Mpeg4CodecStop;
+    pVideoDec->exynos_codec_start            = &Mpeg4CodecStart;
+    pVideoDec->exynos_codec_stop             = &Mpeg4CodecStop;
     pVideoDec->exynos_codec_bufferProcessRun = &Mpeg4CodecOutputBufferProcessRun;
     pVideoDec->exynos_codec_enqueueAllBuffer = &Mpeg4CodecEnQueueAllBuffer;
 
@@ -2392,6 +2347,7 @@ OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(OMX_HANDLETYPE hComponent
 
     pVideoDec->hSharedMemory = Exynos_OSAL_SharedMemory_Open();
     if (pVideoDec->hSharedMemory == NULL) {
+        Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "OMX_ErrorInsufficientResources, Line:%d", __LINE__);
         Exynos_OSAL_Free(pMpeg4Dec);
         pMpeg4Dec = ((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle = NULL;
         Exynos_OMX_VideoDecodeComponentDeinit(pOMXComponent);
index ba899a3e09fb86f047bece8be24c4fee235a82f7..fae31db3f01593bb3a60f4df987eb0c85ea9b27f 100644 (file)
@@ -169,8 +169,7 @@ OMX_BOOL Check_VP8_StartCode(
 
 OMX_ERRORTYPE VP8CodecOpen(EXYNOS_VP8DEC_HANDLE *pVp8Dec)
 {
-    OMX_ERRORTYPE           ret = OMX_ErrorNone;
-
+    OMX_ERRORTYPE            ret        = OMX_ErrorNone;
     ExynosVideoDecOps       *pDecOps    = NULL;
     ExynosVideoDecBufferOps *pInbufOps  = NULL;
     ExynosVideoDecBufferOps *pOutbufOps = NULL;
@@ -183,7 +182,6 @@ OMX_ERRORTYPE VP8CodecOpen(EXYNOS_VP8DEC_HANDLE *pVp8Dec)
         goto EXIT;
     }
 
-    /* MFC Open */
     /* alloc ops structure */
     pDecOps    = (ExynosVideoDecOps *)Exynos_OSAL_Malloc(sizeof(ExynosVideoDecOps));
     pInbufOps  = (ExynosVideoDecBufferOps *)Exynos_OSAL_Malloc(sizeof(ExynosVideoDecBufferOps));
@@ -205,6 +203,7 @@ OMX_ERRORTYPE VP8CodecOpen(EXYNOS_VP8DEC_HANDLE *pVp8Dec)
     pOutbufOps->nSize = sizeof(ExynosVideoDecBufferOps);
 
     Exynos_Video_Register_Decoder(pDecOps, pInbufOps, pOutbufOps);
+
     /* check mandatory functions for decoder ops */
     if ((pDecOps->Init == NULL) || (pDecOps->Finalize == NULL) ||
         (pDecOps->Get_ActualBufferCount == NULL) || (pDecOps->Set_FrameTag == NULL) ||
@@ -243,12 +242,14 @@ EXIT:
 
 OMX_ERRORTYPE VP8CodecClose(EXYNOS_VP8DEC_HANDLE *pVp8Dec)
 {
-    OMX_ERRORTYPE            ret = OMX_ErrorNone;
+    OMX_ERRORTYPE            ret        = OMX_ErrorNone;
     void                    *hMFCHandle = NULL;
     ExynosVideoDecOps       *pDecOps    = NULL;
     ExynosVideoDecBufferOps *pInbufOps  = NULL;
     ExynosVideoDecBufferOps *pOutbufOps = NULL;
 
+    FunctionIn();
+
     if (pVp8Dec == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -261,19 +262,19 @@ OMX_ERRORTYPE VP8CodecClose(EXYNOS_VP8DEC_HANDLE *pVp8Dec)
 
     if (hMFCHandle != NULL) {
         pDecOps->Finalize(hMFCHandle);
-        hMFCHandle = pVp8Dec->hMFCVp8Handle.hMFCHandle = NULL;
+        pVp8Dec->hMFCVp8Handle.hMFCHandle = NULL;
     }
     if (pOutbufOps != NULL) {
         Exynos_OSAL_Free(pOutbufOps);
-        pOutbufOps = pVp8Dec->hMFCVp8Handle.pOutbufOps = NULL;
+        pVp8Dec->hMFCVp8Handle.pOutbufOps = NULL;
     }
     if (pInbufOps != NULL) {
         Exynos_OSAL_Free(pInbufOps);
-        pInbufOps = pVp8Dec->hMFCVp8Handle.pInbufOps = NULL;
+        pVp8Dec->hMFCVp8Handle.pInbufOps = NULL;
     }
     if (pDecOps != NULL) {
         Exynos_OSAL_Free(pDecOps);
-        pDecOps = pVp8Dec->hMFCVp8Handle.pDecOps = NULL;
+        pVp8Dec->hMFCVp8Handle.pDecOps = NULL;
     }
 
     ret = OMX_ErrorNone;
@@ -294,6 +295,8 @@ OMX_ERRORTYPE VP8CodecStart(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortIndex
     EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = NULL;
     EXYNOS_VP8DEC_HANDLE   *pVp8Dec = NULL;
 
+    FunctionIn();
+
     if (pOMXComponent == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -304,6 +307,7 @@ OMX_ERRORTYPE VP8CodecStart(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortIndex
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
+
     pVp8Dec = (EXYNOS_VP8DEC_HANDLE *)pVideoDec->hCodecHandle;
     if (pVp8Dec == NULL) {
         ret = OMX_ErrorBadParameter;
@@ -320,7 +324,6 @@ OMX_ERRORTYPE VP8CodecStart(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortIndex
     else if (nPortIndex == OUTPUT_PORT_INDEX)
         pOutbufOps->Run(hMFCHandle);
 
-
     ret = OMX_ErrorNone;
 
 EXIT:
@@ -339,6 +342,8 @@ OMX_ERRORTYPE VP8CodecStop(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortIndex)
     EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = NULL;
     EXYNOS_VP8DEC_HANDLE   *pVp8Dec = NULL;
 
+    FunctionIn();
+
     if (pOMXComponent == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -355,7 +360,6 @@ OMX_ERRORTYPE VP8CodecStop(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortIndex)
         goto EXIT;
     }
 
-
     hMFCHandle = pVp8Dec->hMFCVp8Handle.hMFCHandle;
     pDecOps    = pVp8Dec->hMFCVp8Handle.pDecOps;
     pInbufOps  = pVp8Dec->hMFCVp8Handle.pInbufOps;
@@ -366,7 +370,6 @@ OMX_ERRORTYPE VP8CodecStop(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32 nPortIndex)
     else if (nPortIndex == OUTPUT_PORT_INDEX)
         pOutbufOps->Stop(hMFCHandle);
 
-
     ret = OMX_ErrorNone;
 
 EXIT:
@@ -385,6 +388,8 @@ OMX_ERRORTYPE VP8CodecOutputBufferProcessRun(OMX_COMPONENTTYPE *pOMXComponent, O
     EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = NULL;
     EXYNOS_VP8DEC_HANDLE   *pVp8Dec = NULL;
 
+    FunctionIn();
+
     if (pOMXComponent == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -443,13 +448,13 @@ OMX_ERRORTYPE VP8CodecEnQueueAllBuffer(OMX_COMPONENTTYPE *pOMXComponent, OMX_U32
     ExynosVideoDecBufferOps *pInbufOps  = pVp8Dec->hMFCVp8Handle.pInbufOps;
     ExynosVideoDecBufferOps *pOutbufOps = pVp8Dec->hMFCVp8Handle.pOutbufOps;
 
+    FunctionIn();
 
     if ((nPortIndex == INPUT_PORT_INDEX) &&
         (pVp8Dec->bSourceStart == OMX_TRUE)) {
         Exynos_CodecBufferReset(pExynosComponent, INPUT_PORT_INDEX);
 
-        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++) {
             Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "pVideoDec->pMFCDecInputBuffer[%d]: 0x%x", i, pVideoDec->pMFCDecInputBuffer[i]);
             Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "pVideoDec->pMFCDecInputBuffer[%d]->VirAddr: 0x%x", i, pVideoDec->pMFCDecInputBuffer[i]->VirAddr);
 
@@ -500,7 +505,6 @@ OMX_ERRORTYPE VP8CodecSrcSetup(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_DATA
     OMX_U32                  inputBufferNumber = 0;
     int i;
 
-
     FunctionIn();
 
     if ((oneFrameSize <= 0) && (pSrcInputData->nFlags & OMX_BUFFERFLAG_EOS)) {
@@ -597,6 +601,7 @@ OMX_ERRORTYPE VP8CodecSrcSetup(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_DATA
     else
         pVp8Dec->hMFCVp8Handle.MFCOutputColorType = bufferConf.eColorFormat = VIDEO_COLORFORMAT_NV12_TILED;
 #endif
+
     if (pOutbufOps->Set_Geometry(hMFCHandle, &bufferConf) != VIDEO_ERROR_NONE) {
         Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "Failed to set geometry for output buffer");
         ret = OMX_ErrorInsufficientResources;
@@ -604,7 +609,7 @@ OMX_ERRORTYPE VP8CodecSrcSetup(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_DATA
     }
 
     /* input buffer enqueue for header parsing */
-    Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "oneFrameSize: %d", oneFrameSize);
+    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "oneFrameSize: %d", oneFrameSize);
     if (pInbufOps->Enqueue(hMFCHandle, (unsigned char **)&pSrcInputData->buffer.singlePlaneBuffer.dataBuffer, (unsigned int *)&oneFrameSize, 1, pSrcInputData->bufferHeader) != VIDEO_ERROR_NONE) {
         Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "Failed to enqueue input buffer for header parsing");
 //        ret = OMX_ErrorInsufficientResources;
@@ -706,7 +711,6 @@ OMX_ERRORTYPE VP8CodecDstSetup(OMX_COMPONENTTYPE *pOMXComponent)
 
     int i, nOutbufs;
 
-
     FunctionIn();
 
     /* get dpb count */
@@ -1015,12 +1019,10 @@ OMX_ERRORTYPE Exynos_VP8Dec_GetConfig(
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     if (pOMXComponent->pComponentPrivate == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
 
     if (pComponentConfigStructure == NULL) {
@@ -1064,12 +1066,10 @@ OMX_ERRORTYPE Exynos_VP8Dec_SetConfig(
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     if (pOMXComponent->pComponentPrivate == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
 
     if (pComponentConfigStructure == NULL) {
@@ -1096,7 +1096,7 @@ EXIT:
 OMX_ERRORTYPE Exynos_VP8Dec_GetExtensionIndex(
     OMX_IN  OMX_HANDLETYPE  hComponent,
     OMX_IN  OMX_STRING      cParameterName,
-    OMX_OUT OMX_INDEXTYPE   *pIndexType)
+    OMX_OUT OMX_INDEXTYPE  *pIndexType)
 {
     OMX_ERRORTYPE             ret              = OMX_ErrorNone;
     OMX_COMPONENTTYPE        *pOMXComponent    = NULL;
@@ -1129,9 +1129,7 @@ OMX_ERRORTYPE Exynos_VP8Dec_GetExtensionIndex(
 
     if (Exynos_OSAL_Strcmp(cParameterName, EXYNOS_INDEX_PARAM_ENABLE_THUMBNAIL) == 0) {
         EXYNOS_VP8DEC_HANDLE *pVp8Dec = (EXYNOS_VP8DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
-
         *pIndexType = OMX_IndexVendorThumbnailMode;
-
         ret = OMX_ErrorNone;
     } else {
         ret = Exynos_OMX_VideoDecodeGetExtensionIndex(hComponent, cParameterName, pIndexType);
@@ -1174,22 +1172,23 @@ EXIT:
 /* MFC Init */
 OMX_ERRORTYPE Exynos_VP8Dec_Init(OMX_COMPONENTTYPE *pOMXComponent)
 {
-    OMX_ERRORTYPE          ret = OMX_ErrorNone;
-    EXYNOS_OMX_BASECOMPONENT *pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
-    EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec = (EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle;
-    EXYNOS_OMX_BASEPORT      *pExynosInputPort = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
-    EXYNOS_OMX_BASEPORT      *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
-    EXYNOS_VP8DEC_HANDLE    *pVp8Dec = (EXYNOS_VP8DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
-    OMX_PTR                hMFCHandle = pVp8Dec->hMFCVp8Handle.hMFCHandle;
+    OMX_ERRORTYPE                  ret               = OMX_ErrorNone;
+    EXYNOS_OMX_BASECOMPONENT      *pExynosComponent  = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
+    EXYNOS_OMX_VIDEODEC_COMPONENT *pVideoDec         = (EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle;
+    EXYNOS_OMX_BASEPORT           *pExynosInputPort  = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
+    EXYNOS_OMX_BASEPORT           *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
+    EXYNOS_VP8DEC_HANDLE          *pVp8Dec           = (EXYNOS_VP8DEC_HANDLE *)pVideoDec->hCodecHandle;
+    OMX_PTR                        hMFCHandle        = pVp8Dec->hMFCVp8Handle.hMFCHandle;
 
     ExynosVideoDecOps       *pDecOps    = NULL;
     ExynosVideoDecBufferOps *pInbufOps  = NULL;
     ExynosVideoDecBufferOps *pOutbufOps = NULL;
 
     CSC_METHOD csc_method = CSC_METHOD_SW;
-
     int i = 0;
 
+    FunctionIn();
+
     pVp8Dec->hMFCVp8Handle.bConfiguredMFCSrc = OMX_FALSE;
     pVp8Dec->hMFCVp8Handle.bConfiguredMFCDst = OMX_FALSE;
     pExynosComponent->bUseFlagEOF = OMX_TRUE;
@@ -1209,8 +1208,7 @@ OMX_ERRORTYPE Exynos_VP8Dec_Init(OMX_COMPONENTTYPE *pOMXComponent)
         Exynos_OSAL_SemaphoreCreate(&pExynosInputPort->codecSemID);
         Exynos_OSAL_QueueCreate(&pExynosInputPort->codecBufferQ, MAX_QUEUE_ELEMENTS);
 
-        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++) {
             /*************/
             /*    TBD    */
             /*************/
@@ -1232,13 +1230,10 @@ OMX_ERRORTYPE Exynos_VP8Dec_Init(OMX_COMPONENTTYPE *pOMXComponent)
             Exynos_CodecBufferEnQueue(pExynosComponent, INPUT_PORT_INDEX, pVideoDec->pMFCDecInputBuffer[i]);
         }
     } else if (pExynosInputPort->bufferProcessType == BUFFER_SHARE) {
-
-
         /*************/
         /*    TBD    */
         /*************/
         /* Does not require any actions. */
-
     }
 
     if ((pExynosOutputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
@@ -1303,9 +1298,6 @@ OMX_ERRORTYPE Exynos_VP8Dec_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
         pVideoDec->csc_handle = NULL;
     }
 
-//    pInbufOps->Stop(hMFCHandle);
-//    pOutbufOps->Stop(hMFCHandle);
-
     Exynos_OSAL_SignalTerminate(pVp8Dec->hDestinationStartEvent);
     pVp8Dec->hDestinationStartEvent = NULL;
     pVp8Dec->bDestinationStart = OMX_FALSE;
@@ -1314,8 +1306,7 @@ OMX_ERRORTYPE Exynos_VP8Dec_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
     pVp8Dec->bSourceStart = OMX_FALSE;
 
     if ((pExynosOutputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
-        for (i = 0; i < MFC_OUTPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_OUTPUT_BUFFER_NUM_MAX; i++) {
             if (pVideoDec->pMFCDecOutputBuffer[i] != NULL) {
                 if (pVideoDec->pMFCDecOutputBuffer[i]->VirAddr[0] != NULL)
                     Exynos_OSAL_SharedMemory_Free(pVideoDec->hSharedMemory, pVideoDec->pMFCDecOutputBuffer[i]->VirAddr[0]);
@@ -1329,38 +1320,29 @@ OMX_ERRORTYPE Exynos_VP8Dec_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
         Exynos_OSAL_QueueTerminate(&pExynosOutputPort->codecBufferQ);
         Exynos_OSAL_SemaphoreTerminate(pExynosOutputPort->codecSemID);
     } else if (pExynosOutputPort->bufferProcessType == BUFFER_SHARE) {
-
-
         /*************/
         /*    TBD    */
         /*************/
         /* Does not require any actions. */
-
-
     }
 
     if ((pExynosInputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
-        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++) {
             if (pVideoDec->pMFCDecInputBuffer[i] != NULL) {
                 if (pVideoDec->pMFCDecInputBuffer[i]->VirAddr != NULL)
-            Exynos_OSAL_SharedMemory_Free(pVideoDec->hSharedMemory, pVideoDec->pMFCDecInputBuffer[i]->VirAddr);
-            Exynos_OSAL_Free(pVideoDec->pMFCDecInputBuffer[i]);
-            pVideoDec->pMFCDecInputBuffer[i] = NULL;
-        }
+                    Exynos_OSAL_SharedMemory_Free(pVideoDec->hSharedMemory, pVideoDec->pMFCDecInputBuffer[i]->VirAddr);
+                Exynos_OSAL_Free(pVideoDec->pMFCDecInputBuffer[i]);
+                pVideoDec->pMFCDecInputBuffer[i] = NULL;
+            }
         }
 
         Exynos_OSAL_QueueTerminate(&pExynosInputPort->codecBufferQ);
         Exynos_OSAL_SemaphoreTerminate(pExynosInputPort->codecSemID);
     } else if (pExynosInputPort->bufferProcessType == BUFFER_SHARE) {
-
-
         /*************/
         /*    TBD    */
         /*************/
         /* Does not require any actions. */
-
-
     }
     VP8CodecClose(pVp8Dec);
 
@@ -1397,7 +1379,7 @@ OMX_ERRORTYPE Exynos_VP8Dec_SrcIn(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_D
     }
 
     if ((Check_VP8_StartCode(pSrcInputData->buffer.singlePlaneBuffer.dataBuffer, oneFrameSize) == OMX_TRUE) ||
-        ((pSrcInputData->nFlags & OMX_BUFFERFLAG_EOS) == OMX_BUFFERFLAG_EOS)){
+        ((pSrcInputData->nFlags & OMX_BUFFERFLAG_EOS) == OMX_BUFFERFLAG_EOS)) {
         pExynosComponent->timeStamp[pVp8Dec->hMFCVp8Handle.indexTimestamp] = pSrcInputData->timeStamp;
         pExynosComponent->nFlags[pVp8Dec->hMFCVp8Handle.indexTimestamp] = pSrcInputData->nFlags;
         Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "input timestamp %lld us (%.2f secs), Tag: %d, nFlags: 0x%x", pSrcInputData->timeStamp, pSrcInputData->timeStamp / 1E6, pVp8Dec->hMFCVp8Handle.indexTimestamp, pSrcInputData->nFlags);
@@ -1413,7 +1395,6 @@ OMX_ERRORTYPE Exynos_VP8Dec_SrcIn(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_D
             Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "%s : %d", __FUNCTION__, __LINE__);
             goto EXIT;
         }
-
         VP8CodecStart(pOMXComponent, INPUT_PORT_INDEX);
         if (pVp8Dec->bSourceStart == OMX_FALSE) {
             pVp8Dec->bSourceStart = OMX_TRUE;
@@ -1430,7 +1411,6 @@ OMX_ERRORTYPE Exynos_VP8Dec_SrcIn(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_D
     ret = OMX_ErrorNone;
 
 EXIT:
-
     FunctionOut();
 
     return ret;
@@ -1473,13 +1453,12 @@ OMX_ERRORTYPE Exynos_VP8Dec_SrcOut(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_
             int i = 0;
             while (pSrcOutputData->buffer.singlePlaneBuffer.dataBuffer != pVideoDec->pMFCDecInputBuffer[i]->VirAddr) {
                 if (i >= MFC_INPUT_BUFFER_NUM_MAX) {
-                    ret = (OMX_ERRORTYPE)OMX_ErrorCodecDecode;
                     Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "Can not find buffer");
+                    ret = (OMX_ERRORTYPE)OMX_ErrorCodecDecode;
                     goto EXIT;
                 }
                 i++;
             }
-
             pVideoDec->pMFCDecInputBuffer[i]->dataSize = 0;
             pSrcOutputData->pPrivate = pVideoDec->pMFCDecInputBuffer[i];
 
@@ -1520,8 +1499,8 @@ OMX_ERRORTYPE Exynos_VP8Dec_DstIn(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_D
 
     codecReturn = pOutbufOps->Enqueue(hMFCHandle, (unsigned char **)pDstInputData->buffer.multiPlaneBuffer.dataBuffer, (unsigned int *)&dataLen, 2, pDstInputData->bufferHeader);
     if (codecReturn != VIDEO_ERROR_NONE) {
-        ret = (OMX_ERRORTYPE)OMX_ErrorCodecDecode;
         Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "%s : %d", __FUNCTION__, __LINE__);
+        ret = (OMX_ERRORTYPE)OMX_ErrorCodecDecode;
         goto EXIT;
     }
     VP8CodecStart(pOMXComponent, OUTPUT_PORT_INDEX);
@@ -1578,22 +1557,6 @@ OMX_ERRORTYPE Exynos_VP8Dec_DstOut(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_
                 break;
             }
         }
-/*
-        if ((CHECK_PORT_BEING_FLUSHED(pExynosOutputPort)) &&
-            (pVideoBuffer != NULL)) {
-            ret = OMX_ErrorNone;
-            break;
-        }
-*/
-#if 0
-        else if (displayStatus == VIDEO_FRAME_STATUS_DECODING_ONLY) {
-            if (pVideoBuffer != NULL) {
-                /* Just Flush Case */
-                ret = OMX_ErrorNone;
-                break;
-            }
-        }
-#endif
     }
 
     if (ret != OMX_ErrorNone)
@@ -1630,13 +1593,13 @@ OMX_ERRORTYPE Exynos_VP8Dec_DstOut(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_
     }
 
     indexTimestamp = pDecOps->Get_FrameTag(hMFCHandle);
-    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%x indexTimestamp: %d", pExynosComponent, indexTimestamp);
+    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "out indexTimestamp: %d", indexTimestamp);
     if ((indexTimestamp < 0) || (indexTimestamp >= MAX_TIMESTAMP)) {
         if ((pExynosComponent->checkTimeStamp.needSetStartTimeStamp != OMX_TRUE) &&
             (pExynosComponent->checkTimeStamp.needCheckStartTimeStamp != OMX_TRUE)) {
             pDstOutputData->timeStamp = pExynosComponent->timeStamp[pVp8Dec->hMFCVp8Handle.outputIndexTimestamp];
             pDstOutputData->nFlags = pExynosComponent->nFlags[pVp8Dec->hMFCVp8Handle.outputIndexTimestamp];
-            Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "indexTimestamp: %d", indexTimestamp);
+            Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "missing out indexTimestamp: %d", indexTimestamp);
         } else {
             pDstOutputData->timeStamp = 0x00;
             pDstOutputData->nFlags = 0x00;
@@ -1662,7 +1625,7 @@ OMX_ERRORTYPE Exynos_VP8Dec_DstOut(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX_
 
     if ((displayStatus == VIDEO_FRAME_STATUS_CHANGE_RESOL) ||
         ((pDstOutputData->nFlags & OMX_BUFFERFLAG_EOS) == OMX_BUFFERFLAG_EOS)) {
-        Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%x displayStatus:%d, nFlags0x%x", pExynosComponent, displayStatus, pDstOutputData->nFlags);
+        Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "displayStatus:%d, nFlags0x%x", displayStatus, pDstOutputData->nFlags);
         pDstOutputData->remainDataLen = 0;
     } else {
         pDstOutputData->remainDataLen = bufferGeometry->nFrameWidth * bufferGeometry->nFrameHeight * 3 / 2;
@@ -1682,7 +1645,6 @@ OMX_ERRORTYPE Exynos_VP8Dec_srcInputBufferProcess(OMX_COMPONENTTYPE *pOMXCompone
     EXYNOS_OMX_BASECOMPONENT *pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     EXYNOS_VP8DEC_HANDLE    *pVp8Dec = (EXYNOS_VP8DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
     EXYNOS_OMX_BASEPORT      *pExynosInputPort = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
-//    EXYNOS_OMX_BASEPORT      *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
 
     FunctionIn();
 
@@ -1714,7 +1676,6 @@ OMX_ERRORTYPE Exynos_VP8Dec_srcOutputBufferProcess(OMX_COMPONENTTYPE *pOMXCompon
     EXYNOS_OMX_BASECOMPONENT *pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     EXYNOS_VP8DEC_HANDLE    *pVp8Dec = (EXYNOS_VP8DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
     EXYNOS_OMX_BASEPORT     *pExynosInputPort = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
-//    EXYNOS_OMX_BASEPORT      *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
 
     FunctionIn();
 
@@ -1754,7 +1715,6 @@ OMX_ERRORTYPE Exynos_VP8Dec_dstInputBufferProcess(OMX_COMPONENTTYPE *pOMXCompone
     OMX_ERRORTYPE             ret = OMX_ErrorNone;
     EXYNOS_OMX_BASECOMPONENT *pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     EXYNOS_VP8DEC_HANDLE    *pVp8Dec = (EXYNOS_VP8DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
-//    EXYNOS_OMX_BASEPORT      *pExynosInputPort = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
     EXYNOS_OMX_BASEPORT      *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
 
     FunctionIn();
@@ -1767,7 +1727,6 @@ OMX_ERRORTYPE Exynos_VP8Dec_dstInputBufferProcess(OMX_COMPONENTTYPE *pOMXCompone
         ret = OMX_ErrorNone;
         goto EXIT;
     }
-
     if (pExynosOutputPort->bufferProcessType == BUFFER_SHARE) {
         if ((pVp8Dec->bDestinationStart == OMX_FALSE) &&
            (!CHECK_PORT_BEING_FLUSHED(pExynosOutputPort))) {
@@ -1795,7 +1754,6 @@ OMX_ERRORTYPE Exynos_VP8Dec_dstOutputBufferProcess(OMX_COMPONENTTYPE *pOMXCompon
     OMX_ERRORTYPE             ret = OMX_ErrorNone;
     EXYNOS_OMX_BASECOMPONENT *pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     EXYNOS_VP8DEC_HANDLE    *pVp8Dec = (EXYNOS_VP8DEC_HANDLE *)((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
-//    EXYNOS_OMX_BASEPORT      *pExynosInputPort = &pExynosComponent->pExynosPort[INPUT_PORT_INDEX];
     EXYNOS_OMX_BASEPORT     *pExynosOutputPort = &pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX];
 
     FunctionIn();
@@ -1816,7 +1774,6 @@ OMX_ERRORTYPE Exynos_VP8Dec_dstOutputBufferProcess(OMX_COMPONENTTYPE *pOMXCompon
             Exynos_OSAL_SignalReset(pVp8Dec->hDestinationStartEvent);
         }
     }
-
     ret = Exynos_VP8Dec_DstOut(pOMXComponent, pDstOutputData);
     if ((ret != OMX_ErrorNone) &&
         (pExynosComponent->currentState == OMX_StateExecuting)) {
@@ -1948,8 +1905,8 @@ OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(
     pVideoDec->exynos_codec_dstInputProcess  = &Exynos_VP8Dec_dstInputBufferProcess;
     pVideoDec->exynos_codec_dstOutputProcess = &Exynos_VP8Dec_dstOutputBufferProcess;
 
-    pVideoDec->exynos_codec_start         = &VP8CodecStart;
-    pVideoDec->exynos_codec_stop          = &VP8CodecStop;
+    pVideoDec->exynos_codec_start            = &VP8CodecStart;
+    pVideoDec->exynos_codec_stop             = &VP8CodecStop;
     pVideoDec->exynos_codec_bufferProcessRun = &VP8CodecOutputBufferProcessRun;
     pVideoDec->exynos_codec_enqueueAllBuffer = &VP8CodecEnQueueAllBuffer;
 
@@ -1959,6 +1916,7 @@ OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(
 
     pVideoDec->hSharedMemory = Exynos_OSAL_SharedMemory_Open();
     if (pVideoDec->hSharedMemory == NULL) {
+        Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "OMX_ErrorInsufficientResources, Line:%d", __LINE__);
         Exynos_OSAL_Free(pVp8Dec);
         pVp8Dec = ((EXYNOS_OMX_VIDEODEC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle = NULL;
         Exynos_OMX_VideoDecodeComponentDeinit(pOMXComponent);
index e792b924ee9ece9feaf1d2debf7dc7b0b71c53c2..fea08f178aaf344f70294b3828103accb0ad1237 100644 (file)
@@ -151,8 +151,7 @@ void Exynos_Wait_ProcessPause(EXYNOS_OMX_BASECOMPONENT *pExynosComponent, OMX_U3
         (pExynosComponent->currentState == OMX_StateIdle) ||
         (pExynosComponent->transientState == EXYNOS_OMX_TransStateLoadedToIdle) ||
         (pExynosComponent->transientState == EXYNOS_OMX_TransStateExecutingToIdle)) &&
-        (pExynosComponent->transientState != EXYNOS_OMX_TransStateIdleToLoaded)&&
-//        ((!CHECK_PORT_BEING_FLUSHED(exynosOMXInputPort) && !CHECK_PORT_BEING_FLUSHED(exynosOMXOutputPort)))) {
+        (pExynosComponent->transientState != EXYNOS_OMX_TransStateIdleToLoaded) &&
         (!CHECK_PORT_BEING_FLUSHED(exynosOMXPort))) {
         Exynos_OSAL_SignalWait(pExynosComponent->pExynosPort[nPortIndex].pauseEvent, DEF_MAX_WAIT_TIME);
         Exynos_OSAL_SignalReset(pExynosComponent->pExynosPort[nPortIndex].pauseEvent);
@@ -387,19 +386,18 @@ OMX_BOOL Exynos_Preprocessor_InputData(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_
                                                         pExynosComponent->callbackData,
                                                         OMX_EventError, OMX_ErrorUndefined, 0, NULL);
                 flagEOF = OMX_FALSE;
-                // We don't need data-reset? // JH Chang
             }
 
             if (((exynosInputPort->bStoreMetaData == OMX_TRUE) && (eColorFormat == OMX_COLOR_FormatAndroidOpaque)) ||
                 (exynosInputPort->bStoreMetaData == OMX_FALSE)) {
-                    Exynos_InputBufferReturn(pOMXComponent);
+                Exynos_InputBufferReturn(pOMXComponent);
             } else {
                 inputUseBuffer->dataValid = OMX_TRUE;
             }
         }
 
         if ((srcInputData->nFlags & OMX_BUFFERFLAG_EOS) == OMX_BUFFERFLAG_EOS) {
-            Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "bSaveFlagEOS : OMX_TRUE");
+            Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "bSaveFlagEOS : OMX_TRUE");
             srcInputData->dataLen = 0;
             srcInputData->remainDataLen = 0;
             pExynosComponent->bSaveFlagEOS = OMX_TRUE;
@@ -451,13 +449,11 @@ OMX_BOOL Exynos_Postprocess_OutputData(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_
                 pExynosComponent->checkTimeStamp.needSetStartTimeStamp = OMX_FALSE;
                 pExynosComponent->checkTimeStamp.needCheckStartTimeStamp = OMX_FALSE;
             } else {
-                // We don't need data-reset? // JH Chang
                 Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "drop frame after seeking");
                 ret = OMX_TRUE;
                 goto EXIT;
             }
         } else if (pExynosComponent->checkTimeStamp.needSetStartTimeStamp == OMX_TRUE) {
-            // We don't need data-reset? // JH Chang
             ret = OMX_TRUE;
             Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "not set check timestame after seeking");
             goto EXIT;
@@ -477,13 +473,9 @@ OMX_BOOL Exynos_Postprocess_OutputData(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_
 
                 ret = OMX_TRUE;
 
-                // We don't need data-reset? // JH Chang
-
                 if ((outputUseBuffer->remainDataLen > 0) ||
                     (outputUseBuffer->nFlags & OMX_BUFFERFLAG_EOS)) {
-                    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%s:%d", __FUNCTION__, __LINE__);
                     Exynos_OutputBufferReturn(pOMXComponent);
-                    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%s:%d", __FUNCTION__, __LINE__);
                 }
             } else {
                 Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "output buffer is smaller than encoded data size Out Length");
@@ -491,7 +483,6 @@ OMX_BOOL Exynos_Postprocess_OutputData(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_
                                                         pExynosComponent->callbackData,
                                                         OMX_EventError, OMX_ErrorUndefined, 0, NULL);
                 ret = OMX_FALSE;
-                // We don't need data-reset? // JH Chang
             }
         } else if (exynosOutputPort->bufferProcessType == BUFFER_SHARE) {
             if ((outputUseBuffer->remainDataLen > 0) ||
@@ -570,7 +561,6 @@ OMX_ERRORTYPE Exynos_OMX_SrcInputBufferProcess(OMX_HANDLETYPE hComponent)
             ret = pVideoEnc->exynos_codec_srcInputProcess(pOMXComponent, pSrcInputData);
             Exynos_ResetCodecData(pSrcInputData);
             Exynos_OSAL_MutexUnlock(srcInputUseBuffer->bufferMutex);
-
             if (ret == OMX_ErrorCodecInit)
                 pVideoEnc->bExitBufferProcessThread = OMX_TRUE;
         }
@@ -618,7 +608,6 @@ OMX_ERRORTYPE Exynos_OMX_SrcOutputBufferProcess(OMX_HANDLETYPE hComponent)
                     if (codecBuffer != NULL)
                         Exynos_CodecBufferEnQueue(pExynosComponent, INPUT_PORT_INDEX, codecBuffer);
                 }
-
                 if (exynosInputPort->bufferProcessType == BUFFER_SHARE) {
                     Exynos_Shared_DataToBuffer(&srcOutputData, srcOutputUseBuffer);
                     Exynos_InputBufferReturn(pOMXComponent);
@@ -755,7 +744,6 @@ OMX_ERRORTYPE Exynos_OMX_DstOutputBufferProcess(OMX_HANDLETYPE hComponent)
 
             if ((exynosOutputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
                 OMX_PTR codecBuffer;
-                //ENCODE_CODEC_EXTRA_BUFFERINFO *pBufferInfo = (ENCODE_CODEC_EXTRA_BUFFERINFO *)&pDstOutputData->extInfo;
                 codecBuffer = pDstOutputData->pPrivate;
                 if (codecBuffer != NULL) {
                     Exynos_CodecBufferEnQueue(pExynosComponent, OUTPUT_PORT_INDEX, codecBuffer);
@@ -789,13 +777,11 @@ static OMX_ERRORTYPE Exynos_OMX_SrcInputProcessThread(OMX_PTR threadData)
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pOMXComponent = (OMX_COMPONENTTYPE *)threadData;
     ret = Exynos_OMX_Check_SizeVersion(pOMXComponent, sizeof(OMX_COMPONENTTYPE));
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     Exynos_OMX_SrcInputBufferProcess(pOMXComponent);
 
@@ -820,13 +806,11 @@ static OMX_ERRORTYPE Exynos_OMX_SrcOutputProcessThread(OMX_PTR threadData)
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pOMXComponent = (OMX_COMPONENTTYPE *)threadData;
     ret = Exynos_OMX_Check_SizeVersion(pOMXComponent, sizeof(OMX_COMPONENTTYPE));
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     Exynos_OMX_SrcOutputBufferProcess(pOMXComponent);
 
@@ -851,13 +835,11 @@ static OMX_ERRORTYPE Exynos_OMX_DstInputProcessThread(OMX_PTR threadData)
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pOMXComponent = (OMX_COMPONENTTYPE *)threadData;
     ret = Exynos_OMX_Check_SizeVersion(pOMXComponent, sizeof(OMX_COMPONENTTYPE));
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     Exynos_OMX_DstInputBufferProcess(pOMXComponent);
 
@@ -882,13 +864,11 @@ static OMX_ERRORTYPE Exynos_OMX_DstOutputProcessThread(OMX_PTR threadData)
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pOMXComponent = (OMX_COMPONENTTYPE *)threadData;
     ret = Exynos_OMX_Check_SizeVersion(pOMXComponent, sizeof(OMX_COMPONENTTYPE));
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     Exynos_OMX_DstOutputBufferProcess(pOMXComponent);
 
@@ -966,14 +946,12 @@ OMX_ERRORTYPE Exynos_OMX_BufferProcess_Terminate(OMX_HANDLETYPE hComponent)
     Exynos_OSAL_ThreadTerminate(pVideoEnc->hDstInputThread);
     pVideoEnc->hDstInputThread = NULL;
 
-    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%s:%d", __FUNCTION__, __LINE__);
     pVideoEnc->exynos_codec_stop(pOMXComponent, INPUT_PORT_INDEX);
     pVideoEnc->exynos_codec_bufferProcessRun(pOMXComponent, INPUT_PORT_INDEX);
     Exynos_OSAL_SignalSet(pExynosComponent->pExynosPort[INPUT_PORT_INDEX].pauseEvent);
     Exynos_OSAL_ThreadTerminate(pVideoEnc->hSrcOutputThread);
     pVideoEnc->hSrcOutputThread = NULL;
 
-    Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "%s:%d", __FUNCTION__, __LINE__);
     pVideoEnc->exynos_codec_stop(pOMXComponent, OUTPUT_PORT_INDEX);
     pVideoEnc->exynos_codec_bufferProcessRun(pOMXComponent, INPUT_PORT_INDEX);
     Exynos_OSAL_SignalSet(pExynosComponent->pExynosPort[OUTPUT_PORT_INDEX].pauseEvent);
index 4a349621242005275c696387cde6486436210e10..88bce4f5d2ad0e3e2e4b25148544ffb0579bf583 100644 (file)
@@ -55,6 +55,9 @@
 #define INPUT_PORT_SUPPORTFORMAT_NUM_MAX    5
 #define OUTPUT_PORT_SUPPORTFORMAT_NUM_MAX   1
 
+#define MFC_INPUT_BUFFER_PLANE              2
+#define MFC_OUTPUT_BUFFER_PLANE             1
+
 // The largest metadata buffer size advertised
 // when metadata buffer mode is used for video encoding
 #define  MAX_INPUT_METADATA_BUFFER_SIZE (64)
@@ -85,7 +88,6 @@ typedef struct _CODEC_ENC_OUTPUT_BUFFER
 typedef struct _EXYNOS_OMX_VIDEOENC_COMPONENT
 {
     OMX_HANDLETYPE hCodecHandle;
-//    EXYNOS_CODEC_NBENC_THREAD NBEncThread;
     OMX_BOOL bFirstFrame;
     CODEC_ENC_INPUT_BUFFER *pMFCEncInputBuffer[MFC_INPUT_BUFFER_NUM_MAX];
     CODEC_ENC_OUTPUT_BUFFER *pMFCEncOutputBuffer[MFC_OUTPUT_BUFFER_NUM_MAX];
index fb98e3444dfdd90050d748fec83efef7963bb03c..ad2bfc3499b156cf7bbd66110857ef65e8e212db 100644 (file)
@@ -305,8 +305,7 @@ OMX_ERRORTYPE Exynos_OMX_FreeBuffer(
                         nPortIndex, NULL);
     }
 
-    for (i = 0; i < pExynosPort->portDefinition.nBufferCountActual; i++) { // JH Chang for test
-//    for (i = 0; i < /*pExynosPort->portDefinition.nBufferCountActual*/MAX_BUFFER_NUM; i++) {
+    for (i = 0; i < /*pExynosPort->portDefinition.nBufferCountActual*/MAX_BUFFER_NUM; i++) {
         if (((pExynosPort->bufferStateAllocate[i] | BUFFER_STATE_FREE) != 0) && (pExynosPort->extendBufferHeader[i].OMXBufferHeader != NULL)) {
             if (pExynosPort->extendBufferHeader[i].OMXBufferHeader->pBuffer == pBufferHdr->pBuffer) {
                 if (pExynosPort->bufferStateAllocate[i] & BUFFER_STATE_ALLOCATED) {
@@ -710,7 +709,6 @@ OMX_ERRORTYPE Exynos_InputBufferGetQueue(EXYNOS_OMX_BASECOMPONENT *pExynosCompon
                 ret = OMX_ErrorUndefined;
                 goto EXIT;
             }
-
             if (message->messageType == EXYNOS_OMX_CommandFakeBuffer) {
                 ret = OMX_ErrorCodecFlush;
                 goto EXIT;
@@ -819,7 +817,6 @@ OMX_ERRORTYPE Exynos_FlushOutputBufferReturn(OMX_COMPONENTTYPE *pOMXComponent, E
                             OUTPUT_PORT_INDEX,
                             bufferHeader->nFlags, NULL);
         }
-
         Exynos_OMX_OutputBufferReturn(pOMXComponent, bufferHeader);
     }
 
@@ -1380,7 +1377,6 @@ OMX_ERRORTYPE Exynos_OMX_VideoEncodeGetConfig(
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     if (pExynosComponent->currentState == OMX_StateInvalid) {
         ret = OMX_ErrorInvalidState;
@@ -1453,7 +1449,6 @@ OMX_ERRORTYPE Exynos_OMX_VideoEncodeSetConfig(
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     if (pExynosComponent->currentState == OMX_StateInvalid) {
         ret = OMX_ErrorInvalidState;
index b5d1fd66d5ecc4b15fa6cf1240ebdd28d06a4d38..beae30dd6f7a56671d717487f160cc72f067380b 100644 (file)
@@ -316,7 +316,6 @@ static void Set_H264Enc_Param(EXYNOS_OMX_BASECOMPONENT *pExynosComponent)
     }
 
     Print_H264Enc_Param(pEncParam);
-
 }
 
 static void Change_H264Enc_Param(EXYNOS_OMX_BASECOMPONENT *pExynosComponent)
@@ -366,7 +365,6 @@ static void Change_H264Enc_Param(EXYNOS_OMX_BASECOMPONENT *pExynosComponent)
     Set_H264Enc_Param(pExynosComponent);
 }
 
-
 OMX_ERRORTYPE GetCodecInputPrivateData(OMX_PTR codecBuffer, OMX_PTR addr[], OMX_U32 size[])
 {
     OMX_ERRORTYPE       ret = OMX_ErrorNone;
@@ -375,7 +373,6 @@ EXIT:
     return ret;
 }
 
-
 OMX_ERRORTYPE GetCodecOutputPrivateData(OMX_PTR codecBuffer, OMX_PTR *pVirtAddr, OMX_U32 *dataSize)
 {
     OMX_ERRORTYPE       ret = OMX_ErrorNone;
@@ -416,7 +413,6 @@ OMX_ERRORTYPE H264CodecOpen(EXYNOS_H264ENC_HANDLE *pH264Enc)
         goto EXIT;
     }
 
-    /* MFC Open */
     /* alloc ops structure */
     pEncOps = (ExynosVideoEncOps *)Exynos_OSAL_Malloc(sizeof(ExynosVideoEncOps));
     pInbufOps = (ExynosVideoEncBufferOps *)Exynos_OSAL_Malloc(sizeof(ExynosVideoEncBufferOps));
@@ -482,6 +478,8 @@ OMX_ERRORTYPE H264CodecClose(EXYNOS_H264ENC_HANDLE *pH264Enc)
     ExynosVideoEncBufferOps *pInbufOps  = NULL;
     ExynosVideoEncBufferOps *pOutbufOps = NULL;
 
+    FunctionIn();
+
     if (pH264Enc == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -684,12 +682,7 @@ OMX_ERRORTYPE H264CodecEnQueueAllBuffer(OMX_COMPONENTTYPE *pOMXComponent, OMX_U3
 
     if ((nPortIndex == INPUT_PORT_INDEX) &&
         (pH264Enc->bSourceStart == OMX_TRUE)) {
-#if 0
-        while (Exynos_OSAL_GetElemNum(&pExynosInputPort->codecBufferQ) > 0)
-            Exynos_CodecBufferDeQueue(pExynosComponent, INPUT_PORT_INDEX, &pDeQ);
-#else
         Exynos_CodecBufferReset(pExynosComponent, INPUT_PORT_INDEX);
-#endif
 
         for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++)  {
             Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "pVideoEnc->pMFCEncInputBuffer[%d]: 0x%x", i, pVideoEnc->pMFCEncInputBuffer[i]);
@@ -704,13 +697,8 @@ OMX_ERRORTYPE H264CodecEnQueueAllBuffer(OMX_COMPONENTTYPE *pOMXComponent, OMX_U3
                (pH264Enc->bDestinationStart == OMX_TRUE)) {
         OMX_U32 dataLen[2] = {0, 0};
         ExynosVideoBuffer *pBuffer = NULL;
-#if 0
-        while (Exynos_OSAL_GetElemNum(&pExynosOutputPort->codecBufferQ) > 0)
-            Exynos_CodecBufferDeQueue(pExynosComponent, OUTPUT_PORT_INDEX, &pDeQ);
-#else
-        Exynos_CodecBufferReset(pExynosComponent, OUTPUT_PORT_INDEX);
-#endif
 
+        Exynos_CodecBufferReset(pExynosComponent, OUTPUT_PORT_INDEX);
 
         for (i = 0; i < MFC_OUTPUT_BUFFER_NUM_MAX; i++) {
             pOutbufOps->Get_Buffer(hMFCHandle, i, &pBuffer);
@@ -1099,9 +1087,8 @@ OMX_ERRORTYPE Exynos_H264Enc_SetParameter(
         EXYNOS_H264ENC_HANDLE *pH264Enc = NULL;
 
         ret = Exynos_OMX_Check_SizeVersion(pSrcProfileLevel, sizeof(OMX_VIDEO_PARAM_PROFILELEVELTYPE));
-        if (ret != OMX_ErrorNone) {
+        if (ret != OMX_ErrorNone)
             goto EXIT;
-        }
 
         if (pSrcProfileLevel->nPortIndex >= ALL_PORT_NUM) {
             ret = OMX_ErrorBadPortIndex;
@@ -1176,7 +1163,6 @@ OMX_ERRORTYPE Exynos_H264Enc_GetConfig(
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     if (pExynosComponent->currentState == OMX_StateInvalid) {
         ret = OMX_ErrorInvalidState;
@@ -1236,14 +1222,13 @@ OMX_ERRORTYPE Exynos_H264Enc_SetConfig(
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     if (pExynosComponent->currentState == OMX_StateInvalid) {
         ret = OMX_ErrorInvalidState;
         goto EXIT;
     }
 
-    pVideoEnc = ((EXYNOS_OMX_VIDEOENC_COMPONENT *)pExynosComponent->hComponentHandle);
+    pVideoEnc = (EXYNOS_OMX_VIDEOENC_COMPONENT *)pExynosComponent->hComponentHandle;
     pH264Enc = (EXYNOS_H264ENC_HANDLE *)((EXYNOS_OMX_VIDEOENC_COMPONENT *)pExynosComponent->hComponentHandle)->hCodecHandle;
 
     switch (nIndex) {
@@ -1309,13 +1294,11 @@ OMX_ERRORTYPE Exynos_H264Enc_GetExtensionIndex(
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     if (pOMXComponent->pComponentPrivate == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
-
     if ((cParameterName == NULL) || (pIndexType == NULL)) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -1397,11 +1380,10 @@ OMX_ERRORTYPE Exynos_H264Enc_Init(OMX_COMPONENTTYPE *pOMXComponent)
     pExynosComponent->bSaveFlagEOS = OMX_FALSE;
 
     eColorFormat = pExynosInputPort->portDefinition.format.video.eColorFormat;
-
     if (pExynosInputPort->bStoreMetaData == OMX_TRUE) {
         if (eColorFormat == OMX_COLOR_FormatAndroidOpaque) {
             pExynosInputPort->bufferProcessType = BUFFER_COPY;
-    } else {
+        } else {
             pExynosInputPort->bufferProcessType = BUFFER_SHARE;
         }
     }
@@ -1632,7 +1614,6 @@ OMX_ERRORTYPE Exynos_H264Enc_Init(OMX_COMPONENTTYPE *pOMXComponent)
 
     pH264Enc->bSourceStart = OMX_FALSE;
     Exynos_OSAL_SignalCreate(&pH264Enc->hSourceStartEvent);
-
     pH264Enc->bDestinationStart = OMX_FALSE;
     Exynos_OSAL_SignalCreate(&pH264Enc->hDestinationStartEvent);
 
@@ -1644,7 +1625,7 @@ OMX_ERRORTYPE Exynos_H264Enc_Init(OMX_COMPONENTTYPE *pOMXComponent)
     pExynosComponent->getAllDelayBuffer = OMX_FALSE;
 
 #if 0//defined(USE_CSC_GSCALER)
-    csc_method = CSC_METHOD_TRY_HW; //in case of Use ION buffer.
+    csc_method = CSC_METHOD_HW; //in case of Use ION buffer.
 #endif
     pVideoEnc->csc_handle = csc_init(csc_method);
     if (pVideoEnc->csc_handle == NULL) {
@@ -1683,9 +1664,6 @@ OMX_ERRORTYPE Exynos_H264Enc_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
         pVideoEnc->csc_handle = NULL;
     }
 
-//    pInbufOps->Stop(hMFCHandle);
-//    pOutbufOps->Stop(hMFCHandle);
-
     Exynos_OSAL_SignalTerminate(pH264Enc->hDestinationStartEvent);
     pH264Enc->hDestinationStartEvent = NULL;
     pH264Enc->bDestinationStart = OMX_FALSE;
@@ -1705,7 +1683,7 @@ OMX_ERRORTYPE Exynos_H264Enc_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
             }
         }
 
-       Exynos_OSAL_QueueTerminate(&pExynosInputPort->codecBufferQ);
+        Exynos_OSAL_QueueTerminate(&pExynosInputPort->codecBufferQ);
         Exynos_OSAL_SemaphoreTerminate(pExynosInputPort->codecSemID);
     } else if (pExynosInputPort->bufferProcessType == BUFFER_SHARE) {
         /*************/
@@ -1715,8 +1693,7 @@ OMX_ERRORTYPE Exynos_H264Enc_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
     }
 
     if ((pExynosOutputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
-        for (i = 0; i < MFC_OUTPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_OUTPUT_BUFFER_NUM_MAX; i++) {
             if (pVideoEnc->pMFCEncOutputBuffer[i] != NULL) {
                 if (pVideoEnc->pMFCEncOutputBuffer[i]->VirAddr != NULL)
                     Exynos_OSAL_SharedMemory_Free(pVideoEnc->hSharedMemory, pVideoEnc->pMFCEncOutputBuffer[i]->VirAddr);
@@ -1969,12 +1946,12 @@ OMX_ERRORTYPE Exynos_H264Enc_DstOut(OMX_COMPONENTTYPE *pOMXComponent, EXYNOS_OMX
         pH264Enc->hMFCH264Handle.headerData.pHeaderSPS =
             (OMX_PTR)pDstOutputData->buffer.singlePlaneBuffer.dataBuffer;
         pH264Enc->hMFCH264Handle.headerData.SPSLen = iSpsSize;
-        
+
         iPpsSize = pDstOutputData->dataLen - iSpsSize;
         pH264Enc->hMFCH264Handle.headerData.pHeaderPPS =
             (OMX_U8 *)pDstOutputData->buffer.singlePlaneBuffer.dataBuffer + iSpsSize;
         pH264Enc->hMFCH264Handle.headerData.PPSLen = iPpsSize;
-        
+
         pDstOutputData->timeStamp = 0;
         pDstOutputData->nFlags |= OMX_BUFFERFLAG_CODECCONFIG;
         pDstOutputData->nFlags |= OMX_BUFFERFLAG_ENDOFFRAME;
@@ -2058,7 +2035,6 @@ OMX_ERRORTYPE Exynos_H264Enc_srcOutputBufferProcess(OMX_COMPONENTTYPE *pOMXCompo
             goto EXIT;
         }
     }
-
     if ((pH264Enc->bSourceStart == OMX_FALSE) &&
        (!CHECK_PORT_BEING_FLUSHED(pExynosInputPort))) {
         Exynos_OSAL_SignalWait(pH264Enc->hSourceStartEvent, DEF_MAX_WAIT_TIME);
@@ -2096,7 +2072,6 @@ OMX_ERRORTYPE Exynos_H264Enc_dstInputBufferProcess(OMX_COMPONENTTYPE *pOMXCompon
         ret = OMX_ErrorNone;
         goto EXIT;
     }
-
     if (pExynosOutputPort->bufferProcessType == BUFFER_SHARE) {
         if ((pH264Enc->bDestinationStart == OMX_FALSE) &&
            (!CHECK_PORT_BEING_FLUSHED(pExynosOutputPort))) {
@@ -2104,7 +2079,6 @@ OMX_ERRORTYPE Exynos_H264Enc_dstInputBufferProcess(OMX_COMPONENTTYPE *pOMXCompon
             Exynos_OSAL_SignalReset(pH264Enc->hDestinationStartEvent);
         }
     }
-
     if (pH264Enc->hMFCH264Handle.bConfiguredMFCDst == OMX_TRUE) {
         ret = Exynos_H264Enc_DstIn(pOMXComponent, pDstInputData);
         if (ret != OMX_ErrorNone) {
@@ -2146,7 +2120,6 @@ OMX_ERRORTYPE Exynos_H264Enc_dstOutputBufferProcess(OMX_COMPONENTTYPE *pOMXCompo
             Exynos_OSAL_SignalReset(pH264Enc->hDestinationStartEvent);
         }
     }
-
     ret = Exynos_H264Enc_DstOut(pOMXComponent, pDstOutputData);
     if ((ret != OMX_ErrorNone) && (pExynosComponent->currentState == OMX_StateExecuting)) {
         Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "%s: %d: Failed - DstOut -> event is thrown to client", __FUNCTION__, __LINE__);
@@ -2289,7 +2262,6 @@ OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(OMX_HANDLETYPE hComponent
     pVideoEnc->exynos_checkInputFrame        = NULL;
     pVideoEnc->exynos_codec_getCodecInputPrivateData  = &GetCodecInputPrivateData;
     pVideoEnc->exynos_codec_getCodecOutputPrivateData = &GetCodecOutputPrivateData;
-//    pVideoEnc->exynos_codec_getCodecBufferNumber      = &GetCodecBufferNumber;
 
     pVideoEnc->hSharedMemory = Exynos_OSAL_SharedMemory_Open();
     if (pVideoEnc->hSharedMemory == NULL) {
index 57bfc965863513d2469a6b4d096bb1b8a695e759..e77335ef0acce84124bde8b8d180d0011f3a165b 100644 (file)
@@ -530,7 +530,6 @@ OMX_ERRORTYPE Mpeg4CodecOpen(EXYNOS_MPEG4ENC_HANDLE *pMpeg4Enc)
         goto EXIT;
     }
 
-    /* MFC Open */
     /* alloc ops structure */
     pEncOps = (ExynosVideoEncOps *)Exynos_OSAL_Malloc(sizeof(ExynosVideoEncOps));
     pInbufOps = (ExynosVideoEncBufferOps *)Exynos_OSAL_Malloc(sizeof(ExynosVideoEncBufferOps));
@@ -596,6 +595,8 @@ OMX_ERRORTYPE Mpeg4CodecClose(EXYNOS_MPEG4ENC_HANDLE *pMpeg4Enc)
     ExynosVideoEncBufferOps *pInbufOps  = NULL;
     ExynosVideoEncBufferOps *pOutbufOps = NULL;
 
+    FunctionIn();
+
     if (pMpeg4Enc == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -798,12 +799,7 @@ OMX_ERRORTYPE Mpeg4CodecEnQueueAllBuffer(OMX_COMPONENTTYPE *pOMXComponent, OMX_U
 
     if ((nPortIndex == INPUT_PORT_INDEX) &&
         (pMpeg4Enc->bSourceStart == OMX_TRUE)) {
-#if 0
-        while (Exynos_OSAL_GetElemNum(&pExynosInputPort->codecBufferQ) > 0)
-            Exynos_CodecBufferDeQueue(pExynosComponent, INPUT_PORT_INDEX, &pDeQ);
-#else
         Exynos_CodecBufferReset(pExynosComponent, INPUT_PORT_INDEX);
-#endif
 
         for (i = 0; i < MFC_INPUT_BUFFER_NUM_MAX; i++)  {
             Exynos_OSAL_Log(EXYNOS_LOG_TRACE, "pVideoEnc->pMFCEncInputBuffer[%d]: 0x%x", i, pVideoEnc->pMFCEncInputBuffer[i]);
@@ -818,13 +814,8 @@ OMX_ERRORTYPE Mpeg4CodecEnQueueAllBuffer(OMX_COMPONENTTYPE *pOMXComponent, OMX_U
                (pMpeg4Enc->bDestinationStart == OMX_TRUE)) {
         OMX_U32 dataLen[2] = {0, 0};
         ExynosVideoBuffer *pBuffer = NULL;
-#if 0
-        while (Exynos_OSAL_GetElemNum(&pExynosOutputPort->codecBufferQ) > 0)
-            Exynos_CodecBufferDeQueue(pExynosComponent, OUTPUT_PORT_INDEX, &pDeQ);
-#else
-        Exynos_CodecBufferReset(pExynosComponent, OUTPUT_PORT_INDEX);
-#endif
 
+        Exynos_CodecBufferReset(pExynosComponent, OUTPUT_PORT_INDEX);
 
         for (i = 0; i < MFC_OUTPUT_BUFFER_NUM_MAX; i++) {
             pOutbufOps->Get_Buffer(hMFCHandle, i, &pBuffer);
@@ -1057,7 +1048,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_GetParameter(
     {
         OMX_S32 codecType;
         OMX_PARAM_COMPONENTROLETYPE *pComponentRole = (OMX_PARAM_COMPONENTROLETYPE *)pComponentParameterStructure;
-
         ret = Exynos_OMX_Check_SizeVersion(pComponentRole, sizeof(OMX_PARAM_COMPONENTROLETYPE));
         if (ret != OMX_ErrorNone) {
             goto EXIT;
@@ -1284,9 +1274,8 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_SetParameter(
         OMX_S32                           codecType;
 
         ret = Exynos_OMX_Check_SizeVersion(pSrcProfileLevel, sizeof(OMX_VIDEO_PARAM_PROFILELEVELTYPE));
-        if (ret != OMX_ErrorNone) {
+        if (ret != OMX_ErrorNone)
             goto EXIT;
-        }
 
         if (pSrcProfileLevel->nPortIndex >= ALL_PORT_NUM) {
             ret = OMX_ErrorBadPortIndex;
@@ -1374,7 +1363,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_GetConfig(
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     if (pExynosComponent->currentState == OMX_StateInvalid) {
         ret = OMX_ErrorInvalidState;
@@ -1419,7 +1407,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_SetConfig(
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
-
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
     if (pExynosComponent->currentState == OMX_StateInvalid) {
         ret = OMX_ErrorInvalidState;
@@ -1476,13 +1463,11 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_GetExtensionIndex(
     if (ret != OMX_ErrorNone) {
         goto EXIT;
     }
-
     if (pOMXComponent->pComponentPrivate == NULL) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
     }
     pExynosComponent = (EXYNOS_OMX_BASECOMPONENT *)pOMXComponent->pComponentPrivate;
-
     if ((cParameterName == NULL) || (pIndexType == NULL)) {
         ret = OMX_ErrorBadParameter;
         goto EXIT;
@@ -1587,11 +1572,10 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_Init(OMX_COMPONENTTYPE *pOMXComponent)
     pExynosComponent->bSaveFlagEOS = OMX_FALSE;
 
     eColorFormat = pExynosInputPort->portDefinition.format.video.eColorFormat;
-
     if (pExynosInputPort->bStoreMetaData == OMX_TRUE) {
         if (eColorFormat == OMX_COLOR_FormatAndroidOpaque) {
             pExynosInputPort->bufferProcessType = BUFFER_COPY;
-    } else {
+        } else {
             pExynosInputPort->bufferProcessType = BUFFER_SHARE;
         }
     }
@@ -1833,7 +1817,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_Init(OMX_COMPONENTTYPE *pOMXComponent)
 
     pMpeg4Enc->bSourceStart = OMX_FALSE;
     Exynos_OSAL_SignalCreate(&pMpeg4Enc->hSourceStartEvent);
-
     pMpeg4Enc->bDestinationStart = OMX_FALSE;
     Exynos_OSAL_SignalCreate(&pMpeg4Enc->hDestinationStartEvent);
 
@@ -1845,7 +1828,7 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_Init(OMX_COMPONENTTYPE *pOMXComponent)
     pExynosComponent->getAllDelayBuffer = OMX_FALSE;
 
 #if 0//defined(USE_CSC_GSCALER)
-    csc_method = CSC_METHOD_TRY_HW; //in case of Use ION buffer.
+    csc_method = CSC_METHOD_HW; //in case of Use ION buffer.
 #endif
     pVideoEnc->csc_handle = csc_init(csc_method);
     if (pVideoEnc->csc_handle == NULL) {
@@ -1884,9 +1867,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
         pVideoEnc->csc_handle = NULL;
     }
 
-//    pInbufOps->Stop(hMFCHandle);
-//    pOutbufOps->Stop(hMFCHandle);
-
     Exynos_OSAL_SignalTerminate(pMpeg4Enc->hDestinationStartEvent);
     pMpeg4Enc->hDestinationStartEvent = NULL;
     pMpeg4Enc->bDestinationStart = OMX_FALSE;
@@ -1914,12 +1894,10 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
         /*    TBD    */
         /*************/
         /* Does not require any actions. */
-
     }
 
     if ((pExynosOutputPort->bufferProcessType & BUFFER_COPY) == BUFFER_COPY) {
-        for (i = 0; i < MFC_OUTPUT_BUFFER_NUM_MAX; i++)
-        {
+        for (i = 0; i < MFC_OUTPUT_BUFFER_NUM_MAX; i++) {
             if (pVideoEnc->pMFCEncOutputBuffer[i] != NULL) {
                 if (pVideoEnc->pMFCEncOutputBuffer[i]->VirAddr != NULL)
                     Exynos_OSAL_SharedMemory_Free(pVideoEnc->hSharedMemory, pVideoEnc->pMFCEncOutputBuffer[i]->VirAddr);
@@ -1936,7 +1914,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_Terminate(OMX_COMPONENTTYPE *pOMXComponent)
         /*    TBD    */
         /*************/
         /* Does not require any actions. */
-
     }
     Mpeg4CodecClose(pMpeg4Enc);
 
@@ -2240,7 +2217,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_srcOutputBufferProcess(OMX_COMPONENTTYPE *pOMXComp
             goto EXIT;
         }
     }
-
     if ((pMpeg4Enc->bSourceStart == OMX_FALSE) &&
        (!CHECK_PORT_BEING_FLUSHED(pExynosInputPort))) {
         Exynos_OSAL_SignalWait(pMpeg4Enc->hSourceStartEvent, DEF_MAX_WAIT_TIME);
@@ -2278,7 +2254,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_dstInputBufferProcess(OMX_COMPONENTTYPE *pOMXCompo
         ret = OMX_ErrorNone;
         goto EXIT;
     }
-
     if (pExynosOutputPort->bufferProcessType == BUFFER_SHARE) {
         if ((pMpeg4Enc->bDestinationStart == OMX_FALSE) &&
            (!CHECK_PORT_BEING_FLUSHED(pExynosOutputPort))) {
@@ -2286,7 +2261,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_dstInputBufferProcess(OMX_COMPONENTTYPE *pOMXCompo
             Exynos_OSAL_SignalReset(pMpeg4Enc->hDestinationStartEvent);
         }
     }
-
     if (pMpeg4Enc->hMFCMpeg4Handle.bConfiguredMFCDst == OMX_TRUE) {
         ret = Exynos_Mpeg4Enc_DstIn(pOMXComponent, pDstInputData);
         if (ret != OMX_ErrorNone) {
@@ -2328,7 +2302,6 @@ OMX_ERRORTYPE Exynos_Mpeg4Enc_dstOutputBufferProcess(OMX_COMPONENTTYPE *pOMXComp
             Exynos_OSAL_SignalReset(pMpeg4Enc->hDestinationStartEvent);
         }
     }
-
     ret = Exynos_Mpeg4Enc_DstOut(pOMXComponent, pDstOutputData);
     if ((ret != OMX_ErrorNone) && (pExynosComponent->currentState == OMX_StateExecuting)) {
         Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "%s: %d: Failed - DstOut -> event is thrown to client", __FUNCTION__, __LINE__);
@@ -2343,13 +2316,7 @@ EXIT:
     return ret;
 }
 
-
-
-
-
-OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(
-    OMX_HANDLETYPE hComponent,
-    OMX_STRING     componentName)
+OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(OMX_HANDLETYPE hComponent, OMX_STRING componentName)
 {
     OMX_ERRORTYPE                  ret              = OMX_ErrorNone;
     OMX_COMPONENTTYPE             *pOMXComponent    = NULL;
@@ -2367,7 +2334,6 @@ OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(
         Exynos_OSAL_Log(EXYNOS_LOG_ERROR, "%s: parameters are null, ret: %X", __FUNCTION__, ret);
         goto EXIT;
     }
-
     if (Exynos_OSAL_Strcmp(EXYNOS_OMX_COMPONENT_MPEG4_ENC, componentName) == 0) {
         codecType = CODEC_TYPE_MPEG4;
     } else if (Exynos_OSAL_Strcmp(EXYNOS_OMX_COMPONENT_H263_ENC, componentName) == 0) {
@@ -2519,7 +2485,6 @@ OSCL_EXPORT_REF OMX_ERRORTYPE Exynos_OMX_ComponentInit(
     pVideoEnc->exynos_checkInputFrame        = NULL;
     pVideoEnc->exynos_codec_getCodecInputPrivateData  = &GetCodecInputPrivateData;
     pVideoEnc->exynos_codec_getCodecOutputPrivateData = &GetCodecOutputPrivateData;
-//    pVideoEnc->exynos_codec_getCodecBufferNumber      = &GetCodecBufferNumber;
 
     pVideoEnc->hSharedMemory = Exynos_OSAL_SharedMemory_Open();
     if (pVideoEnc->hSharedMemory == NULL) {