Commit e87b540b authored by David Härdeman's avatar David Härdeman Committed by Mauro Carvalho Chehab

[media] rc-core: merge rc5 and streamzap decoders

Now that the protocol is part of the scancode, it is pretty easy to merge
the rc5 and streamzap decoders. An additional advantage is that the decoder
is now stricter as it waits for the trailing silence before determining that
a command is a valid rc5/streamzap command (which avoids collisions that I've
seen with e.g. Sony protocols).
Signed-off-by: default avatarDavid Härdeman <david@hardeman.nu>
Signed-off-by: default avatarMauro Carvalho Chehab <m.chehab@samsung.com>
parent 9d2f1d3c
...@@ -84,18 +84,6 @@ config IR_SONY_DECODER ...@@ -84,18 +84,6 @@ config IR_SONY_DECODER
Enable this option if you have an infrared remote control which Enable this option if you have an infrared remote control which
uses the Sony protocol, and you need software decoding support. uses the Sony protocol, and you need software decoding support.
config IR_RC5_SZ_DECODER
tristate "Enable IR raw decoder for the RC-5 (streamzap) protocol"
depends on RC_CORE
select BITREVERSE
default y
---help---
Enable this option if you have IR with RC-5 (streamzap) protocol,
and if the IR is decoded in software. (The Streamzap PC Remote
uses an IR protocol that is almost standard RC-5, but not quite,
as it uses an additional bit).
config IR_SANYO_DECODER config IR_SANYO_DECODER
tristate "Enable IR raw decoder for the Sanyo protocol" tristate "Enable IR raw decoder for the Sanyo protocol"
depends on RC_CORE depends on RC_CORE
......
...@@ -9,7 +9,6 @@ obj-$(CONFIG_IR_RC5_DECODER) += ir-rc5-decoder.o ...@@ -9,7 +9,6 @@ obj-$(CONFIG_IR_RC5_DECODER) += ir-rc5-decoder.o
obj-$(CONFIG_IR_RC6_DECODER) += ir-rc6-decoder.o obj-$(CONFIG_IR_RC6_DECODER) += ir-rc6-decoder.o
obj-$(CONFIG_IR_JVC_DECODER) += ir-jvc-decoder.o obj-$(CONFIG_IR_JVC_DECODER) += ir-jvc-decoder.o
obj-$(CONFIG_IR_SONY_DECODER) += ir-sony-decoder.o obj-$(CONFIG_IR_SONY_DECODER) += ir-sony-decoder.o
obj-$(CONFIG_IR_RC5_SZ_DECODER) += ir-rc5-sz-decoder.o
obj-$(CONFIG_IR_SANYO_DECODER) += ir-sanyo-decoder.o obj-$(CONFIG_IR_SANYO_DECODER) += ir-sanyo-decoder.o
obj-$(CONFIG_IR_SHARP_DECODER) += ir-sharp-decoder.o obj-$(CONFIG_IR_SHARP_DECODER) += ir-sharp-decoder.o
obj-$(CONFIG_IR_MCE_KBD_DECODER) += ir-mce_kbd-decoder.o obj-$(CONFIG_IR_MCE_KBD_DECODER) += ir-mce_kbd-decoder.o
......
/* ir-rc5-decoder.c - handle RC5(x) IR Pulse/Space protocol /* ir-rc5-decoder.c - decoder for RC5(x) and StreamZap protocols
* *
* Copyright (C) 2010 by Mauro Carvalho Chehab * Copyright (C) 2010 by Mauro Carvalho Chehab
* Copyright (C) 2010 by Jarod Wilson <jarod@redhat.com>
* *
* This program is free software; you can redistribute it and/or modify * 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 * it under the terms of the GNU General Public License as published by
...@@ -13,23 +14,22 @@ ...@@ -13,23 +14,22 @@
*/ */
/* /*
* This code handles 14 bits RC5 protocols and 20 bits RC5x protocols. * This decoder handles the 14 bit RC5 protocol, 15 bit "StreamZap" protocol
* There are other variants that use a different number of bits. * and 20 bit RC5x protocol.
* This is currently unsupported.
* It considers a carrier of 36 kHz, with a total of 14/20 bits, where
* the first two bits are start bits, and a third one is a filing bit
*/ */
#include "rc-core-priv.h" #include "rc-core-priv.h"
#include <linux/module.h> #include <linux/module.h>
#define RC5_NBITS 14 #define RC5_NBITS 14
#define RC5_SZ_NBITS 15
#define RC5X_NBITS 20 #define RC5X_NBITS 20
#define CHECK_RC5X_NBITS 8 #define CHECK_RC5X_NBITS 8
#define RC5_UNIT 888888 /* ns */ #define RC5_UNIT 888888 /* ns */
#define RC5_BIT_START (1 * RC5_UNIT) #define RC5_BIT_START (1 * RC5_UNIT)
#define RC5_BIT_END (1 * RC5_UNIT) #define RC5_BIT_END (1 * RC5_UNIT)
#define RC5X_SPACE (4 * RC5_UNIT) #define RC5X_SPACE (4 * RC5_UNIT)
#define RC5_TRAILER (10 * RC5_UNIT) /* In reality, approx 100 */
enum rc5_state { enum rc5_state {
STATE_INACTIVE, STATE_INACTIVE,
...@@ -66,7 +66,7 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev) ...@@ -66,7 +66,7 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev)
goto out; goto out;
again: again:
IR_dprintk(2, "RC5(x) decode started at state %i (%uus %s)\n", IR_dprintk(2, "RC5(x/sz) decode started at state %i (%uus %s)\n",
data->state, TO_US(ev.duration), TO_STR(ev.pulse)); data->state, TO_US(ev.duration), TO_STR(ev.pulse));
if (!geq_margin(ev.duration, RC5_UNIT, RC5_UNIT / 2)) if (!geq_margin(ev.duration, RC5_UNIT, RC5_UNIT / 2))
...@@ -80,12 +80,15 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev) ...@@ -80,12 +80,15 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev)
data->state = STATE_BIT_START; data->state = STATE_BIT_START;
data->count = 1; data->count = 1;
/* We just need enough bits to get to STATE_CHECK_RC5X */
data->wanted_bits = RC5X_NBITS;
decrease_duration(&ev, RC5_BIT_START); decrease_duration(&ev, RC5_BIT_START);
goto again; goto again;
case STATE_BIT_START: case STATE_BIT_START:
if (!ev.pulse && geq_margin(ev.duration, RC5_TRAILER, RC5_UNIT / 2)) {
data->state = STATE_FINISHED;
goto again;
}
if (!eq_margin(ev.duration, RC5_BIT_START, RC5_UNIT / 2)) if (!eq_margin(ev.duration, RC5_BIT_START, RC5_UNIT / 2))
break; break;
...@@ -100,9 +103,7 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev) ...@@ -100,9 +103,7 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev)
if (!is_transition(&ev, &dev->raw->prev_ev)) if (!is_transition(&ev, &dev->raw->prev_ev))
break; break;
if (data->count == data->wanted_bits) if (data->count == CHECK_RC5X_NBITS)
data->state = STATE_FINISHED;
else if (data->count == CHECK_RC5X_NBITS)
data->state = STATE_CHECK_RC5X; data->state = STATE_CHECK_RC5X;
else else
data->state = STATE_BIT_START; data->state = STATE_BIT_START;
...@@ -112,13 +113,10 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev) ...@@ -112,13 +113,10 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev)
case STATE_CHECK_RC5X: case STATE_CHECK_RC5X:
if (!ev.pulse && geq_margin(ev.duration, RC5X_SPACE, RC5_UNIT / 2)) { if (!ev.pulse && geq_margin(ev.duration, RC5X_SPACE, RC5_UNIT / 2)) {
/* RC5X */ data->is_rc5x = true;
data->wanted_bits = RC5X_NBITS;
decrease_duration(&ev, RC5X_SPACE); decrease_duration(&ev, RC5X_SPACE);
} else { } else
/* RC5 */ data->is_rc5x = false;
data->wanted_bits = RC5_NBITS;
}
data->state = STATE_BIT_START; data->state = STATE_BIT_START;
goto again; goto again;
...@@ -126,7 +124,7 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev) ...@@ -126,7 +124,7 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev)
if (ev.pulse) if (ev.pulse)
break; break;
if (data->wanted_bits == RC5X_NBITS) { if (data->is_rc5x && data->count == RC5X_NBITS) {
/* RC5X */ /* RC5X */
u8 xdata, command, system; u8 xdata, command, system;
if (!(dev->enabled_protocols & RC_BIT_RC5X)) { if (!(dev->enabled_protocols & RC_BIT_RC5X)) {
...@@ -141,10 +139,7 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev) ...@@ -141,10 +139,7 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev)
scancode = system << 16 | command << 8 | xdata; scancode = system << 16 | command << 8 | xdata;
protocol = RC_TYPE_RC5X; protocol = RC_TYPE_RC5X;
IR_dprintk(1, "RC5X scancode 0x%06x (toggle: %u)\n", } else if (!data->is_rc5x && data->count == RC5_NBITS) {
scancode, toggle);
} else {
/* RC5 */ /* RC5 */
u8 command, system; u8 command, system;
if (!(dev->enabled_protocols & RC_BIT_RC5)) { if (!(dev->enabled_protocols & RC_BIT_RC5)) {
...@@ -158,9 +153,24 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev) ...@@ -158,9 +153,24 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev)
scancode = system << 8 | command; scancode = system << 8 | command;
protocol = RC_TYPE_RC5; protocol = RC_TYPE_RC5;
IR_dprintk(1, "RC5 scancode 0x%04x (toggle: %u)\n", } else if (!data->is_rc5x && data->count == RC5_SZ_NBITS) {
scancode, toggle); /* RC5 StreamZap */
u8 command, system;
if (!(dev->enabled_protocols & RC_BIT_RC5_SZ)) {
data->state = STATE_INACTIVE;
return 0;
} }
command = (data->bits & 0x0003F) >> 0;
system = (data->bits & 0x02FC0) >> 6;
toggle = (data->bits & 0x01000) ? 1 : 0;
scancode = system << 6 | command;
protocol = RC_TYPE_RC5_SZ;
} else
break;
IR_dprintk(1, "RC5(x/sz) scancode 0x%06x (p: %u, t: %u)\n",
scancode, protocol, toggle);
rc_keydown(dev, protocol, scancode, toggle); rc_keydown(dev, protocol, scancode, toggle);
data->state = STATE_INACTIVE; data->state = STATE_INACTIVE;
...@@ -168,14 +178,14 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev) ...@@ -168,14 +178,14 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev)
} }
out: out:
IR_dprintk(1, "RC5(x) decode failed at state %i (%uus %s)\n", IR_dprintk(1, "RC5(x/sz) decode failed at state %i (%uus %s)\n",
data->state, TO_US(ev.duration), TO_STR(ev.pulse)); data->state, TO_US(ev.duration), TO_STR(ev.pulse));
data->state = STATE_INACTIVE; data->state = STATE_INACTIVE;
return -EINVAL; return -EINVAL;
} }
static struct ir_raw_handler rc5_handler = { static struct ir_raw_handler rc5_handler = {
.protocols = RC_BIT_RC5 | RC_BIT_RC5X, .protocols = RC_BIT_RC5 | RC_BIT_RC5X | RC_BIT_RC5_SZ,
.decode = ir_rc5_decode, .decode = ir_rc5_decode,
}; };
...@@ -183,7 +193,7 @@ static int __init ir_rc5_decode_init(void) ...@@ -183,7 +193,7 @@ static int __init ir_rc5_decode_init(void)
{ {
ir_raw_handler_register(&rc5_handler); ir_raw_handler_register(&rc5_handler);
printk(KERN_INFO "IR RC5(x) protocol handler initialized\n"); printk(KERN_INFO "IR RC5(x/sz) protocol handler initialized\n");
return 0; return 0;
} }
...@@ -196,6 +206,6 @@ module_init(ir_rc5_decode_init); ...@@ -196,6 +206,6 @@ module_init(ir_rc5_decode_init);
module_exit(ir_rc5_decode_exit); module_exit(ir_rc5_decode_exit);
MODULE_LICENSE("GPL"); MODULE_LICENSE("GPL");
MODULE_AUTHOR("Mauro Carvalho Chehab"); MODULE_AUTHOR("Mauro Carvalho Chehab and Jarod Wilson");
MODULE_AUTHOR("Red Hat Inc. (http://www.redhat.com)"); MODULE_AUTHOR("Red Hat Inc. (http://www.redhat.com)");
MODULE_DESCRIPTION("RC5(x) IR protocol decoder"); MODULE_DESCRIPTION("RC5(x/sz) IR protocol decoder");
/* ir-rc5-sz-decoder.c - handle RC5 Streamzap IR Pulse/Space protocol
*
* Copyright (C) 2010 by Mauro Carvalho Chehab
* Copyright (C) 2010 by Jarod Wilson <jarod@redhat.com>
*
* 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 version 2 of the License.
*
* 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.
*/
/*
* This code handles the 15 bit RC5-ish protocol used by the Streamzap
* PC Remote.
* It considers a carrier of 36 kHz, with a total of 15 bits, where
* the first two bits are start bits, and a third one is a filing bit
*/
#include "rc-core-priv.h"
#include <linux/module.h>
#define RC5_SZ_NBITS 15
#define RC5_UNIT 888888 /* ns */
#define RC5_BIT_START (1 * RC5_UNIT)
#define RC5_BIT_END (1 * RC5_UNIT)
enum rc5_sz_state {
STATE_INACTIVE,
STATE_BIT_START,
STATE_BIT_END,
STATE_FINISHED,
};
/**
* ir_rc5_sz_decode() - Decode one RC-5 Streamzap pulse or space
* @dev: the struct rc_dev descriptor of the device
* @ev: the struct ir_raw_event descriptor of the pulse/space
*
* This function returns -EINVAL if the pulse violates the state machine
*/
static int ir_rc5_sz_decode(struct rc_dev *dev, struct ir_raw_event ev)
{
struct rc5_sz_dec *data = &dev->raw->rc5_sz;
u8 toggle, command, system;
u32 scancode;
if (!(dev->enabled_protocols & RC_BIT_RC5_SZ))
return 0;
if (!is_timing_event(ev)) {
if (ev.reset)
data->state = STATE_INACTIVE;
return 0;
}
if (!geq_margin(ev.duration, RC5_UNIT, RC5_UNIT / 2))
goto out;
again:
IR_dprintk(2, "RC5-sz decode started at state %i (%uus %s)\n",
data->state, TO_US(ev.duration), TO_STR(ev.pulse));
if (!geq_margin(ev.duration, RC5_UNIT, RC5_UNIT / 2))
return 0;
switch (data->state) {
case STATE_INACTIVE:
if (!ev.pulse)
break;
data->state = STATE_BIT_START;
data->count = 1;
data->wanted_bits = RC5_SZ_NBITS;
decrease_duration(&ev, RC5_BIT_START);
goto again;
case STATE_BIT_START:
if (!eq_margin(ev.duration, RC5_BIT_START, RC5_UNIT / 2))
break;
data->bits <<= 1;
if (!ev.pulse)
data->bits |= 1;
data->count++;
data->state = STATE_BIT_END;
return 0;
case STATE_BIT_END:
if (!is_transition(&ev, &dev->raw->prev_ev))
break;
if (data->count == data->wanted_bits)
data->state = STATE_FINISHED;
else
data->state = STATE_BIT_START;
decrease_duration(&ev, RC5_BIT_END);
goto again;
case STATE_FINISHED:
if (ev.pulse)
break;
/* RC5-sz */
command = (data->bits & 0x0003F) >> 0;
system = (data->bits & 0x02FC0) >> 6;
toggle = (data->bits & 0x01000) ? 1 : 0;
scancode = system << 6 | command;
IR_dprintk(1, "RC5-sz scancode 0x%04x (toggle: %u)\n",
scancode, toggle);
rc_keydown(dev, RC_TYPE_RC5_SZ, scancode, toggle);
data->state = STATE_INACTIVE;
return 0;
}
out:
IR_dprintk(1, "RC5-sz decode failed at state %i (%uus %s)\n",
data->state, TO_US(ev.duration), TO_STR(ev.pulse));
data->state = STATE_INACTIVE;
return -EINVAL;
}
static struct ir_raw_handler rc5_sz_handler = {
.protocols = RC_BIT_RC5_SZ,
.decode = ir_rc5_sz_decode,
};
static int __init ir_rc5_sz_decode_init(void)
{
ir_raw_handler_register(&rc5_sz_handler);
printk(KERN_INFO "IR RC5 (streamzap) protocol handler initialized\n");
return 0;
}
static void __exit ir_rc5_sz_decode_exit(void)
{
ir_raw_handler_unregister(&rc5_sz_handler);
}
module_init(ir_rc5_sz_decode_init);
module_exit(ir_rc5_sz_decode_exit);
MODULE_LICENSE("GPL");
MODULE_AUTHOR("Jarod Wilson <jarod@redhat.com>");
MODULE_AUTHOR("Red Hat Inc. (http://www.redhat.com)");
MODULE_DESCRIPTION("RC5 (streamzap) IR protocol decoder");
...@@ -15,9 +15,7 @@ ...@@ -15,9 +15,7 @@
static struct rc_map_table streamzap[] = { static struct rc_map_table streamzap[] = {
/* /*
* The Streamzap remote is almost, but not quite, RC-5, as it has an extra * The Streamzap remote is almost, but not quite, RC-5, as it has an extra
* bit in it, which throws the in-kernel RC-5 decoder for a loop. Currently, * bit in it.
* an additional RC-5-sz decoder is being deployed to support it, but it
* may be possible to merge it back with the standard RC-5 decoder.
*/ */
{ 0x28c0, KEY_NUMERIC_0 }, { 0x28c0, KEY_NUMERIC_0 },
{ 0x28c1, KEY_NUMERIC_1 }, { 0x28c1, KEY_NUMERIC_1 },
......
...@@ -54,7 +54,7 @@ struct ir_raw_event_ctrl { ...@@ -54,7 +54,7 @@ struct ir_raw_event_ctrl {
int state; int state;
u32 bits; u32 bits;
unsigned count; unsigned count;
unsigned wanted_bits; bool is_rc5x;
} rc5; } rc5;
struct rc6_dec { struct rc6_dec {
int state; int state;
...@@ -77,12 +77,6 @@ struct ir_raw_event_ctrl { ...@@ -77,12 +77,6 @@ struct ir_raw_event_ctrl {
bool first; bool first;
bool toggle; bool toggle;
} jvc; } jvc;
struct rc5_sz_dec {
int state;
u32 bits;
unsigned count;
unsigned wanted_bits;
} rc5_sz;
struct sanyo_dec { struct sanyo_dec {
int state; int state;
unsigned count; unsigned count;
......
...@@ -63,13 +63,6 @@ MODULE_DEVICE_TABLE(usb, streamzap_table); ...@@ -63,13 +63,6 @@ MODULE_DEVICE_TABLE(usb, streamzap_table);
/* number of samples buffered */ /* number of samples buffered */
#define SZ_BUF_LEN 128 #define SZ_BUF_LEN 128
/* from ir-rc5-sz-decoder.c */
#ifdef CONFIG_IR_RC5_SZ_DECODER_MODULE
#define load_rc5_sz_decode() request_module("ir-rc5-sz-decoder")
#else
#define load_rc5_sz_decode() {}
#endif
enum StreamzapDecoderState { enum StreamzapDecoderState {
PulseSpace, PulseSpace,
FullPulse, FullPulse,
...@@ -452,9 +445,6 @@ static int streamzap_probe(struct usb_interface *intf, ...@@ -452,9 +445,6 @@ static int streamzap_probe(struct usb_interface *intf,
dev_info(sz->dev, "Registered %s on usb%d:%d\n", name, dev_info(sz->dev, "Registered %s on usb%d:%d\n", name,
usbdev->bus->busnum, usbdev->devnum); usbdev->bus->busnum, usbdev->devnum);
/* Load the streamzap not-quite-rc5 decoder too */
load_rc5_sz_decode();
return 0; return 0;
rc_dev_fail: rc_dev_fail:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment