Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(74)

Issue 2980793002: V4L2ImageProcessor: Use VIDIOC_S_SELECTION for buffer region control (Closed)

Created:
3 years, 5 months ago by mojahsu1
Modified:
3 years, 4 months ago
Reviewers:
wuchengli
CC:
chromium-reviews, posciak+watch_chromium.org, piman+watch_chromium.org, feature-media-reviews_chromium.org, Pawel Osciak
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

V4L2ImageProcessor: Use VIDIOC_S_SELECTION for buffer region control Applications have to use the selection API to work with v4l2 mem2mem devices. The old crop ioctls are not suitable for such hardware. For backward compatibility we may fallback to use old crop APIs if the codec driver doesn't support selection interfaces. BUG=615857 TEST=Run VDA tests and play video on elm with fixed kernel driver for selection path. Run VEA tests and play video on peach_pi for fallback path. CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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/2980793002 Cr-Commit-Position: refs/heads/master@{#494022} Committed: https://chromium.googlesource.com/chromium/src/+/1bcbdbe7292b842776463402c1b5e6db83bafb77

Patch Set 1 #

Patch Set 2 : V4L2ImageProcessor: Use VIDIOC_S_SELECTION for buffer region control #

Unified diffs Side-by-side diffs Delta from patch set Stats (+39 lines, -16 lines) Patch
M media/gpu/v4l2_image_processor.cc View 1 2 chunks +39 lines, -16 lines 0 comments Download

Messages

Total messages: 12 (7 generated)
mojahsu1
3 years, 5 months ago (2017-07-13 04:08:24 UTC) #4
wuchengli
lgtm. Please merger after https://chromium-review.googlesource.com/c/566742 is merged.
3 years, 5 months ago (2017-07-18 05:59:01 UTC) #5
wuchengli
3 years, 5 months ago (2017-07-18 06:02:15 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2980793002/20001
3 years, 4 months ago (2017-08-14 07:23:24 UTC) #9
commit-bot: I haz the power
3 years, 4 months ago (2017-08-14 09:02:08 UTC) #12
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/1bcbdbe7292b842776463402c1b5...

Powered by Google App Engine
This is Rietveld 408576698