[Bug 707361] video: Add support for 64x32 tiled NV12 color format
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Wed Nov 13 13:34:32 PST 2013
https://bugzilla.gnome.org/show_bug.cgi?id=707361
GStreamer | gst-plugins-base | git
Nicolas Dufresne <nicolas.dufresne> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |nicolas.dufresne at collabora.
| |co.uk
--- Comment #3 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2013-11-13 21:34:25 UTC ---
Ok, adding this to libgstvideo was not that easy. The problem was that we need
the video height to be able to figure-out where are the pixels in the buffer.
As I thought breaking this API again was a bad idea, I managed to implement it
in a backward compatible way by introducing pack2_func/unpack2_func. Not the
cutest thing ever, but backward compatible. The only known "external" user of
this is videmixer, since it's copying videoconvert.c.
Note that pack/unpack are not very optimal. It would be made better by playing
with the pack_lines parameters and do pack/unpack 32 lines per call. Though no
one ever tried using that. In genenral this should not be a problem as we
expect the format to be passed as-is to the display.
For the conversion code, I have found the source of what the videolan guy has
done, that's what I'm based on and it came with very nice explanation. Note
that I have kept his reduction, as it made sense.
I'm very open to comment, suggestion, better ideas. Patches will follow...
--
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.
More information about the gstreamer-bugs
mailing list