[Bug 775959] Support OMX_COLOR_FormatYUV420PackedSemiPlanar and OMX_COLOR_Format24bitBGR888

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=775959

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #341775|none                        |reviewed
             status|                            |

--- Comment #2 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Review of attachment 341775:
 --> (https://bugzilla.gnome.org/review?bug=775959&attachment=341775)

According to the spec:

> This format differs from OMX_COLOR_FormatYUV420Se
> miPlanar in that each slice of data shall contain a plane of Y, U and V 
> data, whereas the OMX_COLOR_FormatYUV420Se
> miPlanar format transfers each plane in its entirety. 

This sounds to me like the Packed version is not actually planar but indeed
packed. So not the same as NV12.
Can you confirm that it is the same on the RPi? If so, we should probably add
some configuration to gstomx.conf, some new hack flag, that maps the
Packed*Planar things to the same as the non-Packed*Planar.


But it's not clear to me how exactly that should look like.

::: omx/gstomxvideo.c
@@ +78,3 @@
       break;
+    case OMX_COLOR_Format24bitBGR888:
+      format = GST_VIDEO_FORMAT_BGR;

This one seems ok

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.