build: add meson build definition

Currently only been tested on the RPi within gst-build.

examples/testegl: update for libgstgl changes

Back to development

Release 1.11.1

omxaudioenc: set base class format instead of just source pad caps

omx*dec: Flush before we stop the srcpad loop

Flushing could otherwise hang if output port queue of pending buffers was empty

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

Fix broken build due to syntax error

Problem was introduced by 8716c23e2b0553db2982eb84c437f493c974385e

Prevent early EOS by resetting eos flag once GST_OMX_ACQUIRE_BUFFER_EOS is triggered

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

Add clear debug logging on all the paths out of gst_omx_port_acquire_buffer()

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

Add a signals-premature-eos hack for egl_render

egl_render seems to have a bug and signals EOS before it has finished
pushing out all data; this hack simply makes acquire_buffer() wait
a bit more before signalling EOS, in case egl_render decides to spit
out some more data.

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

Add support for OMX_COLOR_FormatYUV420PackedSemiPlanar and OMX_COLOR_Format24bitBGR888

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

Automatic update of common submodule

From ac2f647 to 39ac2f5

common: use https protocol for common submodule

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

omx: Fix compiler warning with latest clang

gstomx.c:1376:42: error: implicit conversion from enumeration type 'GstOMXAcquireBufferReturn' to different enumeration type 'OMX_ERRORTYPE'
      (aka 'enum OMX_ERRORTYPE') [-Werror,-Wenum-conversion]
  g_return_val_if_fail (!port->tunneled, GST_OMX_ACQUIRE_BUFFER_ERROR);
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~

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

videodecoder: remove use of EGLImage orientation

New code should use GstVideoTransformationMeta

(The defaults do the right thing here though).

Back to development

Release 1.10.0

Release 1.9.90

configure: Depend on gstreamer 1.9.2.1

omx: Also search for gstomx.conf in the autoconf --sysconfdir

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

Automatic update of common submodule

From b18d820 to f980fd9

Automatic update of common submodule

From ac2f647 to b18d820

Back to development

Release 1.9.2

Automatic update of common submodule

From ac2f647 to f49c55e

Back to development

Release 1.9.1

omxvideodec: Use GST_VIDEO_FRAME_PLANE_STRIDE() to get output buffer's stride

GST_VIDEO_FRAME_PLANE_STRIDE() should be used to get the actual buffer stride,
as reported in the buffers's GstVideoMeta, when copying data to that buffer.

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

Automatic update of common submodule

From ac2f647 to f363b32

omxaudioenc: implement GstPreset interface

To allow user to use GstPreset to quickly save and load a set of
parameters.

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

omxvideoenc: implement GstPreset interface

To allow user to use GstPreset to quickly save and load a set of
parameters.

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

omx: #define OMX_SKIP64BIT on the RPi as required by their API

Also add generic support for OMX_SKIP64BIT to gst-omx, in case other
implementations also #define that for whatever reason.

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

omxvideodec: Implement ::drain() virtual method

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

configure: remove AG_GST_PARSE_SUBSYSTEM_DISABLES

This would check which subsystems are disabled in core by grepping
gstconfig.h. Only problem is: gstconfig.h has moved into libdir now
so we've been checking a non-existent file for a while now. The
macro would just sets GST_DISABLE_* for use in configure.ac and
Makefile.am, but we don't use that anywhere so just get rid of it
(the one place where we use GST_DISABLE_GST_DEBUG is in a .c file
which gets the define from the gstconfig.h include).

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

omxvideodec : Use gstglmemoryegl for the RPi

Modified to use gstglmemoryegl to avoid texture creation/copy operations
at the glupload.

[Matthew Waters]: gst-indent the sources and port testegl to GstGLMemoryEGL

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

Automatic update of common submodule

From 6f2d209 to ac2f647

Automatic update of common submodule

From b64f03f to 6f2d209

examples: update egl example for gstgl API changes

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

Automatic update of common submodule

From 86e4663 to b64f03f

configure: Make -Bsymbolic check work with clang.

Update the -Bsymbolic check with the version glib has. This version
works with clang.

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

Automatic update of common submodule

From b319909 to 86e4663

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

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

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

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

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

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

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

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

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

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