Commit 67780d6a authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

V4L/DVB: ir-nec-decoder: Cleanups

Remove dead code and properly name a few constants
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 2f16f631
...@@ -14,38 +14,14 @@ ...@@ -14,38 +14,14 @@
#include <media/ir-core.h> #include <media/ir-core.h>
#define NEC_NBITS 32
#define NEC_UNIT 559979 /* ns */ #define NEC_UNIT 559979 /* ns */
#define NEC_HEADER_MARK (16 * NEC_UNIT) #define NEC_HEADER_MARK (16 * NEC_UNIT)
#define NEC_HEADER_SPACE (8 * NEC_UNIT) #define NEC_HEADER_SPACE (8 * NEC_UNIT)
#define NEC_REPEAT_SPACE (4 * NEC_UNIT) #define NEC_REPEAT_SPACE (4 * NEC_UNIT)
#define NEC_MARK (NEC_UNIT) #define NEC_MARK (NEC_UNIT)
#define NEC_0_SYMBOL (NEC_UNIT) #define NEC_0_SPACE (NEC_UNIT)
#define NEC_1_SYMBOL (3 * NEC_UNIT) #define NEC_1_SPACE (3 * NEC_UNIT)
/* Start time: 4.5 ms + 560 us of the next pulse */
#define MIN_START_TIME (3900000 + 560000)
#define MAX_START_TIME (5100000 + 560000)
/* Bit 1 time: 2.25ms us */
#define MIN_BIT1_TIME 2050000
#define MAX_BIT1_TIME 2450000
/* Bit 0 time: 1.12ms us */
#define MIN_BIT0_TIME 920000
#define MAX_BIT0_TIME 1320000
/* Total IR code is 110 ms, including the 9 ms for the start pulse */
#define MAX_NEC_TIME 4000000
/* 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 */
/* Used to register nec_decoder clients */ /* Used to register nec_decoder clients */
static LIST_HEAD(decoder_list); static LIST_HEAD(decoder_list);
...@@ -223,11 +199,11 @@ static int handle_event(struct input_dev *input_dev, ...@@ -223,11 +199,11 @@ static int handle_event(struct input_dev *input_dev,
if (last_bit) if (last_bit)
goto err; goto err;
if ((ev->delta.tv_nsec >= NEC_0_SYMBOL - NEC_UNIT / 2) && if ((ev->delta.tv_nsec >= NEC_0_SPACE - NEC_UNIT / 2) &&
(ev->delta.tv_nsec < NEC_0_SYMBOL + NEC_UNIT / 2)) (ev->delta.tv_nsec < NEC_0_SPACE + NEC_UNIT / 2))
bit = 0; bit = 0;
else if ((ev->delta.tv_nsec >= NEC_1_SYMBOL - NEC_UNIT / 2) && else if ((ev->delta.tv_nsec >= NEC_1_SPACE - NEC_UNIT / 2) &&
(ev->delta.tv_nsec < NEC_1_SYMBOL + NEC_UNIT / 2)) (ev->delta.tv_nsec < NEC_1_SPACE + NEC_UNIT / 2))
bit = 1; bit = 1;
else { else {
IR_dprintk(1, "Decode failed at %d-th bit (%s) @%luus\n", IR_dprintk(1, "Decode failed at %d-th bit (%s) @%luus\n",
...@@ -256,7 +232,7 @@ static int handle_event(struct input_dev *input_dev, ...@@ -256,7 +232,7 @@ static int handle_event(struct input_dev *input_dev,
data->nec_code.not_command |= 1 << (shift - 24); data->nec_code.not_command |= 1 << (shift - 24);
} }
} }
if (++data->count == 32) { if (++data->count == NEC_NBITS) {
u32 scancode; u32 scancode;
/* /*
* Fixme: may need to accept Extended NEC protocol? * Fixme: may need to accept Extended NEC protocol?
......
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