V4L-DVB: ir-core: remove the ancillary buffer
authorMauro Carvalho Chehab <mchehab@redhat.com>
Sun, 4 Apr 2010 13:44:51 +0000 (10:44 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 19 May 2010 15:56:57 +0000 (12:56 -0300)
Now that the decoders are state machine, there's no need to create
an ancillary buffer while decoding the protocol. Just call the decoders
code directly, event by event.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/IR/ir-nec-decoder.c
drivers/media/IR/ir-raw-event.c
drivers/media/IR/ir-rc5-decoder.c
include/media/ir-core.h

index 087211c5496a416127f0fe9b9672d68dfab00fe8..28d77356355679a5397a5049efdb212707066c58 100644 (file)
@@ -125,14 +125,14 @@ static struct attribute_group decoder_attribute_group = {
 
 
 /**
- * handle_event() - Decode one NEC pulse or space
+ * ir_nec_decode() - Decode one NEC pulse or space
  * @input_dev: the struct input_dev descriptor of the device
  * @ev:                event array with type/duration of pulse/space
  *
  * This function returns -EINVAL if the pulse violates the state machine
  */
-static int handle_event(struct input_dev *input_dev,
-                       struct ir_raw_event *ev)
+static int ir_nec_decode(struct input_dev *input_dev,
+                        struct ir_raw_event *ev)
 {
        struct decoder_data *data;
        struct ir_input_dev *ir_dev = input_get_drvdata(input_dev);
@@ -289,32 +289,6 @@ checksum_err:
        return -EINVAL;
 }
 
-/**
- * ir_nec_decode() - Decodes all NEC pulsecodes on a given array
- * @input_dev: the struct input_dev descriptor of the device
- * @evs:       event array with type/duration of pulse/space
- * @len:       length of the array
- * This function returns the number of decoded pulses
- */
-static int ir_nec_decode(struct input_dev *input_dev,
-                        struct ir_raw_event *evs,
-                        int len)
-{
-       struct ir_input_dev *ir_dev = input_get_drvdata(input_dev);
-       struct decoder_data *data;
-       int pos = 0;
-       int rc = 0;
-
-       data = get_decoder_data(ir_dev);
-       if (!data || !data->enabled)
-               return 0;
-
-       for (pos = 0; pos < len; pos++)
-               handle_event(input_dev, &evs[pos]);
-
-       return rc;
-}
-
 static int ir_nec_register(struct input_dev *input_dev)
 {
        struct ir_input_dev *ir_dev = input_get_drvdata(input_dev);
index 617e437e2bebb22181e93733b2153d2eeb05fd4f..57990a3379221a11a2a30c689b50b28b9ec71190 100644 (file)
@@ -138,37 +138,33 @@ int ir_raw_event_handle(struct input_dev *input_dev)
 {
        struct ir_input_dev             *ir = input_get_drvdata(input_dev);
        int                             rc;
-       struct ir_raw_event             *evs;
+       struct ir_raw_event             ev;
        int                             len, i;
 
        /*
         * Store the events into a temporary buffer. This allows calling more than
         * one decoder to deal with the received data
         */
-       len = kfifo_len(&ir->raw->kfifo) / sizeof(*evs);
+       len = kfifo_len(&ir->raw->kfifo) / sizeof(ev);
        if (!len)
                return 0;
-       evs = kmalloc(len * sizeof(*evs), GFP_ATOMIC);
 
        for (i = 0; i < len; i++) {
-               rc = kfifo_out(&ir->raw->kfifo, &evs[i], sizeof(*evs));
-               if (rc != sizeof(*evs)) {
+               rc = kfifo_out(&ir->raw->kfifo, &ev, sizeof(ev));
+               if (rc != sizeof(ev)) {
                        IR_dprintk(1, "overflow error: received %d instead of %zd\n",
-                                  rc, sizeof(*evs));
+                                  rc, sizeof(ev));
                        return -EINVAL;
                }
                IR_dprintk(2, "event type %d, time before event: %07luus\n",
-                       evs[i].type, (evs[i].delta.tv_nsec + 500) / 1000);
+                       ev.type, (ev.delta.tv_nsec + 500) / 1000);
+               rc = RUN_DECODER(decode, input_dev, &ev);
        }
 
        /*
         * Call all ir decoders. This allows decoding the same event with
-        * more than one protocol handler. It returns the number of keystrokes
-        * sent to the event interface
+        * more than one protocol handler.
         */
-       rc = RUN_DECODER(decode, input_dev, evs, len);
-
-       kfree(evs);
 
        return rc;
 }
index 4b7eafecd84284914c595454bb429e502e7b4ead..61b58397a3c2a410e35dd154f8baf272d0c2ca47 100644 (file)
@@ -142,7 +142,7 @@ static struct attribute_group decoder_attribute_group = {
  *
  * This function returns -EINVAL if the pulse violates the state machine
  */
-static int handle_event(struct input_dev *input_dev,
+static int ir_rc5_decode(struct input_dev *input_dev,
                        struct ir_raw_event *ev)
 {
        struct decoder_data *data;
@@ -273,32 +273,6 @@ err2:
        return -EINVAL;
 }
 
-/**
- * ir_rc5_decode() - Decodes all RC-5 pulsecodes on a given array
- * @input_dev: the struct input_dev descriptor of the device
- * @evs:       event array with type/duration of pulse/space
- * @len:       length of the array
- * This function returns the number of decoded pulses
- */
-static int ir_rc5_decode(struct input_dev *input_dev,
-                        struct ir_raw_event *evs,
-                        int len)
-{
-       struct ir_input_dev *ir_dev = input_get_drvdata(input_dev);
-       struct decoder_data *data;
-       int pos = 0;
-       int rc = 0;
-
-       data = get_decoder_data(ir_dev);
-       if (!data || !data->enabled)
-               return 0;
-
-       for (pos = 0; pos < len; pos++)
-               handle_event(input_dev, &evs[pos]);
-
-       return rc;
-}
-
 static int ir_rc5_register(struct input_dev *input_dev)
 {
        struct ir_input_dev *ir_dev = input_get_drvdata(input_dev);
index 40900739cb84b5853489bef167c08a9b87b9d1c7..1c1e8d9bd10a07299f5d550579f33841b72fa7aa 100644 (file)
@@ -101,8 +101,7 @@ struct ir_raw_handler {
        struct list_head list;
 
        int (*decode)(struct input_dev *input_dev,
-                     struct ir_raw_event *evs,
-                     int len);
+                     struct ir_raw_event *ev);
        int (*raw_register)(struct input_dev *input_dev);
        int (*raw_unregister)(struct input_dev *input_dev);
 };