History log of /hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
eb6da54ec5a57e2f7219330d36a37213a850db17 08-Aug-2010 James Dong <jdong@google.com> TI's video encoder labels codec config data with OMX_BUFFERFLAG_CODECCONFIG

- Also, added a workaround to send SPS and PPS in a single buffer.
This has better be fixed so that the first output buffer contains the
codec config data (rather than a 0-length buffer)

Change-Id: Id9ae42debfc676c8dff3095304217911021d1c66
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
a7b22907b3fe1a2d644e8ef023bf8d8fde16f99e 30-Mar-2010 James Dong <jdong@google.com> Fix a bug where the conditional wait is not correct for state transition from OMX_StateIdle to OMX_StateLoaded
In addition, since we have already returned all buffers to the client after completing the state transition
from OMX_StateExecuting to OMX_StateIdle, we don't need to process OMX_FillThisBuffer() in the state of
OMX_StateLoaded.

bug - 2501987

Change-Id: I7506092b97b8fb93bcfb8cefaceaf6ca98e86e06
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
24ef98f0d3b5d00b1d30c8ebf0791ef708170a32 19-Mar-2010 James Dong <jdong@google.com> The input and output buffers may not be returned after the state
transition from executing to idle is completed from TI's omx
video encoder. This is to enforce the return of all the buffers; and provide
explict check on it.

bug - 2484098

Change-Id: I2ff6279957ff643e292710dec3d4547cc66bfce1
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
43a7253b0439ca1a6f69496e23343164735a6b12 17-Mar-2010 James Dong <jdong@google.com> Fix comparisons: unsigned values < 0

This patch was originally from MOT/TI
Modified to fix a small bug in the original patch.

bug - 2152611

Change-Id: Iee30350a6772389561826e63ad209e8143f71a13
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
1535a78759d5f314a441cbc86a90f3769bc7a921 17-Feb-2010 James Dong <jdong@google.com> Return OMX_ErrorNone from OMX_GetState when OMX_GetState timed out
Plus, tell the caller that the omx component is in a invalid state.

The timeout can happen when LCML initialization fails, for instance,
when a 720p video is passed to the TI's video decoder.

bug - 2414276
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
6bf5fadc2cc19bc298c4e998657e35cf03644d3d 17-Feb-2010 James Dong <jdong@google.com> Fix the problem where the first h264 frame in the recorded video is not marked as sync frame
The root cause was that the socket node was not sending the info for SLICE bitstream mode
Changes include:
1. Change I-Sync frames to be IDR frames for h264
2. Inform encoder of framerate changes to ensure correct bitrate output

Depends on:
1. the new socket node binary from TI

This patch was originally from MOT/TI.

bug - 2161894
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
b65f253fdc60f6208a54911bee793d225f23cdf8 04-Feb-2010 James Dong <jdong@google.com> Revert "Revert "Omx component change due to TI's socket node update from 25.9 to 25.14""

This reverts commit eaaadaf5ebb5e352e2ed4a12714f5d2363da0dcd.
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
eaaadaf5ebb5e352e2ed4a12714f5d2363da0dcd 02-Feb-2010 James Dong <jdong@google.com> Revert "Omx component change due to TI's socket node update from 25.9 to 25.14"

This reverts commit 36733d62c0d4c565da10b34978e587d0fda0f1ad.
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
36733d62c0d4c565da10b34978e587d0fda0f1ad 28-Jan-2010 James Dong <jdong@google.com> Omx component change due to TI's socket node update from 25.9 to 25.14
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
bcf063040ece6f99de7b16cdf9ad45c92cc42552 10-Dec-2009 James Dong <jdong@google.com> Fix a small bug ComponentRoleEnum() in OMX video encoder where a "goto" statement was prematurely called.
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
0f4780739a4c3baf4831235038314314c9c8ab09 05-Dec-2009 James Dong <jdong@google.com> Fix a small mistake where the initializer for an array actually contains more elements than it claims.
Remove the fixed value MACRO
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
c854c1215804f5940ea3d7ae3ba18d8fb6fab317 30-Nov-2009 James Dong <jdong@google.com> Address assignment was incorrect for finding the supported profile and level.
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
28bb1bbc618ee23e505399ec6f18549b1d67f9a3 29-Oct-2009 Nikhil Mande <nikhil@ti.com> Fix in h264 output buffer size calculation

Previous buffer calculation was incorrect, for D1 it was 2.1 MB now
it is 600 KB. The calculations are now based on H.264 annex A spec.
The smaller size results in reducing cache flush latencies in the bridge improving
performance. With this change D1 can now get up to ~29 FPS.

Credit to Jeff Vanhoof for noticing the large buffer size.

Originally from: https://partner.source.android.com/g/#change,1400
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
0c1fc978146aeb80d968d0b27abe70bf84c62ed4 30-Sep-2009 Benny Wong <Benny.Wong@motorola.com> To get GetState to block for pending state transitions to complete for video encode

Originally from: https://partner.source.android.com/g/#change,1200
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
b3c895c1b4ef6c6986c7c5bebfadd43de3671995 29-Sep-2009 Benny Wong <Benny.Wong@motorola.com> Fixed issues in querying & setting video encoder profiles & levels

Originally from: https://partner.source.android.com/g/#change,1150
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
687658b40ff09d22b78c5701904224e9568a6e8a 20-Aug-2009 snigdha <snigdha.sinha@motorola.com> Fix Iframe in H264 and change IframeInterval to 30
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
b44cdb1b44b144d09e8ec950d1e8e4c280064056 30-Jul-2009 Rebecca Schultz Zavin <rebecca@android.com> New version of ti OMX code to support ti mm release 25.9

Signed-off-by: Rebecca Schultz Zavin <rebecca@android.com>
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c
fb3766f18a2c18b6f4798a6a631fdb88fcacd1dc 17-Jul-2009 Rebecca Schultz Zavin <rebecca@android.com> Adding omap3 omx code

Signed-off-by: Rebecca Schultz Zavin <rebecca@android.com>
/hardware/ti/omap3/omx/video/src/openmax_il/video_encode/src/OMX_VideoEncoder.c