Commit 9f154782 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

V4L/DVB: saa7134: don't wait too much to generate an IR event on raw_decode

At raw_decode mode, the key is processed after the end of a timer. The
previous code resets the timer every time something is received at the IR
port. While this works fine with IR's that don't implement repeat, like
Avermedia RM-JX IR, it keeps waiting until keydown, on IR's that implement
NEC repeat command, like the Terratec yellow.

The solution is to change the behaviour to do the timeout after the first
received data.

The timeout is currently set to 15 ms, as it works fine with NEC protcocol.
It may need some adjustments to support other protocols and to better handle
spurious detections that may happen with some IR sensors.
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent ada39630
...@@ -405,6 +405,7 @@ void ir_keyup(struct input_dev *dev) ...@@ -405,6 +405,7 @@ void ir_keyup(struct input_dev *dev)
if (!ir->keypressed) if (!ir->keypressed)
return; return;
IR_dprintk(1, "keyup key 0x%04x\n", ir->keycode);
input_report_key(dev, ir->keycode, 0); input_report_key(dev, ir->keycode, 0);
input_sync(dev); input_sync(dev);
ir->keypressed = 0; ir->keypressed = 0;
......
...@@ -14,21 +14,51 @@ ...@@ -14,21 +14,51 @@
#include <media/ir-core.h> #include <media/ir-core.h>
/* Start time: 4.5 ms */ /* Start time: 4.5 ms + 560 us of the next pulse */
#define MIN_START_TIME 3900000 #define MIN_START_TIME (3900000 + 560000)
#define MAX_START_TIME 5100000 #define MAX_START_TIME (5100000 + 560000)
/* Pulse time: 560 us */ /* Bit 1 time: 2.25ms us */
#define MIN_PULSE_TIME 460000 #define MIN_BIT1_TIME 2050000
#define MAX_PULSE_TIME 660000 #define MAX_BIT1_TIME 2450000
/* Bit 1 space time: 2.25ms-560 us */ /* Bit 0 time: 1.12ms us */
#define MIN_BIT1_TIME 1490000 #define MIN_BIT0_TIME 920000
#define MAX_BIT1_TIME 1890000 #define MAX_BIT0_TIME 1320000
/* Bit 0 space time: 1.12ms-560 us */ /* Total IR code is 110 ms, including the 9 ms for the start pulse */
#define MIN_BIT0_TIME 360000 #define MAX_NEC_TIME 4000000
#define MAX_BIT0_TIME 760000
/* Total IR code is 110 ms, including the 9 ms for the start pulse */
#define MIN_REPEAT_TIME 99000000
#define MAX_REPEAT_TIME 112000000
/* Repeat time: 2.25ms us */
#define MIN_REPEAT_START_TIME 2050000
#define MAX_REPEAT_START_TIME 3000000
#define REPEAT_TIME 240 /* ms */
/** is_repeat - Check if it is a NEC repeat event
* @input_dev: the struct input_dev descriptor of the device
* @pos: the position of the first event
* @len: the length of the buffer
*/
static int is_repeat(struct ir_raw_event *evs, int len, int pos)
{
if ((evs[pos].delta.tv_nsec < MIN_REPEAT_START_TIME) ||
(evs[pos].delta.tv_nsec > MAX_REPEAT_START_TIME))
return 0;
if (++pos >= len)
return 0;
if ((evs[pos].delta.tv_nsec < MIN_REPEAT_TIME) ||
(evs[pos].delta.tv_nsec > MAX_REPEAT_TIME))
return 0;
return 1;
}
/** /**
* __ir_nec_decode() - Decode one NEC pulsecode * __ir_nec_decode() - Decode one NEC pulsecode
...@@ -36,49 +66,59 @@ ...@@ -36,49 +66,59 @@
* @evs: event array with type/duration of pulse/space * @evs: event array with type/duration of pulse/space
* @len: length of the array * @len: length of the array
* @pos: position to start seeking for a code * @pos: position to start seeking for a code
* This function returns the decoded ircode or -EINVAL if no pulse got decoded * This function returns -EINVAL if no pulse got decoded,
* 0 if buffer is empty and 1 if one keycode were handled.
*/ */
static int __ir_nec_decode(struct input_dev *input_dev, static int __ir_nec_decode(struct input_dev *input_dev,
struct ir_raw_event *evs, struct ir_raw_event *evs,
int len, int *pos) int len, int *pos)
{ {
struct ir_input_dev *ir = input_get_drvdata(input_dev);
int count = -1; int count = -1;
int ircode = 0, not_code = 0; int ircode = 0, not_code = 0;
/* Be sure that the first event is an start one and is a pulse */ /* Be sure that the first event is an start one and is a pulse */
for (; *pos < len; (*pos)++) { for (; *pos < len; (*pos)++) {
if (evs[*pos].type & (IR_START_EVENT | IR_PULSE)) /* Very long delays are considered as start events */
if (evs[*pos].delta.tv_nsec > MAX_NEC_TIME)
break;
if (evs[*pos].type & IR_START_EVENT)
break; break;
IR_dprintk(1, "%luus: Spurious NEC %s\n",
(evs[*pos].delta.tv_nsec + 500) / 1000,
(evs[*pos].type & IR_SPACE) ? "space" : "pulse");
} }
if (*pos >= len)
return 0;
(*pos)++; /* First event doesn't contain data */ (*pos)++; /* First event doesn't contain data */
if (*pos >= len) if (evs[*pos].type != IR_PULSE)
goto err;
/* Check if it is a NEC repeat event */
if (is_repeat(evs, len, *pos)) {
*pos += 2;
if (ir->keypressed) {
mod_timer(&ir->raw->timer_keyup,
jiffies + msecs_to_jiffies(REPEAT_TIME));
IR_dprintk(1, "NEC repeat event\n");
return 1;
} else {
IR_dprintk(1, "missing NEC repeat event\n");
return 0; return 0;
}
}
/* First space should have 4.5 ms otherwise is not NEC protocol */ /* First space should have 4.5 ms otherwise is not NEC protocol */
if ((evs[*pos].delta.tv_nsec < MIN_START_TIME) | if ((evs[*pos].delta.tv_nsec < MIN_START_TIME) ||
(evs[*pos].delta.tv_nsec > MAX_START_TIME) | (evs[*pos].delta.tv_nsec > MAX_START_TIME))
(evs[*pos].type != IR_SPACE))
goto err; goto err;
/*
* FIXME: need to implement the repeat sequence
*/
count = 0; count = 0;
for ((*pos)++; *pos < len; (*pos)++) { for ((*pos)++; *pos < len; (*pos)++) {
int bit; int bit;
if ((evs[*pos].delta.tv_nsec < MIN_PULSE_TIME) |
(evs[*pos].delta.tv_nsec > MAX_PULSE_TIME) |
(evs[*pos].type != IR_PULSE))
goto err;
if (++*pos >= len)
goto err;
if (evs[*pos].type != IR_SPACE)
goto err;
if ((evs[*pos].delta.tv_nsec > MIN_BIT1_TIME) && if ((evs[*pos].delta.tv_nsec > MIN_BIT1_TIME) &&
(evs[*pos].delta.tv_nsec < MAX_BIT1_TIME)) (evs[*pos].delta.tv_nsec < MAX_BIT1_TIME))
bit = 1; bit = 1;
...@@ -107,6 +147,7 @@ static int __ir_nec_decode(struct input_dev *input_dev, ...@@ -107,6 +147,7 @@ static int __ir_nec_decode(struct input_dev *input_dev,
if (++count == 32) if (++count == 32)
break; break;
} }
*pos++;
/* /*
* Fixme: may need to accept Extended NEC protocol? * Fixme: may need to accept Extended NEC protocol?
...@@ -119,12 +160,15 @@ static int __ir_nec_decode(struct input_dev *input_dev, ...@@ -119,12 +160,15 @@ static int __ir_nec_decode(struct input_dev *input_dev,
IR_dprintk(1, "NEC scancode 0x%04x\n", ircode); IR_dprintk(1, "NEC scancode 0x%04x\n", ircode);
ir_keydown(input_dev, ircode); ir_keydown(input_dev, ircode);
ir_keyup(input_dev); mod_timer(&ir->raw->timer_keyup,
jiffies + msecs_to_jiffies(REPEAT_TIME));
return ircode; return 1;
err: err:
IR_dprintk(1, "NEC decoded failed at bit %d while decoding %luus time\n", IR_dprintk(1, "NEC decoded failed at bit %d (%s) while decoding %luus time\n",
count, (evs[*pos].delta.tv_nsec + 500) / 1000); count,
(evs[*pos].type & IR_SPACE) ? "space" : "pulse",
(evs[*pos].delta.tv_nsec + 500) / 1000);
return -EINVAL; return -EINVAL;
} }
...@@ -145,7 +189,7 @@ int ir_nec_decode(struct input_dev *input_dev, ...@@ -145,7 +189,7 @@ int ir_nec_decode(struct input_dev *input_dev,
int rc = 0; int rc = 0;
while (pos < len) { while (pos < len) {
if (__ir_nec_decode(input_dev, evs, len, &pos) >= 0) if (__ir_nec_decode(input_dev, evs, len, &pos) > 0)
rc++; rc++;
} }
......
...@@ -17,6 +17,13 @@ ...@@ -17,6 +17,13 @@
/* Define the max number of bit transitions per IR keycode */ /* Define the max number of bit transitions per IR keycode */
#define MAX_IR_EVENT_SIZE 256 #define MAX_IR_EVENT_SIZE 256
static void ir_keyup_timer(unsigned long data)
{
struct input_dev *input_dev = (struct input_dev *)data;
ir_keyup(input_dev);
}
int ir_raw_event_register(struct input_dev *input_dev) int ir_raw_event_register(struct input_dev *input_dev)
{ {
struct ir_input_dev *ir = input_get_drvdata(input_dev); struct ir_input_dev *ir = input_get_drvdata(input_dev);
...@@ -27,6 +34,11 @@ int ir_raw_event_register(struct input_dev *input_dev) ...@@ -27,6 +34,11 @@ int ir_raw_event_register(struct input_dev *input_dev)
size = sizeof(struct ir_raw_event) * MAX_IR_EVENT_SIZE * 2; size = sizeof(struct ir_raw_event) * MAX_IR_EVENT_SIZE * 2;
size = roundup_pow_of_two(size); size = roundup_pow_of_two(size);
init_timer(&ir->raw->timer_keyup);
ir->raw->timer_keyup.function = ir_keyup_timer;
ir->raw->timer_keyup.data = (unsigned long)input_dev;
set_bit(EV_REP, input_dev->evbit);
rc = kfifo_alloc(&ir->raw->kfifo, size, GFP_KERNEL); rc = kfifo_alloc(&ir->raw->kfifo, size, GFP_KERNEL);
return rc; return rc;
...@@ -40,6 +52,8 @@ void ir_raw_event_unregister(struct input_dev *input_dev) ...@@ -40,6 +52,8 @@ void ir_raw_event_unregister(struct input_dev *input_dev)
if (!ir->raw) if (!ir->raw)
return; return;
del_timer_sync(&ir->raw->timer_keyup);
kfifo_free(&ir->raw->kfifo); kfifo_free(&ir->raw->kfifo);
kfree(ir->raw); kfree(ir->raw);
ir->raw = NULL; ir->raw = NULL;
......
...@@ -425,8 +425,11 @@ static void saa7134_input_timer(unsigned long data) ...@@ -425,8 +425,11 @@ static void saa7134_input_timer(unsigned long data)
void ir_raw_decode_timer_end(unsigned long data) void ir_raw_decode_timer_end(unsigned long data)
{ {
struct saa7134_dev *dev = (struct saa7134_dev *)data; struct saa7134_dev *dev = (struct saa7134_dev *)data;
struct card_ir *ir = dev->remote;
ir_raw_event_handle(dev->remote->dev); ir_raw_event_handle(dev->remote->dev);
ir->active = 0;
} }
void saa7134_ir_start(struct saa7134_dev *dev, struct card_ir *ir) void saa7134_ir_start(struct saa7134_dev *dev, struct card_ir *ir)
...@@ -462,6 +465,7 @@ void saa7134_ir_start(struct saa7134_dev *dev, struct card_ir *ir) ...@@ -462,6 +465,7 @@ void saa7134_ir_start(struct saa7134_dev *dev, struct card_ir *ir)
init_timer(&ir->timer_end); init_timer(&ir->timer_end);
ir->timer_end.function = ir_raw_decode_timer_end; ir->timer_end.function = ir_raw_decode_timer_end;
ir->timer_end.data = (unsigned long)dev; ir->timer_end.data = (unsigned long)dev;
ir->active = 0;
} }
} }
...@@ -477,8 +481,10 @@ void saa7134_ir_stop(struct saa7134_dev *dev) ...@@ -477,8 +481,10 @@ void saa7134_ir_stop(struct saa7134_dev *dev)
del_timer_sync(&ir->timer_end); del_timer_sync(&ir->timer_end);
else if (ir->nec_gpio) else if (ir->nec_gpio)
tasklet_kill(&ir->tlet); tasklet_kill(&ir->tlet);
else if (ir->raw_decode) else if (ir->raw_decode) {
del_timer_sync(&ir->timer_end); del_timer_sync(&ir->timer_end);
ir->active = 0;
}
ir->running = 0; ir->running = 0;
} }
...@@ -951,38 +957,23 @@ static int saa7134_raw_decode_irq(struct saa7134_dev *dev) ...@@ -951,38 +957,23 @@ static int saa7134_raw_decode_irq(struct saa7134_dev *dev)
unsigned long timeout; unsigned long timeout;
int count, pulse, oldpulse; int count, pulse, oldpulse;
/* Disable IR IRQ line */
saa_clearl(SAA7134_IRQ2, SAA7134_IRQ2_INTE_GPIO18);
/* Generate initial event */ /* Generate initial event */
saa_clearb(SAA7134_GPIO_GPMODE3, SAA7134_GPIO_GPRESCAN); saa_clearb(SAA7134_GPIO_GPMODE3, SAA7134_GPIO_GPRESCAN);
saa_setb(SAA7134_GPIO_GPMODE3, SAA7134_GPIO_GPRESCAN); saa_setb(SAA7134_GPIO_GPMODE3, SAA7134_GPIO_GPRESCAN);
pulse = saa_readl(SAA7134_GPIO_GPSTATUS0 >> 2) & ir->mask_keydown; pulse = saa_readl(SAA7134_GPIO_GPSTATUS0 >> 2) & ir->mask_keydown;
ir_raw_event_store(dev->remote->dev, pulse? IR_PULSE : IR_SPACE); ir_raw_event_store(dev->remote->dev, pulse? IR_PULSE : IR_SPACE);
#if 1
/* Wait up to 10 ms for event change */
oldpulse = pulse;
for (count = 0; count < 1000; count++) {
udelay(10);
/* rising SAA7134_GPIO_GPRESCAN reads the status */
saa_clearb(SAA7134_GPIO_GPMODE3, SAA7134_GPIO_GPRESCAN);
saa_setb(SAA7134_GPIO_GPMODE3, SAA7134_GPIO_GPRESCAN);
pulse = saa_readl(SAA7134_GPIO_GPSTATUS0 >> 2)
& ir->mask_keydown;
if (pulse != oldpulse)
break;
}
/* Store final event */ /*
ir_raw_event_store(dev->remote->dev, pulse? IR_PULSE : IR_SPACE); * Wait 15 ms from the start of the first IR event before processing
#endif * the event. This time is enough for NEC protocol. May need adjustments
/* Wait 15 ms before deciding to do something else */ * to work with other protocols.
*/
if (!ir->active) {
timeout = jiffies + jiffies_to_msecs(15); timeout = jiffies + jiffies_to_msecs(15);
mod_timer(&ir->timer_end, timeout); mod_timer(&ir->timer_end, timeout);
ir->active = 1;
/* Enable IR IRQ line */ }
saa_setl(SAA7134_IRQ2, SAA7134_IRQ2_INTE_GPIO18);
return 1; return 1;
} }
......
...@@ -18,6 +18,7 @@ ...@@ -18,6 +18,7 @@
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/kfifo.h> #include <linux/kfifo.h>
#include <linux/time.h> #include <linux/time.h>
#include <linux/timer.h>
extern int ir_core_debug; extern int ir_core_debug;
#define IR_dprintk(level, fmt, arg...) if (ir_core_debug >= level) \ #define IR_dprintk(level, fmt, arg...) if (ir_core_debug >= level) \
...@@ -63,6 +64,7 @@ struct ir_raw_event { ...@@ -63,6 +64,7 @@ struct ir_raw_event {
struct ir_raw_event_ctrl { struct ir_raw_event_ctrl {
struct kfifo kfifo; /* fifo for the pulse/space events */ struct kfifo kfifo; /* fifo for the pulse/space events */
struct timespec last_event; /* when last event occurred */ struct timespec last_event; /* when last event occurred */
struct timer_list timer_keyup; /* timer for key release */
}; };
struct ir_input_dev { struct ir_input_dev {
......
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