[Bug 651439] Wrong video raw format fourcc assignment for I420

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=651439
  GStreamer | gst-openmax | unspecified

Felipe Contreras <felipe.contreras> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED

--- Comment #6 from Felipe Contreras <felipe.contreras at gmail.com> 2011-05-30 15:28:45 UTC ---
(In reply to comment #4)
> Created an attachment (id=188884)
 View: https://bugzilla.gnome.org/attachment.cgi?id=188884
 Review: https://bugzilla.gnome.org/review?bug=651439&attachment=188884

> proposed patch for fix I420 openmax mapping.

Looks fine, but please use 'git format-patch' to generate your patches from a
commit, with a summary and commit message. This time I wrote one for you:

---
From: Kan Hu <kan.hu at linaro.org>
Subject: [PATCH] base_videodec: workaround for non-packed I420 type

Some people use OMX_COLOR_FormatYUV420Planar, some
OMX_COLOR_FormatYUV420PackedPlanar. Strictly speaking the format needed
in GStreamer is the packed one, but since there's no notion of planes in
GStreamer, considering them the same cannot hurt.

Would be better if the implementations are fixed, specially if we
implement stride/plane support in GStreamer/gst-openmax.
---

Applied and pushed.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.