omxvideodec: Check return values of buffer pool config parsing functions

CID 1214588

omx: Remove dead code, buf can never be NULL here as we just check for that the line above

CID 1214596

omx: Fix comparisons in gst_omx_state_to_string() case to actually make sense

CID 1214593

omx: Make sure to compare the error codes as unsigned integers so that comparisons >2**31 actually work

CID 1214592

omx: Fix comparisons in gst_omx_command_to_string() default cause to actually work

CID 1214591

omxaudioenc: 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.

omxvideoenc: 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 a 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.

omxaudiosink: implement _delay only in the RaspberryPI

Make code implementation conditionally built for RaspberryPI because
OMX_IndexConfigAudioRenderingLatency seems to be a Broadcom extension.

On other targets the query position might not be accurate without
implementing _delay appropriatelly.

omxaudioenc: Correctly scale nTickCount by OMX_TICKS_PER_SECOND

omxaudioenc: Drain encoder on NULL buffer and don't drain on flushing

omxvideodec: Set nTickCount based on the buffer's duration instead of something wrong

omxvideoenc: Set nTickCount to the whole duration of the buffer instead of a wrong calculation

omxaudiosink: Fix format string compiler warnings

omxaudiosink: Implements OpenMAX based audio sinks

Provides omxanalogaudiosink and omxhdmiaudiosink elements on
the Raspberry PI.

- omxanalogaudiosink is capable to render raw mono or stereo audio
through the jack output.
- omxhdmiaudiosink is capable to render raw audio up to 8 channels
and transmit ac3/dts(IEC 61937) through the HDMI output.
- sinks provide a clock derived from rendered samples
- sinks support the GstStreamVolume interface by implementing
the volume and mute properties.

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

Automatic update of common submodule

From bcb1518 to 211fa5f

omxvideodec: can negotiate caps with memory:EGLImage feature when using EGLImage allocator

Previously when using gst EGLImage allocator the caps was
video/x-raw, format=RGBA instead of
video/x-raw(memory:EGLImage), format=RGBA

Kepp previous behavior in case negotiation fails with caps feature.
It means it will still have a chance to use EGLImage even if the
feature is not in the caps.

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

omxvp8dec: fix typo in GST_TYPE_OMX_VP8_DEC define

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

omxvideodec: don't unref caps before logging field from it

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

example: disable testegl since libgstegl has been removed

As decided in bug #703343

Not compatible with the new libgstgl API.
A portage has been started, attachment 272800.

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

omxvideodec: use new libgstgl API since libgstegl has been removed

There is no point to retrieve a ref/unref type
instead of an EGLDisplay directly. It's like for EGLImage.

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

configure.ac: check for libgstgl since libgstegl has been removed

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

omxvideodec: add support of more color format

Add support for ABGR, ARGB, RGB16, BGR16, YUY2, UYVY, YVYU, GRAY8 and
NV16 color format.

omxvideodec: simplify color format conversion in fill_buffer function

omxbufferpool: make video stride and offset calculation easier

It will be easier to support more color format.

omx: add an helper to convert OMX color format to GStreamer color format

omxvideodec: Implement pipeline draining to support adaptive scenarios

When draining due a format change also drain
the pipeline to reclaim back all buffers.

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

examples: fix several memory leaks in the testegl example

Ensure to call to image_data_free in order to release GPU resources.
Also ensure to destroy EGLImage and GLTexture from proper
thread/context.

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

examples: keep a ref on the buffer instead of the memory

Like in eglglessink

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

omxvideodec: fixes race condition during seeks

Acording 6.1.3 Seek Event Sequence in the OpenMAX IL 1.1.2 spec
document in order to flush the component it needs to be in
paused state.

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

omxvideodec: use flush because reset is deprecated

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

omxvideodec: populate the most downstream output port on reset

Make seeking work when using egl_render component

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

omxbufferpool: return buffers to the pool instead of freeing them

We have to return the buffers back to the pool in when stopping to
not mess with the GstBufferPool accounting.
The OMX buffers will be freed when those won't be in charge of the
pool in the chained up call to 'stop'.
Fixes segfaults on finalize and pool not being properly deactivated.

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

omxvideodec: add missing unlock in the error path

Signed-off-by: Christian König <christian.koenig@amd.com>
https://bugzilla.gnome.org/show_bug.cgi?id=726958

fix filemode

omxbufferpool: fix memory leak if used on output port

When using GstOMXBufferPool on an output port, it internally uses
a GPtrArray to manage the GstBuffers instead of the default queue
from the GstBufferPool base class.

In this case GstBufferPool::default_free_buffer is not called when
the pool is stopped. Because the queue is empty. So explicitely
call gst_omx_buffer_pool_free_buffer on each buffer contained in
the GPtrArray.

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

omxh264enc: Fix compiler warnings

omxh264enc: IDR interval, SPS and PPS headers for rpi

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

omxvideo: fix debug category initialisation

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

omxbufferpool: add proper type definitions

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

omx: consolidate message waiting code

Add a wait_message helper function and remove all those duplicated code.

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

omx: Copy old copyright notice into the new file

omx: Rename function from _4_ to _for_ for clarity

omxvideo: start sharing more code between video decoder and encoder

Identical functionality spread of two different components.
We can't use a common base class because of different inheritance,
but let's try to share the code anyway.

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

examples: Only build RPi EGL example if RPi was chosen as target

