mediatek: provide NV12 buffers for video decoding for MT8183
crrev.com/c/1767450 changed the default android flexible buffer format
back to YV12, but the video decoding scenario on MT8183 requires NV12
buffers. Add an exception for this case.
BUG=b:141732718
TEST=Play H.264 video with acceleration using both Chrome and Android's
Youtube.
Change-Id: Ie4b80389b22650450ea45f5cdea36a00360899a9
Reviewed-on: https://chromium-review.googlesource.com/1833362
Tested-by: Alexandre Courbot <acourbot@chromium.org>
Commit-Ready: Alexandre Courbot <acourbot@chromium.org>
Legacy-Commit-Queue: Commit Bot <commit-bot@chromium.org>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>
diff --git a/mediatek.c b/mediatek.c
index 270d810..4492207 100644
--- a/mediatek.c
+++ b/mediatek.c
@@ -241,8 +241,9 @@
return DRM_FORMAT_XBGR8888;
case DRM_FORMAT_FLEX_YCbCr_420_888:
#ifdef MTK_MT8183
- /* Only for MT8183 Camera subsystem requires NV12 */
- if (use_flags & (BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE))
+ /* MT8183 camera and decoder subsystems require NV12. */
+ if (use_flags &
+ (BO_USE_CAMERA_READ | BO_USE_CAMERA_WRITE | BO_USE_HW_VIDEO_DECODER))
return DRM_FORMAT_NV12;
#endif
return DRM_FORMAT_YVU420;