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 745639 - video: support YUYV fourcc as alias for YUY2
video: support YUYV fourcc as alias for YUY2
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
unspecified
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-03-04 20:17 UTC by Aleix Conchillo Flaqué
Modified: 2018-11-03 11:35 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
YUYV is YUY2 (982 bytes, patch)
2015-03-04 20:19 UTC, Aleix Conchillo Flaqué 		none Details | Review
YUYV is YUY2 fixup (943 bytes, patch)
2015-03-04 20:21 UTC, Aleix Conchillo Flaqué 		none Details | Review

Description Aleix Conchillo Flaqué 2015-03-04 20:17:38 UTC 
Calling gst_video_format_from_fourcc with YUYV returns GST_VIDEO_FORMAT_UNKNOWN,

YUYV is an alias for YUY2 so it should return YUY2 safely.

I've found this by reading the pixelformat from v4l2.
Comment 1 Aleix Conchillo Flaqué 2015-03-04 20:19:51 UTC 
Created attachment 298577 [details] [review]
YUYV is YUY2
Comment 2 Aleix Conchillo Flaqué 2015-03-04 20:21:37 UTC 
Created attachment 298579 [details] [review]
YUYV is YUY2 fixup

One less line.
Comment 3 Nicolas Dufresne (ndufresne) 2015-03-04 21:11:13 UTC 
Current list is based on X11/XV fourcc. Would be nice to go over V4L2, DRM and XV fourccs. If there is clash, I would rather suggest to deprecate and eventually remove this function.
Comment 4 Aleix Conchillo Flaqué 2015-03-04 21:52:38 UTC 
(In reply to Nicolas Dufresne (stormer) from comment #3)
> Current list is based on X11/XV fourcc. Would be nice to go over V4L2, DRM
> and XV fourccs. If there is clash, I would rather suggest to deprecate and
> eventually remove this function.

I see. Should we mark this as invalid, then?
Comment 5 Nicolas Dufresne (ndufresne) 2015-03-04 22:02:34 UTC 
Would be nice to make this exercise first. If there is no clash, I'd say why not. If there is clash, it means we'll have a hard time supporting everyone.
Comment 6 Aleix Conchillo Flaqué 2015-03-04 22:10:01 UTC 
Got it, I misunderstood.

Isn't fourcc just a four characters code? I don't see how there could be a clash. 

May be you meant something else?
Comment 7 Nicolas Dufresne (ndufresne) 2015-03-04 22:22:34 UTC 
the same fourcc might not mean the same thing for V4L2, DRM, X11, Microsoft, ISOMP4, etc.
Comment 8 Tim-Philipp Müller 2015-03-08 19:26:59 UTC 
I think it's a reasonable patch. Only problem is what to do for _to_fourcc() then? :)
Comment 9 GStreamer system administrator 2018-11-03 11:35:27 UTC 
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/issues/168.