Merge branch 'upstream/master' into tizen

Change-Id: I6eb03c12b9672f6ad09ad1535351dbc6c0380dc3

Release 1.2.0

Change-Id: I65b90a7d4caa26f56fe65a95fb086f8ac5d9e016

Remember the last_error after a failed set state call to avoid blocking the next get state call

gst_omx_video_dec_flush() blocks forever in
http://cgit.freedesktop.org/gstreamer/gst-omx/tree/omx/gstomxvideodec.c?id=9adf0ff82903cad5331e40975ae91ed5d11bc102#n2110
when the previous call to gst_omx_component_set_state() fails in
http://cgit.freedesktop.org/gstreamer/gst-omx/tree/omx/gstomx.c?id=9adf0ff82903cad5331e40975ae91ed5d11bc102#n827.
To mitigate that, I set "last_error" to true, so the code in
http://cgit.freedesktop.org/gstreamer/gst-omx/tree/omx/gstomx.c?id=9adf0ff82903cad5331e40975ae91ed5d11bc102#n862
exits early and doesn't block.

https://bugzilla.gnome.org/show_bug.cgi?id=758274

Properly handle drain requests while flushing

Without this commit the decoder streaming thread stops without ever attending
the drain request, leaving the decoder input thread waiting forever.

https://bugzilla.gnome.org/show_bug.cgi?id=758274

omx: fix hacks leak on class init

Automatic update of common submodule

From b99800a to b319909

Automatic update of common submodule

From 9aed1d7 to b99800a

Added gst-omx.conf for building 64-bit

Change-Id: I3b1d389f121405129f78a5fe3808885f155f679a
Signed-off-by: Sejun Park <sejun79.park@samsung.com>

omxaacdec: Do not accept unknown layouts

It was defaulting to RAW when an unknown layout was received but
the caps template would actually forbid that on the caps query
or accept-caps anyway.

omxaudiodec: use default pad accept-caps handling

Instead of the audiodecoder one. The OMX audioo decoders have their
valid input in the template pad, so just check against that to
avoid doing a query downstream.

omxvideodec: use default pad accept-caps handling

Instead of the videodecoder one. The OMX video decoders have their
valid input in the template pad, so just check against that to
avoid doing a query downstream.

omxvideoenc: Add keyframe support for the Rpi, using OMX_IndexConfigBrcmVideoRequestIFrame

https://bugzilla.gnome.org/show_bug.cgi?id=753085

Changed plugin-name of omx

Change-Id: I6a426f66b6e7902112d04bdf3dfa1b57d37a0243

gst-omx: Fix memory leaks when context parse fails

When g_option_context_parse fails, context and error variables are not getting free'd
which results in memory leaks. Free'ing the same.

And replacing g_error_free with g_clear_error, which checks if the error being passed
is not NULL and sets the variable to NULL on free'ing.

https://bugzilla.gnome.org/show_bug.cgi?id=753865

Sending mpeg4 codec data to MFC. Now MFC is able to parse mpeg4 header data.
Signed-off-by: Barun Kumar Singh <barun.singh@samsung.com>
Change-Id: I5a37415505619627cb1f231516cc7e44f2686bb3

omxh264dec: implement is_format_change

The omxvideodecoder class only checks some of the caps parameters but if
other fields change such as h264 profile and/or level it wouldn't trigger a
reconfiguration.

https://bugzilla.gnome.org/show_bug.cgi?id=752376

Modify configuration for setting gstomx.conf

Change-Id: Ibc432bedef28da16acf3ddb96eb6258410a3e0bc

Fixed issue when operating with user mode

Change-Id: I7289dfa6ddf5437f3f88fa8278916e26e941c2b6

Add dump func. for debuffing & Fix build error

Change-Id: Ibbb59212d8b73e24f49417b71eb038232b45ac71
Signed-off-by: Sejun Park <sejun79.park@samsung.com>

Fixing "g_ptr_array_set_size" crash.
Increasing omx elements priority.
Signed-off-by: Barun Kumar Singh <barun.singh@samsung.com>
Change-Id: I388fa4d7bd3da9c60ed30fd4659815e5356a011b

