diff options
| author | Sebastian Dröge <sebastian.droege@collabora.co.uk> | 2013-03-10 11:27:34 +0100 |
|---|---|---|
| committer | Sebastian Dröge <sebastian.droege@collabora.co.uk> | 2013-03-10 11:31:55 +0100 |
| commit | fb3b213b3efe3053903caa7f8c9049ce11aff91c (patch) | |
| tree | e334bd204a9a3b11422187b87965d47984328e8e | |
| parent | b9bf849a390b1041f6561e477b66671ebfd98c34 (diff) | |
| download | gst-omx-fb3b213b3efe3053903caa7f8c9049ce11aff91c.tar.gz | |
omxvideoenc: Allocate output buffers as early as possible
| -rw-r--r-- | omx/gstomxvideoenc.c | 28 |
1 files changed, 18 insertions, 10 deletions
diff --git a/omx/gstomxvideoenc.c b/omx/gstomxvideoenc.c index 0e81b3b..d3e7676 100644 --- a/omx/gstomxvideoenc.c +++ b/omx/gstomxvideoenc.c @@ -1229,13 +1229,22 @@ gst_omx_video_enc_set_format (GstVideoEncoder * encoder, if (gst_omx_port_allocate_buffers (self->enc_in_port) != OMX_ErrorNone) return FALSE; - /* And disable output port */ - if (gst_omx_port_set_enabled (self->enc_out_port, FALSE) != OMX_ErrorNone) - return FALSE; + /* If the output port is already configured with the new format we can + * allocate buffers here already */ + if (self->enc_out_port->port_def.format.video.nFrameHeight == info->height + && self->enc_out_port->port_def.format.video.nFrameWidth == + info->width) { + if (gst_omx_port_allocate_buffers (self->enc_out_port) != OMX_ErrorNone) + return FALSE; + } else { + /* And disable output port */ + if (gst_omx_port_set_enabled (self->enc_out_port, FALSE) != OMX_ErrorNone) + return FALSE; - if (gst_omx_port_wait_enabled (self->enc_out_port, - 1 * GST_SECOND) != OMX_ErrorNone) - return FALSE; + if (gst_omx_port_wait_enabled (self->enc_out_port, + 1 * GST_SECOND) != OMX_ErrorNone) + return FALSE; + } if (gst_omx_component_get_state (self->enc, GST_CLOCK_TIME_NONE) != OMX_StateIdle) @@ -1245,6 +1254,9 @@ gst_omx_video_enc_set_format (GstVideoEncoder * encoder, OMX_StateExecuting) != OMX_ErrorNone) return FALSE; + if (gst_omx_port_populate (self->enc_out_port) != OMX_ErrorNone) + return FALSE; + if (gst_omx_component_get_state (self->enc, GST_CLOCK_TIME_NONE) != OMX_StateExecuting) return FALSE; @@ -1254,10 +1266,6 @@ gst_omx_video_enc_set_format (GstVideoEncoder * encoder, gst_omx_port_set_flushing (self->enc_in_port, 5 * GST_SECOND, FALSE); gst_omx_port_set_flushing (self->enc_out_port, 5 * GST_SECOND, FALSE); - if (!needs_disable) - if (gst_omx_port_populate (self->enc_out_port) != OMX_ErrorNone) - return FALSE; - if (gst_omx_component_get_last_error (self->enc) != OMX_ErrorNone) { GST_ERROR_OBJECT (self, "Component in error state: %s (0x%08x)", gst_omx_component_get_last_error_string (self->enc), |