After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 692446 - Add mpeg2 support
Add mpeg2 support
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-omx
git master
Other Linux
: Normal normal
: 1.0.0
Assigned To: Felipe Contreras (banned)
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-01-24 13:02 UTC by Julian Scheel
Modified: 2013-01-24 22:51 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
mpeg2 decoder for gst-omx (7.95 KB, patch)
2013-01-24 13:02 UTC, Julian Scheel
committed Details | Review

Description Julian Scheel 2013-01-24 13:02:53 UTC
Created attachment 234303 [details] [review]
mpeg2 decoder for gst-omx

Attached is a patch which implements a mpeg2 video decoder in the gst-omx plugin. This has been tested on raspberry pi, which works fine. Obviously one has to set decode_MPG2=<key> in config.txt for this to work.

Comments are welcome.
Comment 1 Nicolas Dufresne (ndufresne) 2013-01-24 22:00:15 UTC
Note, you have set the wrong component, this patch is for gst-omx, not gst-openmax.
Comment 2 Tim-Philipp Müller 2013-01-24 22:36:23 UTC
There was only a "gst-openmax" component, which I've renamed to "gst-omx" shortly after the patch was filed :)
Comment 3 Tim-Philipp Müller 2013-01-24 22:51:19 UTC
Didn't test it, but it looks fine, and compiles, and Sebastian is on holiday for a while, so don't really see any reason not to push this. Thanks for the patch!

 commit 3d7a50944d938c49f2fd95111abfd47051a17818
 Author: Julian Scheel <julian@jusst.de>
 Date:   Thu Jan 24 14:02:36 2013 +0100

    omx: add mpeg2 video decoder
    
    This adds a decoder class for mpeg2, as well as an extended
    configuration for raspberry pi.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=692446
    
    Signed-off-by: Julian Scheel <julian@jusst.de>