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

Issue 675347 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Dec 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Feature

Blocking:
issue 655890



Sign in to add a comment

add multi-planar API support in s5p-jpeg

Project Member Reported by jcliang@chromium.org, Dec 17 2016

Issue description

To switch chrome JDA to multi-planar we need to add support for multi-planar API in s5p-jpeg first.
 
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 21 2016

Labels: merge-merged-chromeos-3.8
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/kernel/+/0f89c9e0920470acb234110cb9631f0544b63cfc

commit 0f89c9e0920470acb234110cb9631f0544b63cfc
Author: Ricky Liang <jcliang@chromium.org>
Date: Sat Dec 17 12:33:42 2016

CHROMIUM: [media] s5p-jpeg: add support for multi-planar APIs

This patch adds multi-planar APIs to the s5p-jpeg driver. The multi-planar
APIs are identical to the exisiting single-planar APIs except the plane
format info is stored in the v4l2_pixel_format_mplan struct instead
of the v4l2_pixel_format struct.

It might be possible to add multi-planar pixel format support (e.g. support
V4L2_PIX_FMT_YUV420M in addition to V4L2_PIX_FMT_YUV420), but let's leave
that as a strech goal.

BUG= chromium:675347 
TEST=Built and deployed kernel to my peach_pi. Verify with
     jpeg_decode_acceleraotr_unittest and make sure JPEG hardware decode
     is used in Hangout.

Change-Id: I96ba24ef4be569134b67f30f0ed76d65650fb58f
Signed-off-by: Ricky Liang <jcliang@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/421310
Reviewed-by: Pawel Osciak <posciak@chromium.org>

[modify] https://crrev.com/0f89c9e0920470acb234110cb9631f0544b63cfc/drivers/media/platform/s5p-jpeg/jpeg-core.h
[modify] https://crrev.com/0f89c9e0920470acb234110cb9631f0544b63cfc/drivers/media/platform/s5p-jpeg/jpeg-core.c

Status: Verified (was: Started)

Sign in to add a comment