[media] media: rc: nuvoton: remove unneeded call to ir_raw_event_handle
authorHeiner Kallweit <hkallweit1@gmail.com>
Tue, 2 Aug 2016 05:45:30 +0000 (02:45 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Thu, 22 Sep 2016 14:37:01 +0000 (11:37 -0300)
ir_raw_event_handle is called anyway after the hw fifo content stored
in nvt->buf[] has been written to the kfifo. There is not really a
benefit in the potential additional call to ir_raw_event_handle
whilst nvt->buf[] is being processed.
Getting rid of this additional call allows to simplify the code.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/rc/nuvoton-cir.c

index e4158a962c7a48b15a53370a6fcee745c05b98d0..fc462f6bd6bd44dc3076dcab473239a91c415954 100644 (file)
@@ -769,21 +769,11 @@ static void nvt_process_rx_ir_data(struct nvt_dev *nvt)
                        rawir.pulse ? "pulse" : "space", rawir.duration);
 
                ir_raw_event_store_with_filter(nvt->rdev, &rawir);
-
-               /*
-                * BUF_PULSE_BIT indicates end of IR data, BUF_REPEAT_BYTE
-                * indicates end of IR signal, but new data incoming. In both
-                * cases, it means we're ready to call ir_raw_event_handle
-                */
-               if ((sample == BUF_PULSE_BIT) && (i + 1 < nvt->pkts)) {
-                       nvt_dbg("Calling ir_raw_event_handle (signal end)\n");
-                       ir_raw_event_handle(nvt->rdev);
-               }
        }
 
        nvt->pkts = 0;
 
-       nvt_dbg("Calling ir_raw_event_handle (buffer empty)\n");
+       nvt_dbg("Calling ir_raw_event_handle\n");
        ir_raw_event_handle(nvt->rdev);
 
        nvt_dbg_verbose("%s done", __func__);