omxbufferpool: Fix header include guard

omxvideodec: separate the buffer pool from the decoder

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

omx: simplify tunnel functions

Specifying the component is error prone and unnecessary.

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

omxvideodec: fix memory leak in gst_omx_video_dec_allocate_output_buffers

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

omxvideodec: fix memory leak gst_omx_video_dec_negotiate

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

omxvideoenc: fix a memory leak in gst_omx_video_enc_getcaps

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

Makefile.am: build examples

There's no reason to not build the examples now that are ported to 1.x.

examples: avoid a clashing name with the recently added GstEGLImagePool

Fixes build with current master.

omxvideoenc: simplify _find_nearest_frame

Just the same as we did with the decoder. Also give the
function a gst_omx_video_enc prefix to distinct it from
the decoder function.

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

omxvideodec: simplify _find_nearest_frame

No need to make it more complicated and error prone than
necessary. Also give the function a gst_omx_video_dec prefix
to distinct it from the encoder function.

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

omxvideodec: remove dead code

This code doesn't seems to be used for quite a while,
remove it before it starts to rot.

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

omx: fix two serious message handling bugs

Waiting for the next message if we already got one
is nonsense and can lead to lockups.

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

omxvideoenc: fix startup race condition

The reset function shouldn't start the src pad
loop if it wasn't started before.

Signed-off-by: Christian König <christian.koenig@amd.com>

omxvideodec: fix startup race condition

The reset function shouldn't start the src pad
loop if it wasn't started before.

Signed-off-by: Christian König <christian.koenig@amd.com>

Automatic update of common submodule

From fe1672e to bcb1518

Automatic update of common submodule

From 1a07da9 to fe1672e

Automatic update of common submodule

From d48bed3 to 1a07da9

omx: Don't handle FLUSHING and NOT_LINKED as errors

Also don't stop the task on NOT_LINKED. We're not a demuxer.

Automatic update of common submodule

From dbedaa0 to d48bed3

Automatic update of common submodule

From 865aa20 to dbedaa0

examples: simplify the thread synchronization code

Make everithing more simple and fix the races conditions remaining in
the previous approaches.

rpi: The WMV/VC1 decoder can only do WMV9 and VC1, no older versions

Automatic update of common submodule

From 6b03ba7 to 865aa20

examples: fix another race condition

Fix a race condition that caused randome deadlocks on EOS.

rpi: fix a copy paste error in the config file

examples: fix a race condition when seeking

Fixes a race condition that caused pipeline deadlock during seeks.

examples: display QoS statistics

examples: use dedicated thread for rendering the scene

Produces smother animation and prevents dropping frames due busy
mainloop.

examples: don't force an specific audio sink

Let playbin2 choose the audiosink available in the system.

examples: drop remnants of initial appsink attempt

Automatic update of common submodule

From b613661 to 6b03ba7

Automatic update of common submodule

From 74a6857 to b613661

Automatic update of common submodule

From 01a7a46 to 74a6857

omx: don't use the 'z' modifier to print size_t

gcc will warn in some cases even if the size of the type
is exactly that of size_t on the platform.

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

gstomxvideoenc: Set bitrate in setcaps

Otherwise it gets lost whenever we configure new caps

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

examples: enable audio in testegl example

configure: Allow build without gstreamer-egl

examples: testegl example port to 1.x

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

Automatic update of common submodule

From 098c0d7 to 01a7a46

omx: Take lock on EOS to update the flow return value

Fixes "GThread-ERROR **: file gthread-posix.c: line 171
(g_mutex_free_posix_impl): error 'Device or resource busy' during
'pthread_mutex_destroy ((pthread_mutex_t *) mutex)'" in _finalize.

Automatic update of common submodule

From 5edcd85 to 098c0d7

examples: add an example aplication based OpenGL ES + EGL

Application that shows how to integrate playbin with an OpenGL ES
scene through EGL. Renders a video on the surfaces of an animated cube.

The code is not ported to 1.x so it's not built by default.

omxvideodec: Redesign video size change reconfiguration code

Ensure stop the decoder before clossing the tunnel.

Makefile.am: Add -I common/m4

This allows autoreconf to work correctly and automatic regeneration
of autotools files if something changed.

omx: Add pads based on element type

This allows to later add sources and sink that only have a srcpad
or sinkpad.

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

omx: fixed type error in printf call

%zu expects size_t

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

Check for gstreamer-egl

And don't use if not available.

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

configure: error out if no OMX target has been set explicitly with --with-omx-target=x

Avoids people building for e.g. the Raspberry Pi and then wondering
why things don't work as expected (since structs are packed differently
there).

Automatic update of common submodule

From 3cb3d3c to 5edcd85

omx: more printf format fixes

Fix printf formats again, so that gst-omx compiles warning-
free on the Raspberry Pi as well. Unfortunately OMX_UINT32
maybe be typedefed to uint32_t or unsigned long, which
doesn't work well with our debugging printf format strings,
so just use %u for those and cast to guint.

omx: fixes unused variable 'comp' when GStreamer is built without debug

omxvideodec: don't use 'self->dec_out_port' anymore and use just 'port'

Fixes some criticals.

omxvideodec: fixes 'port' may be used uninitialized in this function

omxvideodec: silence warnings building for RPI related to 'vcos_*'

omxvideodec: Use new type from libgstvideo