Incorporating review comments.
Adding MMVideoBuffer support and fixing plane size calculation for H264
Signed-off-by: Barun Kumar Singh <barun.singh@samsung.com>
Change-Id: I7c7a85386a561956d1ea81119ec9b1ac48fc9b26

omxvideodec: unref allocator after getting it from allocation query

Otherwise a reference will be leaked for each allocator. It only happens
when target platform is Raspberry Pi and when we have GL support.

https://bugzilla.gnome.org/show_bug.cgi?id=751867

Automatic update of common submodule

From f74b2df to 9aed1d7

Automatic update of common submodule

From 6015d26 to f74b2df

Automatic update of common submodule

From d9a3353 to 6015d26

Automatic update of common submodule

From d37af32 to d9a3353

Automatic update of common submodule

From 21ba2e5 to d37af32

Automatic update of common submodule

From c408583 to 21ba2e5

Automatic update of common submodule

From c8fb372 to c408583

Removing dependency of X11, dri2proto,libdri2, xfixes.
gbs build successful for wayland repository
Signed-off-by: Barun Kumar Singh <barun.singh@samsung.com>
Change-Id: I1c52142a6680db4e5649cf17e4a213950eae4db0

Fixing encoder plane size calculating.
Signed-off-by: Barun Kumar Singh <barun.singh@samsung.com>
Change-Id: I7a2eb58809b36ffbd585b8a35e33392acfdc6b9f

omxaudiodec: Add an output adapter for chunking the output into codec frames

Otherwise the base class will be confused.
See https://bugzilla.gnome.org/show_bug.cgi?id=685730

Incorporating MPEG2 review comments.
Enabling h264 encoder.
Removing unnecessary commented code.
Signed-off-by: Barun Kumar Singh <barun.singh@samsung.com>
Change-Id: Ibe9ac74078bcb96d2b0c4b7f857dfe93b7b7dce0

Remove obsolete Android build cruft

This is not needed any longer.

Remove INSTALL file

autotools automatically generate this, and when using different versions
for autogen.sh there will always be changes to a file tracked by git.

Add m4 directory so aclocal doesn't complain in autogen.sh

Might come in handy, and these warnings seem to be
fatal in some environments.

You may need to git clean -x -d -f your tree before
git pulling/merging.

Update autogen.sh to latest version

Automatic update of common submodule

From bc76a8b to c8fb372

 Adding libtbm support.
 Signed-off-by : Barun Kumar Singh <barun.singh@samsung.com>
Change-Id: Idefb94949ee67844cf38524a99d27935c0530973

omxaudiodec: add comment explaining duplicate code path

Revert "omxaudiodec: remove duplicate code path"

This reverts commit a8d26ff27a8b43f589424a59294f9057641b2a46.

omx: handle both errors in the two steps of update_port_definition

Also consider potential errors in the _get_parameter() in the return of the
update_port_definition function.

CID #1287052

omxaudioenc: impossible if statement

ret is set to GST_STATE_CHANGE_SUCCESS and never touched, so it is impossible
for it to be anything else at the if check. Remove the if check.

CID #1287053

omxaudiodec: impossible if statement

ret is set to GST_STATE_CHANGE_SUCCESS and never touched, so it is impossible
for it to be anything else at the if check. Remove it.

CID #1287054

omxaudiodec: remove duplicate code path

omx: call handle_messages() only once in acquire_buffer() to avoid potential deadlock

There is one rare case where calling handle_messages() more than once can cause a deadlock
in the video decoder element:

- sink pad thread starts the src pad task (gst_omx_video_dec_loop())
- _video_dec_loop() calls gst_omx_port_acquire_buffer() on dec_out_port
- blocks in gst_omx_component_wait_message() releasing comp->lock and comp->messages_lock
  (initially, there are no buffers configured on that port, so it waits for OMX_EventPortSettingsChanged)
- the sink pad thread pushes a buffer to the decoder with gst_omx_port_release_buffer()
- _release_buffer() grabs comp->lock and sends the buffer to OMX, which consumes it immediately
- EmptyBufferDone gets called at this point, which signals _wait_message() to unblock
- the message from EmptyBufferDone is processed in gst_omx_component_handle_messages()
  called from gst_omx_port_release_buffer()
