From: Shuah Khan <shuah.kh@samsung.com>
To: gregkh@linuxfoundation.org, m.chehab@samsung.com, tj@kernel.org,
rafael.j.wysocki@intel.com, linux@roeck-us.net,
toshi.kani@hp.com
Cc: Shuah Khan <shuah.kh@samsung.com>,
linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
shuahkhan@gmail.com
Subject: [RFC PATCH 0/2] managed token devres interfaces
Date: Wed, 9 Apr 2014 09:21:06 -0600 [thread overview]
Message-ID: <cover.1397050852.git.shuah.kh@samsung.com> (raw)
Media devices often have hardware resources that are shared
across several functions. For instance, TV tuner cards often
have MUXes, converters, radios, tuners, etc. that are shared
across various functions. However, v4l2, alsa, DVB, usbfs, and
all other drivers have no knowledge of what resources are
shared. For example, users can't access DVB and alsa at the same
time, or the DVB and V4L analog API at the same time, since many
only have one converter that can be in either analog or digital
mode. Accessing and/or changing mode of a converter while it is
in use by another function results in video stream error.
A shared devres that can be locked and unlocked by various drivers
that control media functions on a single media device is needed to
address the above problems.
A token devres that can be looked up by a token for locking, try
locking, unlocking will help avoid adding data structure
dependencies between various media drivers. This token is a unique
string that can be constructed from a common data structure such as
struct device, bus_name, and hardware address.
The devm_token_* interfaces manage access to token resource.
Interfaces:
devm_token_create()
devm_token_destroy()
devm_token_lock()
devm_token_unlock()
Usage:
Create token:
Call devm_token_create() with a token id which is a unique
string.
Lock token: Call devm_token_lock() to lock or try lock a token.
Unlock token: Call devm_token_unlock().
Destroy token: Call devm_token_destroy() to delete the token.
A new devres_* interface to update the status of this token resource
to busy when locked and free when unlocked is necessary to implement
this new managed resource.
devres_update() searches for the resource that matches supplied match
criteria similar to devres_find(). When a match is found, it calls
the update function caller passed in.
This patch set adds a new devres_update) interface and token devres
interfaces.
Test Cases for token devres interfaces: (passed)
- Create, lock, unlock, and destroy sequence.
- Try lock while it is locked. Returns -EBUSY as expected.
- Try lock after destroy. Returns -ENODEV as expected.
- Unlock while it is unlocked. Returns 0 as expected. This is a no-op.
- Try unlock after destroy. Returns -ENODEV as expected.
Special notes for Mauro Chehab:
- Please evaluate if these token devres interfaces cover all media driver
use-cases. If not what is needed to cover them.
- For use-case testing, I generated a string from em28xx device, as this
is common for all em28xx extensions: (hope this holds true when em28xx
uses snd-usb-audio
- Construct string with (dev is struct em28xx *dev)
format: "tuner:%s-%s-%d"
with the following:
dev_name(&dev->udev->dev)
dev->udev->bus->bus_name
dev->tuner_addr
- I added test code to em28xx_card_setup() to test the interfaces:
example token from this test code generated with the format above:
usb 8-1: devm_token_create(): created token: tuner:8-1-0000:00:10.1-0
Shuah Khan (2):
drivers/base: add new devres_update() interface to devres_*
drivers/base: add managed token devres interfaces
drivers/base/Makefile | 2 +-
drivers/base/devres.c | 36 ++++++++
drivers/base/token_devres.c | 204 ++++++++++++++++++++++++++++++++++++++++++
include/linux/device.h | 4 +
include/linux/token_devres.h | 19 ++++
5 files changed, 264 insertions(+), 1 deletion(-)
create mode 100644 drivers/base/token_devres.c
create mode 100644 include/linux/token_devres.h
--
1.7.10.4
next reply other threads:[~2014-04-09 15:21 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-09 15:21 Shuah Khan [this message]
2014-04-09 15:21 ` [RFC PATCH 1/2] drivers/base: add new devres_update() interface to devres_* Shuah Khan
2014-04-09 15:21 ` [RFC PATCH 2/2] drivers/base: add managed token devres interfaces Shuah Khan
2014-04-16 21:58 ` Tejun Heo
2014-04-17 20:01 ` Shuah Khan
2014-04-17 20:10 ` Tejun Heo
2014-04-17 20:22 ` Tejun Heo
2014-04-17 23:27 ` Shuah Khan
2014-04-17 20:50 ` Shuah Khan
2014-04-09 19:17 ` [RFC PATCH 0/2] " Greg KH
2014-04-09 22:44 ` Shuah Khan
2014-04-10 11:04 ` One Thousand Gnomes
2014-04-10 11:38 ` Mauro Carvalho Chehab
2014-04-10 11:46 ` One Thousand Gnomes
2014-04-10 14:39 ` Mauro Carvalho Chehab
2014-04-11 20:28 ` Shuah Khan
2014-04-14 22:48 ` Shuah Khan
2014-04-10 14:32 ` Shuah Khan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1397050852.git.shuah.kh@samsung.com \
--to=shuah.kh@samsung.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=m.chehab@samsung.com \
--cc=rafael.j.wysocki@intel.com \
--cc=shuahkhan@gmail.com \
--cc=tj@kernel.org \
--cc=toshi.kani@hp.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.