* [PATCH 1/4] dvb-usb-cxusb: New RC map for Geniatech Mygica T230.
@ 2016-12-09 0:16 CrazyCat
2017-02-03 11:01 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 2+ messages in thread
From: CrazyCat @ 2016-12-09 0:16 UTC (permalink / raw)
To: linux-media
Updated RC map for Geniatech DVB-T/T2 sticks.
Signed-off-by: CrazyCat <crazycat69@narod.ru>
---
drivers/media/usb/dvb-usb/cxusb.c | 42 +++++++++++++++++++++++++++++++++++++--
1 file changed, 40 insertions(+), 2 deletions(-)
diff --git a/drivers/media/usb/dvb-usb/cxusb.c b/drivers/media/usb/dvb-usb/cxusb.c
index 9b8771e..3edc30d 100644
--- a/drivers/media/usb/dvb-usb/cxusb.c
+++ b/drivers/media/usb/dvb-usb/cxusb.c
@@ -653,6 +653,44 @@ static int cxusb_d680_dmb_rc_query(struct dvb_usb_device *d, u32 *event,
{ 0x0025, KEY_POWER },
};
+static struct rc_map_table rc_map_t230_table[] = {
+ { 0x0000, KEY_0 },
+ { 0x0001, KEY_1 },
+ { 0x0002, KEY_2 },
+ { 0x0003, KEY_3 },
+ { 0x0004, KEY_4 },
+ { 0x0005, KEY_5 },
+ { 0x0006, KEY_6 },
+ { 0x0007, KEY_7 },
+ { 0x0008, KEY_8 },
+ { 0x0009, KEY_9 },
+ { 0x000a, KEY_MUTE },
+ { 0x000b, KEY_STOP }, /* Stop */
+ { 0x000c, KEY_POWER2 }, /* Turn on/off application */
+ { 0x000d, KEY_OK }, /* OK */
+ { 0x000e, KEY_CAMERA }, /* Snapshot */
+ { 0x000f, KEY_ZOOM }, /* Full Screen/Restore */
+ { 0x0010, KEY_RIGHT }, /* Right arrow */
+ { 0x0011, KEY_LEFT }, /* Left arrow */
+ { 0x0012, KEY_CHANNELUP },
+ { 0x0013, KEY_CHANNELDOWN },
+ { 0x0014, KEY_SHUFFLE },
+ { 0x0016, KEY_PAUSE },
+ { 0x0017, KEY_PLAY }, /* Play */
+ { 0x001e, KEY_TIME }, /* Time Shift */
+ { 0x001f, KEY_RECORD },
+ { 0x0020, KEY_UP },
+ { 0x0021, KEY_DOWN },
+ { 0x0025, KEY_POWER }, /* Turn off computer */
+ { 0x0026, KEY_REWIND }, /* FR << */
+ { 0x0027, KEY_FASTFORWARD }, /* FF >> */
+ { 0x0029, KEY_ESC },
+ { 0x002b, KEY_VOLUMEUP },
+ { 0x002c, KEY_VOLUMEDOWN },
+ { 0x002d, KEY_CHANNEL }, /* CH Surfing */
+ { 0x0038, KEY_VIDEO }, /* TV/AV/S-Video/YPbPr */
+};
+
static int cxusb_dee1601_demod_init(struct dvb_frontend* fe)
{
static u8 clock_config [] = { CLOCK_CTL, 0x38, 0x28 };
@@ -2317,8 +2355,8 @@ struct dvb_usb_device_properties cxusb_bluebird_dualdig4_rev2_properties = {
.rc.legacy = {
.rc_interval = 100,
- .rc_map_table = rc_map_d680_dmb_table,
- .rc_map_size = ARRAY_SIZE(rc_map_d680_dmb_table),
+ .rc_map_table = rc_map_t230_table,
+ .rc_map_size = ARRAY_SIZE(rc_map_t230_table),
.rc_query = cxusb_d680_dmb_rc_query,
},
--
1.9.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 1/4] dvb-usb-cxusb: New RC map for Geniatech Mygica T230.
2016-12-09 0:16 [PATCH 1/4] dvb-usb-cxusb: New RC map for Geniatech Mygica T230 CrazyCat
@ 2017-02-03 11:01 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 2+ messages in thread
From: Mauro Carvalho Chehab @ 2017-02-03 11:01 UTC (permalink / raw)
To: CrazyCat; +Cc: linux-media
Em Fri, 09 Dec 2016 02:16:24 +0200
CrazyCat <crazycat69@narod.ru> escreveu:
> Updated RC map for Geniatech DVB-T/T2 sticks.
>
> Signed-off-by: CrazyCat <crazycat69@narod.ru>
> ---
> drivers/media/usb/dvb-usb/cxusb.c | 42 +++++++++++++++++++++++++++++++++++++--
> 1 file changed, 40 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/media/usb/dvb-usb/cxusb.c b/drivers/media/usb/dvb-usb/cxusb.c
> index 9b8771e..3edc30d 100644
> --- a/drivers/media/usb/dvb-usb/cxusb.c
> +++ b/drivers/media/usb/dvb-usb/cxusb.c
> @@ -653,6 +653,44 @@ static int cxusb_d680_dmb_rc_query(struct dvb_usb_device *d, u32 *event,
> { 0x0025, KEY_POWER },
> };
>
> +static struct rc_map_table rc_map_t230_table[] = {
> + { 0x0000, KEY_0 },
> + { 0x0001, KEY_1 },
> + { 0x0002, KEY_2 },
> + { 0x0003, KEY_3 },
> + { 0x0004, KEY_4 },
> + { 0x0005, KEY_5 },
> + { 0x0006, KEY_6 },
> + { 0x0007, KEY_7 },
> + { 0x0008, KEY_8 },
> + { 0x0009, KEY_9 },
> + { 0x000a, KEY_MUTE },
> + { 0x000b, KEY_STOP }, /* Stop */
> + { 0x000c, KEY_POWER2 }, /* Turn on/off application */
> + { 0x000d, KEY_OK }, /* OK */
> + { 0x000e, KEY_CAMERA }, /* Snapshot */
> + { 0x000f, KEY_ZOOM }, /* Full Screen/Restore */
> + { 0x0010, KEY_RIGHT }, /* Right arrow */
> + { 0x0011, KEY_LEFT }, /* Left arrow */
> + { 0x0012, KEY_CHANNELUP },
> + { 0x0013, KEY_CHANNELDOWN },
> + { 0x0014, KEY_SHUFFLE },
> + { 0x0016, KEY_PAUSE },
> + { 0x0017, KEY_PLAY }, /* Play */
> + { 0x001e, KEY_TIME }, /* Time Shift */
> + { 0x001f, KEY_RECORD },
> + { 0x0020, KEY_UP },
> + { 0x0021, KEY_DOWN },
> + { 0x0025, KEY_POWER }, /* Turn off computer */
> + { 0x0026, KEY_REWIND }, /* FR << */
> + { 0x0027, KEY_FASTFORWARD }, /* FF >> */
> + { 0x0029, KEY_ESC },
> + { 0x002b, KEY_VOLUMEUP },
> + { 0x002c, KEY_VOLUMEDOWN },
> + { 0x002d, KEY_CHANNEL }, /* CH Surfing */
> + { 0x0038, KEY_VIDEO }, /* TV/AV/S-Video/YPbPr */
> +};
> +
> static int cxusb_dee1601_demod_init(struct dvb_frontend* fe)
> {
> static u8 clock_config [] = { CLOCK_CTL, 0x38, 0x28 };
> @@ -2317,8 +2355,8 @@ struct dvb_usb_device_properties cxusb_bluebird_dualdig4_rev2_properties = {
>
> .rc.legacy = {
> .rc_interval = 100,
> - .rc_map_table = rc_map_d680_dmb_table,
> - .rc_map_size = ARRAY_SIZE(rc_map_d680_dmb_table),
> + .rc_map_table = rc_map_t230_table,
> + .rc_map_size = ARRAY_SIZE(rc_map_t230_table),
> .rc_query = cxusb_d680_dmb_rc_query,
> },
This driver was converted to use the RC core. So, a trivial rebase is
needed.
Yet, there's something here that I'm not comfortable: Why are you
*replacing* the entry, instead of creating a new one? If this is a
new device, then you should be adding a new entry there (or some
logic that would allow detecting the type).
Regards,
Mauro
>
Thanks,
Mauro
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-02-03 11:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-12-09 0:16 [PATCH 1/4] dvb-usb-cxusb: New RC map for Geniatech Mygica T230 CrazyCat
2017-02-03 11:01 ` Mauro Carvalho Chehab
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).