- gst_omx_port_release_buffer releases comp->lock
- the src pad thread now gets to run, grabbing comp->lock while it exits from _wait_message()
- _acquire_buffer() calls the _handle_messages() on the next line after _wait_message(),
  which does nothing (no pending messages)
- then it goes to "retry:" and calls _handle_messages() again, which also does nothing
  (still no pending messages)
- scheduler switches to a videocore thread that calls EventHandler, informing us about the
  OMX_EventPortSettingsChanged event that just arrived
- EventHandler graps comp->messages_lock, but not comp->lock, so it can run in parallel at
  this point just fine.
- scheduler switches back to the src pad thread (which is in the middle of _acquire_buffer())
- the next _handle_messages() which is right before if (g_queue_is_empty (&port->pending_buffers))
  processes the OMX_EventPortSettingsChanged
- the buffer queue is still empty, so that thread blocks again in _wait_message()
- the sink pad thread tries to acquire the next input port buffer
- _acquire_buffer() also blocks this thread in:
   if (comp->pending_reconfigure_outports) { ... _wait_message() ... }
- DEADLOCK. gstreamer is waiting for omx to do something, omx waits for gstreamer to do something.

By removing those extra _handle_messages() calls, we can ensure that all the checks of
_acquire_buffer() will re-run. In the above case, after the scheduler switches back to
the middle of _acquire_buffer(), the code will enter _wait_message(), which will see that
there are pending messages and will return immediately, going back to "retry:" and
re-doing all the checks properly.

https://bugzilla.gnome.org/show_bug.cgi?id=741854

omx: cleanup code a bit to remove else statement

https://bugzilla.gnome.org/show_bug.cgi?id=745191

Automatic update of common submodule

From f2c6b95 to bc76a8b

Automatic update of common submodule

From ef1ffdc to f2c6b95

Automatic update of common submodule

From 7bb2bce to ef1ffdc

omx: Add omxamrdec

https://bugzilla.gnome.org/show_bug.cgi?id=739333

omx: Fix a missing g_free() in error path

This fixes a memory leak with g_strdup() when an error occurs.

https://bugzilla.gnome.org/show_bug.cgi?id=731141

Change-Id: I333df2cc1197e2c04ca65f2f82889e9ce7c975d2
Signed-off-by: Kazunori Kobayashi <kkobayas@igel.co.jp>

Automatic update of common submodule

From 84d06cd to 7bb2bce

Automatic update of common submodule

From a8c8939 to 84d06cd

Automatic update of common submodule

From 1f5d3c3 to a8c8939

omxaudiodec: Clean up code a bit to get rid of useless NULL checks

omxaudiodec: Unmap input buffers after usage

https://bugzilla.gnome.org/show_bug.cgi?id=736314

omxvideoenc: Setup aspect ratio on RPi

Needs firmware from yesterday or newer to work with all possible
aspect ratios. Before that it only supported a fixed list.

https://bugzilla.gnome.org/show_bug.cgi?id=732533

omxaudioenc: Implement the hack flag GST_OMX_HACK_NO_COMPONENT_RECONFIGURE

omxaudioenc: Use the base class' open/close vfuncs instead of calling them ourselves

omxvideoenc: Implement the hack flag GST_OMX_HACK_NO_COMPONENT_RECONFIGURE

Fix a video encoder stall problem on RPi when changing the aspect ratio.

https://bugzilla.gnome.org/show_bug.cgi?id=732533

omx: Let base classes handle EOS

https://bugzilla.gnome.org//show_bug.cgi?id=734774

omxaudiodec: Fix flushing logic and make it more similar to the video decoder

Release 1.2.0

config: Update ranks to PRIMARY+1 to have higher preference than avdec_*

See https://bugzilla.gnome.org/show_bug.cgi?id=732161

omxaudiosink: Set port to not flushing in prepare() and keep it at flushing in unprepare()

https://bugzilla.gnome.org/show_bug.cgi?id=733168

omxaudiodec: Implement setting of fallback channel positions

omx: Add AAC audio decoder

Added Samsung specific code.
Change-Id: Ibfd412781bd521ba3e15a5046ce0db660a02ca61

omxaudiodec: Get PCM parameters from the out port, not the in port

