[Bug 707361] video: Add support for 64x32 tiled NV12 color format

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Fri Jan 3 11:28:47 PST 2014


https://bugzilla.gnome.org/show_bug.cgi?id=707361
  GStreamer | gst-plugins-base | git

--- Comment #49 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2014-01-03 19:28:43 UTC ---
Review of attachment 264876:
 --> (https://bugzilla.gnome.org/review?bug=707361&attachment=264876)

After fixing these two, it seems to work correctly for me.

::: gst-libs/gst/video/video-frame.c
@@ +275,3 @@
+
+    sx_tiles = sinfo->stride[plane] >> ws;
+    sy_tiles = sinfo->stride[tidx];

This is not right, you need to scale the height.

sy_tiles = GST_VIDEO_FORMAT_INFO_SCALE_HEIGHT (finfo, plane, 
    sinfo->stride[tidx]);

@@ +278,3 @@
+
+    dx_tiles = dinfo->stride[plane] >> ws;
+    dy_tiles = dinfo->stride[tidx];

Same

-- 
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