[Bug 792812] kmsallocator: add alignment when create fb

[GStreamer (GNOME Bugzilla)]

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

Víctor Manuel Jáquez Leal <vjaquez at igalia.com> changed:

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

--- Comment #3 from Víctor Manuel Jáquez Leal <vjaquez at igalia.com> ---
Review of attachment 367278:
 --> (https://bugzilla.gnome.org/review?bug=792812&attachment=367278)

::: sys/kms/gstkmsutils.c
@@ +140,3 @@
+    case DRM_FORMAT_UYVY:
+    case DRM_FORMAT_YUYV:
+    case DRM_FORMAT_YVYU:

If I understand correctly [1], DRM_FORMAT_YUYV, DRM_FORMAT_YVYU,
DRM_FORMAT_UYVY, etc., have different horizontal and vertical subsampling, thus
this looks wrong.




1.
http://elixir.free-electrons.com/linux/latest/source/drivers/gpu/drm/drm_fourcc.c#L105

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