From: Anssi Hannula <anssi.hannula@iki.fi>
To: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: linux-media@vger.kernel.org, linux-input@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 5/7] [media] ati_remote: add keymap for Medion X10 RF remote
Date: Mon, 08 Aug 2011 17:31:49 +0300 [thread overview]
Message-ID: <4E3FF355.6090807@iki.fi> (raw)
In-Reply-To: <20110808055754.GB7329@core.coreip.homeip.net>
On 08.08.2011 08:57, Dmitry Torokhov wrote:
> On Sun, Aug 07, 2011 at 01:18:11AM +0300, Anssi Hannula wrote:
>> Add keymap for the Medion X10 RF remote which uses the ati_remote
>> driver, and default to it based on the usb id.
>
> Since rc-core supports loading custom keytmaps should we ass medion
> keymap here?
>
> I think we should keep the original keymap to avoid regressions, but new
> keymaps should be offloaded to udev.
Well, I simply followed the convention, as all other remotes under
media/ have the default table in-kernel.
I'm not against putting it off-kernel, but in that case the same should
be done for all new media devices. Is that the plan?
> Thanks.
>
>>
>> The keymap is adapted from an earlier patch by
>> Jan Losinski <losinski@wh2.tu-dresden.de>:
>> https://lkml.org/lkml/2011/4/18/104
>> with KEY_PLAYPAUSE replaced by KEY_PAUSE.
>>
>> Signed-off-by: Anssi Hannula <anssi.hannula@iki.fi>
>> ---
>> drivers/media/rc/ati_remote.c | 17 +++--
>> drivers/media/rc/keymaps/Makefile | 1 +
>> drivers/media/rc/keymaps/rc-medion-x10.c | 116 ++++++++++++++++++++++++++++++
>> include/media/rc-map.h | 1 +
>> 4 files changed, 128 insertions(+), 7 deletions(-)
>> create mode 100644 drivers/media/rc/keymaps/rc-medion-x10.c
>>
>> diff --git a/drivers/media/rc/ati_remote.c b/drivers/media/rc/ati_remote.c
>> index 74cc6b1..4576462 100644
>> --- a/drivers/media/rc/ati_remote.c
>> +++ b/drivers/media/rc/ati_remote.c
>> @@ -151,11 +151,11 @@ MODULE_PARM_DESC(mouse, "Enable mouse device, default = yes");
>> #define err(format, arg...) printk(KERN_ERR format , ## arg)
>>
>> static struct usb_device_id ati_remote_table[] = {
>> - { USB_DEVICE(ATI_REMOTE_VENDOR_ID, LOLA_REMOTE_PRODUCT_ID) },
>> - { USB_DEVICE(ATI_REMOTE_VENDOR_ID, LOLA2_REMOTE_PRODUCT_ID) },
>> - { USB_DEVICE(ATI_REMOTE_VENDOR_ID, ATI_REMOTE_PRODUCT_ID) },
>> - { USB_DEVICE(ATI_REMOTE_VENDOR_ID, NVIDIA_REMOTE_PRODUCT_ID) },
>> - { USB_DEVICE(ATI_REMOTE_VENDOR_ID, MEDION_REMOTE_PRODUCT_ID) },
>> + { USB_DEVICE(ATI_REMOTE_VENDOR_ID, LOLA_REMOTE_PRODUCT_ID), .driver_info = (unsigned long)RC_MAP_ATI_X10 },
>> + { USB_DEVICE(ATI_REMOTE_VENDOR_ID, LOLA2_REMOTE_PRODUCT_ID), .driver_info = (unsigned long)RC_MAP_ATI_X10 },
>> + { USB_DEVICE(ATI_REMOTE_VENDOR_ID, ATI_REMOTE_PRODUCT_ID), .driver_info = (unsigned long)RC_MAP_ATI_X10 },
>> + { USB_DEVICE(ATI_REMOTE_VENDOR_ID, NVIDIA_REMOTE_PRODUCT_ID), .driver_info = (unsigned long)RC_MAP_ATI_X10 },
>> + { USB_DEVICE(ATI_REMOTE_VENDOR_ID, MEDION_REMOTE_PRODUCT_ID), .driver_info = (unsigned long)RC_MAP_MEDION_X10 },
>> {} /* Terminating entry */
>> };
>>
>> @@ -714,8 +714,6 @@ static void ati_remote_rc_init(struct ati_remote *ati_remote)
>>
>> usb_to_input_id(ati_remote->udev, &rdev->input_id);
>> rdev->dev.parent = &ati_remote->interface->dev;
>> -
>> - rdev->map_name = RC_MAP_ATI_X10;
>> }
>>
>> static int ati_remote_initialize(struct ati_remote *ati_remote)
>> @@ -827,6 +825,11 @@ static int ati_remote_probe(struct usb_interface *interface, const struct usb_de
>> snprintf(ati_remote->mouse_name, sizeof(ati_remote->mouse_name),
>> "%s mouse", ati_remote->rc_name);
>>
>> + if (id->driver_info)
>> + rc_dev->map_name = (const char *)id->driver_info;
>> + else
>> + rc_dev->map_name = RC_MAP_ATI_X10;
>> +
>> ati_remote_rc_init(ati_remote);
>> mutex_init(&ati_remote->open_mutex);
>>
>> diff --git a/drivers/media/rc/keymaps/Makefile b/drivers/media/rc/keymaps/Makefile
>> index 3ca9265b7..c3d0f34 100644
>> --- a/drivers/media/rc/keymaps/Makefile
>> +++ b/drivers/media/rc/keymaps/Makefile
>> @@ -48,6 +48,7 @@ obj-$(CONFIG_RC_MAP) += rc-adstech-dvb-t-pci.o \
>> rc-lirc.o \
>> rc-lme2510.o \
>> rc-manli.o \
>> + rc-medion-x10.o \
>> rc-msi-digivox-ii.o \
>> rc-msi-digivox-iii.o \
>> rc-msi-tvanywhere.o \
>> diff --git a/drivers/media/rc/keymaps/rc-medion-x10.c b/drivers/media/rc/keymaps/rc-medion-x10.c
>> new file mode 100644
>> index 0000000..ff62aab
>> --- /dev/null
>> +++ b/drivers/media/rc/keymaps/rc-medion-x10.c
>> @@ -0,0 +1,116 @@
>> +/*
>> + * Medion X10 RF remote keytable
>> + *
>> + * Copyright (C) 2011 Anssi Hannula <anssi.hannula@ıki.fi>
>> + *
>> + * This file is based on a keytable provided by
>> + * Jan Losinski <losinski@wh2.tu-dresden.de>
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License as published by
>> + * the Free Software Foundation; either version 2 of the License, or
>> + * (at your option) any later version.
>> + *
>> + * This program is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> + * GNU General Public License for more details.
>> + *
>> + * You should have received a copy of the GNU General Public License along
>> + * with this program; if not, write to the Free Software Foundation, Inc.,
>> + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
>> + */
>> +
>> +#include <media/rc-map.h>
>> +
>> +static struct rc_map_table medion_x10[] = {
>> + { 0xf12c, KEY_TV }, /* TV */
>> + { 0xf22d, KEY_VCR }, /* VCR */
>> + { 0xc904, KEY_DVD }, /* DVD */
>> + { 0xcb06, KEY_AUDIO }, /* MUSIC */
>> +
>> + { 0xf32e, KEY_RADIO }, /* RADIO */
>> + { 0xca05, KEY_DIRECTORY }, /* PHOTO */
>> + { 0xf42f, KEY_INFO }, /* TV-PREVIEW */
>> + { 0xf530, KEY_LIST }, /* CHANNEL-LST */
>> +
>> + { 0xe01b, KEY_SETUP }, /* SETUP */
>> + { 0xf631, KEY_VIDEO }, /* VIDEO DESKTOP */
>> +
>> + { 0xcd08, KEY_VOLUMEDOWN }, /* VOL - */
>> + { 0xce09, KEY_VOLUMEUP }, /* VOL + */
>> + { 0xd00b, KEY_CHANNELUP }, /* CHAN + */
>> + { 0xd10c, KEY_CHANNELDOWN }, /* CHAN - */
>> + { 0xc500, KEY_MUTE }, /* MUTE */
>> +
>> + { 0xf732, KEY_RED }, /* red */
>> + { 0xf833, KEY_GREEN }, /* green */
>> + { 0xf934, KEY_YELLOW }, /* yellow */
>> + { 0xfa35, KEY_BLUE }, /* blue */
>> + { 0xdb16, KEY_TEXT }, /* TXT */
>> +
>> + { 0xd20d, KEY_1 },
>> + { 0xd30e, KEY_2 },
>> + { 0xd40f, KEY_3 },
>> + { 0xd510, KEY_4 },
>> + { 0xd611, KEY_5 },
>> + { 0xd712, KEY_6 },
>> + { 0xd813, KEY_7 },
>> + { 0xd914, KEY_8 },
>> + { 0xda15, KEY_9 },
>> + { 0xdc17, KEY_0 },
>> + { 0xe11c, KEY_SEARCH }, /* TV/RAD, CH SRC */
>> + { 0xe520, KEY_DELETE }, /* DELETE */
>> +
>> + { 0xfb36, KEY_KEYBOARD }, /* RENAME */
>> + { 0xdd18, KEY_SCREEN }, /* SNAPSHOT */
>> +
>> + { 0xdf1a, KEY_UP }, /* up */
>> + { 0xe722, KEY_DOWN }, /* down */
>> + { 0xe21d, KEY_LEFT }, /* left */
>> + { 0xe41f, KEY_RIGHT }, /* right */
>> + { 0xe31e, KEY_OK }, /* OK */
>> +
>> + { 0xfc37, KEY_SELECT }, /* ACQUIRE IMAGE */
>> + { 0xfd38, KEY_EDIT }, /* EDIT IMAGE */
>> +
>> + { 0xe924, KEY_REWIND }, /* rewind (<<) */
>> + { 0xea25, KEY_PLAY }, /* play ( >) */
>> + { 0xeb26, KEY_FORWARD }, /* forward (>>) */
>> + { 0xec27, KEY_RECORD }, /* record ( o) */
>> + { 0xed28, KEY_STOP }, /* stop ([]) */
>> + { 0xee29, KEY_PAUSE }, /* pause ('') */
>> +
>> + { 0xe621, KEY_PREVIOUS }, /* prev */
>> + { 0xfe39, KEY_SWITCHVIDEOMODE }, /* F SCR */
>> + { 0xe823, KEY_NEXT }, /* next */
>> + { 0xde19, KEY_MENU }, /* MENU */
>> + { 0xff3a, KEY_LANGUAGE }, /* AUDIO */
>> +
>> + { 0xc702, KEY_POWER }, /* POWER */
>> +};
>> +
>> +static struct rc_map_list medion_x10_map = {
>> + .map = {
>> + .scan = medion_x10,
>> + .size = ARRAY_SIZE(medion_x10),
>> + .rc_type = RC_TYPE_OTHER,
>> + .name = RC_MAP_MEDION_X10,
>> + }
>> +};
>> +
>> +static int __init init_rc_map_medion_x10(void)
>> +{
>> + return rc_map_register(&medion_x10_map);
>> +}
>> +
>> +static void __exit exit_rc_map_medion_x10(void)
>> +{
>> + rc_map_unregister(&medion_x10_map);
>> +}
>> +
>> +module_init(init_rc_map_medion_x10)
>> +module_exit(exit_rc_map_medion_x10)
>> +
>> +MODULE_LICENSE("GPL");
>> +MODULE_AUTHOR("Anssi Hannula <anssi.hannula@iki.fi>");
>> diff --git a/include/media/rc-map.h b/include/media/rc-map.h
>> index 09e4451..86e7e85 100644
>> --- a/include/media/rc-map.h
>> +++ b/include/media/rc-map.h
>> @@ -106,6 +106,7 @@ void rc_map_init(void);
>> #define RC_MAP_LIRC "rc-lirc"
>> #define RC_MAP_LME2510 "rc-lme2510"
>> #define RC_MAP_MANLI "rc-manli"
>> +#define RC_MAP_MEDION_X10 "rc-medion-x10"
>> #define RC_MAP_MSI_DIGIVOX_II "rc-msi-digivox-ii"
>> #define RC_MAP_MSI_DIGIVOX_III "rc-msi-digivox-iii"
>> #define RC_MAP_MSI_TVANYWHERE_PLUS "rc-msi-tvanywhere-plus"
>> --
>> 1.7.4.4
>>
>
--
Anssi Hannula
next prev parent reply other threads:[~2011-08-08 14:31 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-18 10:41 [PATCH 0/1] Input: ati_remote - extend keytable for medion remotes Jan Losinski
2011-04-18 10:45 ` [PATCH 1/1] " Jan Losinski
2011-05-19 8:54 ` Jan Losinski
2011-05-25 16:32 ` Dmitry Torokhov
2011-07-01 14:01 ` Steffen Barszus
2011-08-06 15:32 ` Steffen Barszus
2011-08-06 21:31 ` Anssi Hannula
2011-08-06 22:18 ` [PATCH 0/7] ati_remote: move to rc-core and other updates Anssi Hannula
2011-08-06 22:18 ` [PATCH 1/7] [media] move ati_remote driver from input/misc to media/rc Anssi Hannula
2011-08-06 22:18 ` [PATCH 2/7] [media] ati_remote: migrate to the rc subsystem Anssi Hannula
2011-08-06 22:18 ` [PATCH 3/7] [media] ati_remote: parent input devices to usb interface Anssi Hannula
2011-08-06 22:18 ` [PATCH 4/7] [media] ati_remote: fix check for a weird byte Anssi Hannula
2011-08-06 22:18 ` [PATCH 5/7] [media] ati_remote: add keymap for Medion X10 RF remote Anssi Hannula
2011-08-08 5:57 ` Dmitry Torokhov
2011-08-08 14:31 ` Anssi Hannula [this message]
2011-08-08 15:57 ` Jarod Wilson
2011-08-06 22:18 ` [PATCH 6/7] [media] ati_remote: add support for SnapStream Firefly remote Anssi Hannula
2011-08-06 22:18 ` [PATCH 7/7] [media] ati_remote: update Kconfig description Anssi Hannula
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=4E3FF355.6090807@iki.fi \
--to=anssi.hannula@iki.fi \
--cc=dmitry.torokhov@gmail.com \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).