Age | Commit message (Collapse) | Author |
|
Change-Id: Id4515d1e5a70211844efc75f9a3bbf6498c2a9de
Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
|
|
Fix below compiler error generated with latest toolchain
omx/base/omx_base_dio_plugin/src/omx_base_dio_non_tunnel.c:232:29:
error: incompatible integer to pointer conversion assigning to 'OMX_PTR'
(aka 'void *') from 'OMX_U32' (aka 'unsigned int') [-Werror,-Wint-conversion]
pPort->nCachedBufferCnt = pPort->sPortDef.nBufferCountActual;
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Change-Id: I1f029fb9b8d29f906dfe9ce83e503ae66550778b
Signed-off-by: Vishal Mahaveer <vishalm@ti.com>
|
|
The OMX Video Decoder was not checking for whether a buffer was
still locked for reference by the decoder codec, and it was
therefore possible that "locked" output buffers could be passed
back to the decoder to use again while it was still being used for
reference. For certain complex streams this resulted in visible
artifacts in the decoded output.
The codec provides two lists of buffers back after each
process call: a list of "displayable" buffers, and a list of
"freed" buffers. A buffer may be displayable before it is
freed by the codec. Once a buffer is displayable, it can
be sent to for rendering. TI OMX component indicates that the
buffer is displayable by sending it to stagefright. Once
stagefright receives and displays it, it returns it back
to the TI OMX component. At this point, the TI OMX component
should not send the buffer back to the codec until the
codec indicates that it is also "freed."
This patch adds the necessary logic to track the "locked"
state of the output buffers. Before passing an output buffer
to the codec process call, the buffer must no longer be in
the "locked" state. Buffers are marked as locked when they
are passed to the codec process call, and are marked as
un-locked when the codec frees the buffer through the
freeBufId list.
Change-Id: I4608fa5fbf367849f4cacde3b572a47a7249f71c
Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
|
|
The dma_buf_fd for the decoder o/p port buffers are
duped by the binder, and hence omx/dio class should
not close any of those fds.
Change-Id: I2e568fe572dc4b6f3819984b365b5560465d6d9b
Signed-off-by: Sunita Nadampalli <sunitan@ti.com>
|
|
Change-Id: I0252480d66e5d1afcbf1d448674a2095c91fe77f
Signed-off-by: Sunita Nadampalli <sunitan@ti.com>
|