From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Antti Palosaari <crope@iki.fi>
Cc: linux-media <linux-media@vger.kernel.org>
Subject: Re: RC-core ".driver_name"
Date: Sun, 27 May 2012 11:38:36 -0300 [thread overview]
Message-ID: <4FC23C6C.8040002@redhat.com> (raw)
In-Reply-To: <4FC0BF3E.2080509@iki.fi>
Em 26-05-2012 08:32, Antti Palosaari escreveu:
> Is there any reason RC-core .driver_name should be set as a module name which registers remote?
>
> http://lxr.free-electrons.com/source/drivers/media/rc/rc-main.c
>
> I see .driver_name is passed to the hotplug:
> if (dev->driver_name)
> ADD_HOTPLUG_VAR("DRV_NAME=%s", dev->driver_name);
>
>
> ir-keytable command shows that name:
> # ir-keytable
> Driver af9015, table rc-digitalnow-tinytwin
>
>
> I would like to use set name same as the device name - not the driver name. And af9015 is not the module name, correct is IMHO dvb_usb_af9015.
The RC hot-plug logic was conceived to use 2 parameters to decide what's the
right keytable to be loaded on userspace:
- an unique per-driver string;
- an unique per-board string.
On your above example, the per-driver string is "af9015", and the per-board one is "rc-digitalnow-tinytwin".
With those two strings, userspace can uniquely associate a keytable with the driver.
The ir-keytable tool also allows saying that everything on with "af9015" will use a
certain keytable, with this syntax at /etc/rc_maps.cfg:
af9015 * ./keycodes/rc5_hauppauge
So, I don't see any reason to change.
Also, changing it will break userspace API, as an old ir-keytable won't properly
handle the tables on a new Kernel. So, it is too late to change it.
Regards,
Mauro
prev parent reply other threads:[~2012-05-27 14:38 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-26 11:32 RC-core ".driver_name" Antti Palosaari
2012-05-27 14:38 ` Mauro Carvalho Chehab [this message]
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=4FC23C6C.8040002@redhat.com \
--to=mchehab@redhat.com \
--cc=crope@iki.fi \
--cc=linux-media@vger.kernel.org \
/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.