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

Issue 6897006: CHROMIUM: media: tegra: avp: Allow AVP to have multiple clients. (Closed)

Created:
9 years, 8 months ago by amartin
Modified:
9 years, 7 months ago
Reviewers:
Olof Johansson
CC:
chromium-os-reviews_chromium.org, vb+kernel_google.com, Mandeep Singh Baines
Visibility:
Public.

Description

CHROMIUM: media: tegra: avp: Allow AVP to have multiple clients. Ref-count the AVP open/release requests; if AVP already opened, just inc the refcount and let the request succeed. Change-Id: Ib493e8e0aff6c339a0a9ccda8c1fd8f06932f665 Signed-off-by: Chong Zhang <chzhang@nvidia.com>; Signed-off-by: Allen Martin <amartin@nvidia.com>; R=olofj@chromium.org BUG=chromium-os-partner:3353 TEST=tested with omx H.264 decode under X11

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -16 lines) Patch
M drivers/media/video/tegra/avp/avp.c View 4 chunks +13 lines, -16 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
amartin
9 years, 8 months ago (2011-04-21 22:31:35 UTC) #1
amartin
Fixed title
9 years, 8 months ago (2011-04-21 22:50:10 UTC) #2
amartin
9 years, 8 months ago (2011-04-21 22:50:21 UTC) #3
Olof Johansson
9 years, 8 months ago (2011-04-21 23:01:02 UTC) #4
Beat me to sending a comment about it. :) Thanks, applied.

Powered by Google App Engine
This is Rietveld 408576698