New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 655890 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Feature

Blocked on:
issue 675347



Sign in to add a comment

Support MPLANE buffer type in chrome jpeg decode accelerator

Project Member Reported by henryhsu@chromium.org, Oct 14 2016

Issue description

Currently, V4L2JDA only supports SPLANE.
We have to support MPLANE.

 
Components: OS>Kernel>Video
Owner: jcliang@chromium.org
Status: Assigned (was: Untriaged)
Blockedon: 675347
Labels: -Pri-2 videoshortlist Pri-1
Summary: Support MPLANE buffer type in chrome jpeg decode accelerator (was: Support MPLANE buffer type in chrome)
Project Member

Comment 7 by bugdroid1@chromium.org, Dec 23 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b98a5568d356aea64ef47c4039836ff49faa1a5f

commit b98a5568d356aea64ef47c4039836ff49faa1a5f
Author: jcliang <jcliang@chromium.org>
Date: Fri Dec 23 10:03:40 2016

media/gpu: switch v4l2_jpeg_decode_accelerator to use multi-planar APIs

This patch switches v4l2 JDA to use multi-planar APIs and buffers.

BUG= 655890 
BUG=chrome-os-partner:43703
TEST=Deploy my self-built chrome on device and test with hangout.
TEST=Pass jpeg_decode_accelerator_unittest.
R=henryhsu@chromium.org, posciak@chromium.org, wuchengli@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2559423002
Cr-Commit-Position: refs/heads/master@{#440610}

[modify] https://crrev.com/b98a5568d356aea64ef47c4039836ff49faa1a5f/media/gpu/generic_v4l2_device.cc
[modify] https://crrev.com/b98a5568d356aea64ef47c4039836ff49faa1a5f/media/gpu/v4l2_device.cc
[modify] https://crrev.com/b98a5568d356aea64ef47c4039836ff49faa1a5f/media/gpu/v4l2_jpeg_decode_accelerator.cc
[modify] https://crrev.com/b98a5568d356aea64ef47c4039836ff49faa1a5f/media/gpu/v4l2_jpeg_decode_accelerator.h

Status: Fixed (was: Assigned)
Project Member

Comment 9 by bugdroid1@chromium.org, Jan 3 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/1307bd4244383e2411f8356105bc7719d39d0993

commit 1307bd4244383e2411f8356105bc7719d39d0993
Author: Kuang-che Wu <kcwu@chromium.org>
Date: Tue Jan 03 09:54:42 2017

avtest_label_detect: add capability check for HW JPEG MPLANE decode

BUG= chromium:655890 
TEST=run label detector on device with multi-planar decode capability

Change-Id: Icb3d70db7ee72c4c296b1be5fc8798fe3d915b44
Reviewed-on: https://chromium-review.googlesource.com/423301
Commit-Ready: Kuang-che Wu <kcwu@chromium.org>
Tested-by: Kuang-che Wu <kcwu@chromium.org>
Reviewed-by: Ricky Liang <jcliang@chromium.org>

[modify] https://crrev.com/1307bd4244383e2411f8356105bc7719d39d0993/avtest_label_detect/util_v4l2.c

Project Member

Comment 10 by bugdroid1@chromium.org, Jan 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/ecae6a447b1659cc7d9f9b4dfde1978387bbc12e

commit ecae6a447b1659cc7d9f9b4dfde1978387bbc12e
Author: Ricky Liang <jcliang@chromium.org>
Date: Thu Jan 12 02:21:50 2017

avtest_label_detect: check v4l2 MPLANE buffer type for JPEG hw decode

We've changed chrome JPEG hardware decoder to support only MPLANE buffer
type. Let's change the label detector accordingly.

BUG= chromium:655890 
TEST=run label detector on device with multi-planar decode capability

Change-Id: I9e39dd851ce2eaa74fecc78b92a2c08e78f7fd03
Reviewed-on: https://chromium-review.googlesource.com/427478
Commit-Ready: Ricky Liang <jcliang@chromium.org>
Tested-by: Ricky Liang <jcliang@chromium.org>
Reviewed-by: Ricky Liang <jcliang@chromium.org>

[modify] https://crrev.com/ecae6a447b1659cc7d9f9b4dfde1978387bbc12e/avtest_label_detect/detect_hw_video_acc.c

Sign in to add a comment