omxaudiodec: Implement hack for not disabling the output port after set_format until the output format is known

Needed on some OMX implementations, e.g. the one from Atmel. It does
not send the settings-changed event on the output port if it is
disabled.

omx: Add audio decoder base class and a subclass for MP3

omx: Link to gmodule-2.0-no-export for being able to use the g_module_*() API

https://bugzilla.gnome.org/show_bug.cgi?id=732518

examples: #define GST_USE_UNSTABLE_API for libgstgl

omxh264enc: Properly accumulate headers and push before the next frame

Fixes output of encoding on RPi, where each header buffer (SPS and PPS)
is in a separate OMX buffer.

https://bugzilla.gnome.org/show_bug.cgi?id=726669

omxvideoenc: Implement flush() instead of the deprecated reset()

rpi: It's 44100Hz, not 41400Hz

configure.ac: require gstgl >= 1.3.3

example: enable testegl

See https://bugzilla.gnome.org/show_bug.cgi?id=728940

testegl: do matrix mutlplication in the shader

See https://bugzilla.gnome.org/show_bug.cgi?id=728940

testegl: add a comment for the parse command

See https://bugzilla.gnome.org/show_bug.cgi?id=728940

testegl: convert code from GLESv1 to GLESv2

See https://bugzilla.gnome.org/show_bug.cgi?id=728940

testegl: port to gstgl API

- append a glfilter just before fakesink
  So that we get gltexture or eglimages

- propagate our EGLDisplay to the pipeline
  see GST_QUERY_CONTEXT

- share our EGLContext with the iternal gl context
  of the pipeline, see GST_QUERY_ALLOCATION

- use GstVideoGLTextureUploadMeta to upload
  the incoming gltexture or eglimage to our gl texture

TODO: convert from GLESv1 to GLESv2

See https://bugzilla.gnome.org/show_bug.cgi?id=728940

omxbufferpool: Copy buffers if the stride does not match and we can't use video meta

https://bugzilla.gnome.org/show_bug.cgi?id=731672

omx: Only include OMX_VideoExt.h conditionally

It does not exist on the RPi for example.

configure.ac: Require GStreamer core/base >= 1.2.2

Needed at least for gst_video_decoder_release_frame().

omxbufferpool: Fix format string compiler warning

omxbufferpool: Initialize debug category

omxbufferpool: Properly convert OMX alignment to GStreamer alignment

GStreamer uses a bitmask for the alignment while OMX uses the
alignment itself. Let's convert.

https://bugzilla.gnome.org/show_bug.cgi?id=710564

omxh264enc: Don't let baseclass finish frames for SPS/PPS buffers

Otherwise we a) send them twice, and b) finish a frame for something
that does not even include a frame.

https://bugzilla.gnome.org/show_bug.cgi?id=726669

omxvideo: Include the separate headers too for compatibility with 1.0.x

omxvp8dec: use VP8 definition from OMX_VideoExt.h

https://bugzilla.gnome.org/show_bug.cgi?id=726957

configure: add --with-omx-header-path option for external omx headers

https://bugzilla.gnome.org/show_bug.cgi?id=726957

omxvideodec: fix a query leak

Also add a debug message if query fails.

https://bugzilla.gnome.org/show_bug.cgi?id=731898

omxvideodec: release frames with old PTS to avoid memory issue

Interlaced stream could make the decoder use two input frames to produce
one output frame causing the gstvideodecoder frame list to grow.
Assuming the video decoder output frame in display order rather than in
decoding order, this commit add a way to release frames with PTS less
than current output frame.

https://bugzilla.gnome.org/show_bug.cgi?id=730995

omx: Fix a missing g_free() in error path

This fixes a memory leak with g_strdup() when an error occurs.

https://bugzilla.gnome.org/show_bug.cgi?id=731141

omxvideodec: add missing stream unlock in error path

omx: Don't handle disabling/enabling ports exactly like flushing

Otherwise we might abort a flush operation in another thread when
enabling/disabling ports, leading to deadlocks sometimes.

https://bugzilla.gnome.org/show_bug.cgi?id=730989

omxvideodec: Don't leak buffer pool config in error cases

CID 1216158

Automatic update of common submodule

From 211fa5f to 1f5d3c3