[Bug 728045] omxvideodec: support cropping information

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=728045
  GStreamer | gst-omx | git

--- Comment #9 from Aurélien Zanelli <aurelien.zanelli at parrot.com> 2014-06-25 08:18:53 UTC ---
(In reply to comment #8)
> To clarify a bit (I know you know Sebastian, it's for others). Both requires
> VideoMeta to be supported downstream. VideoCrop require downstream to support
> it. VideoAlignement only require downstream support it if downstream pool is
> being used (currently only supported by ximagsink and xvimagsink). In OMX, it
> is likely that downstream pool won't be used, hence VideoAlignment being a good
> choice. Note that using this you can then use gst_video_frame_copy() to copy to
> a buffer with default alignment (if there is not way around).

Ok, it could be easier for others elements to play with VideoAlignment or frame
stride/offset to handle crop rectangle. So in which cases should we use
VideoCrop meta ?
At this moment, in current gst-omx implementation, it always use its internal
bufferpool, but in the future, we could want to use an external pool with the
OMX UseBuffer method.

For information, in my case, I plan to use:
omxvideodec --> v4l2sink io-mode=userptr
to achieve a zero copy rendering path.

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