media: rc-core: rename ir_raw_event_reset to ir_raw_event_overflow
authorSean Young <sean@mess.org>
Sat, 15 Jan 2022 10:12:35 +0000 (11:12 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Fri, 28 Jan 2022 18:32:50 +0000 (19:32 +0100)
The driver report a reset event when the hardware reports and overflow.
There is no reason to have a generic "reset" event.

Signed-off-by: Sean Young <sean@mess.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
25 files changed:
drivers/media/pci/cx23885/cx23885-input.c
drivers/media/rc/fintek-cir.c
drivers/media/rc/igorplugusb.c
drivers/media/rc/iguanair.c
drivers/media/rc/ir-hix5hd2.c
drivers/media/rc/ir-imon-decoder.c
drivers/media/rc/ir-jvc-decoder.c
drivers/media/rc/ir-mce_kbd-decoder.c
drivers/media/rc/ir-nec-decoder.c
drivers/media/rc/ir-rc5-decoder.c
drivers/media/rc/ir-rc6-decoder.c
drivers/media/rc/ir-rcmm-decoder.c
drivers/media/rc/ir-sanyo-decoder.c
drivers/media/rc/ir-sharp-decoder.c
drivers/media/rc/ir-sony-decoder.c
drivers/media/rc/ir-xmp-decoder.c
drivers/media/rc/ite-cir.c
drivers/media/rc/lirc_dev.c
drivers/media/rc/nuvoton-cir.c
drivers/media/rc/rc-core-priv.h
drivers/media/rc/rc-ir-raw.c
drivers/media/rc/st_rc.c
drivers/media/rc/sunxi-cir.c
drivers/media/rc/winbond-cir.c
include/media/rc-core.h

index 19c34e5510ee67ac7151cb6cfc05cd19768ca69e..d2e84c6457e0abf7f5fbda7511869c9769505bc2 100644 (file)
@@ -55,7 +55,7 @@ static void cx23885_input_process_measurements(struct cx23885_dev *dev,
        } while (num != 0);
 
        if (overrun)
-               ir_raw_event_reset(kernel_ir->rc);
+               ir_raw_event_overflow(kernel_ir->rc);
        else if (handle)
                ir_raw_event_handle(kernel_ir->rc);
 }
index b0d580566e4eaeb5c2d92f5511e0c76368c074d8..3fb0968efd57d3f7caad1994013e108b7b0f5195 100644 (file)
@@ -287,7 +287,7 @@ static void fintek_process_rx_ir_data(struct fintek_dev *fintek)
                        if (fintek->rem)
                                fintek->parser_state = PARSE_IRDATA;
                        else
-                               ir_raw_event_reset(fintek->rdev);
+                               ir_raw_event_overflow(fintek->rdev);
                        break;
                case SUBCMD:
                        fintek->rem = fintek_cmdsize(fintek->cmd, sample);
index 3e9988ee785f0cf4f6a3804c253a8defb53f3586..b40dbf5001869eb5e7954248091a752f8931e88e 100644 (file)
@@ -67,7 +67,7 @@ static void igorplugusb_irdata(struct igorplugusb *ir, unsigned len)
                if (overflow > 0) {
                        dev_warn(ir->dev, "receive overflow, at least %u lost",
                                                                overflow);
-                       ir_raw_event_reset(ir->rc);
+                       ir_raw_event_overflow(ir->rc);
                }
 
                do {
index f8d080e41f4c1fd0e73123e90184aed54339097c..c9cb8277723f409d3616784e054b5c0be7c12dcf 100644 (file)
@@ -109,7 +109,7 @@ static void process_ir_data(struct iguanair *ir, unsigned len)
                        break;
                case CMD_RX_OVERFLOW:
                        dev_warn(ir->dev, "receive overflow\n");
-                       ir_raw_event_reset(ir->rc);
+                       ir_raw_event_overflow(ir->rc);
                        break;
                default:
                        dev_warn(ir->dev, "control code %02x received\n",
index e0be6471afe5f3494532a8c96b4e5eca67b77540..4ff954b11dc777ce7ebd118cc13b92372fb6fb09 100644 (file)
@@ -194,7 +194,7 @@ static irqreturn_t hix5hd2_ir_rx_interrupt(int irq, void *data)
                 * IR_INTS availably since logic would not clear
                 * fifo when overflow, drv do the job
                 */
-               ir_raw_event_reset(priv->rdev);
+               ir_raw_event_overflow(priv->rdev);
                symb_num = readl_relaxed(priv->base + IR_DATAH);
                for (i = 0; i < symb_num; i++)
                        readl_relaxed(priv->base + IR_DATAL);
index 41dbbef27fa64678e3cc70b61a9e93eb7799e52c..dc68f64e7b5106d2abac4aabd512c415c5e44782 100644 (file)
@@ -95,7 +95,7 @@ static int ir_imon_decode(struct rc_dev *dev, struct ir_raw_event ev)
        struct imon_dec *data = &dev->raw->imon;
 
        if (!is_timing_event(ev)) {
-               if (ev.reset)
+               if (ev.overflow)
                        data->state = STATE_INACTIVE;
                return 0;
        }
index 470f2e1fd50778ee38bed5bdc54f76db5e8bbdaa..8b10954d2b6b2ead0ac22fe9f8c13cc727dc94e0 100644 (file)
@@ -40,7 +40,7 @@ static int ir_jvc_decode(struct rc_dev *dev, struct ir_raw_event ev)
        struct jvc_dec *data = &dev->raw->jvc;
 
        if (!is_timing_event(ev)) {
-               if (ev.reset)
+               if (ev.overflow)
                        data->state = STATE_INACTIVE;
                return 0;
        }
index 1524dc0fc566e25530b6969201ed5a289619f225..66e8feb9a569a884aeac6a82d36b377f8939d37b 100644 (file)
@@ -221,7 +221,7 @@ static int ir_mce_kbd_decode(struct rc_dev *dev, struct ir_raw_event ev)
        struct lirc_scancode lsc = {};
 
        if (!is_timing_event(ev)) {
-               if (ev.reset)
+               if (ev.overflow)
                        data->state = STATE_INACTIVE;
                return 0;
        }
index b4c3e4baf34d53203093cba9e2932d21e006e382..37b99432ad0dd574c9330b8ceb2e24caef725c08 100644 (file)
@@ -44,7 +44,7 @@ static int ir_nec_decode(struct rc_dev *dev, struct ir_raw_event ev)
        u8 address, not_address, command, not_command;
 
        if (!is_timing_event(ev)) {
-               if (ev.reset)
+               if (ev.overflow)
                        data->state = STATE_INACTIVE;
                return 0;
        }
index d58b6226afeb2c482d1ef26b98af92f06f46af07..82d7f6ad2338c533839249d84aaad1b1835ba8df 100644 (file)
@@ -45,7 +45,7 @@ static int ir_rc5_decode(struct rc_dev *dev, struct ir_raw_event ev)
        enum rc_proto protocol;
 
        if (!is_timing_event(ev)) {
-               if (ev.reset)
+               if (ev.overflow)
                        data->state = STATE_INACTIVE;
                return 0;
        }
index 0657ad5eef481f2b33704cbfa7927a7b97c90ca0..3b2c8bab3e7344925f5e78469de92887788e21cf 100644 (file)
@@ -85,7 +85,7 @@ static int ir_rc6_decode(struct rc_dev *dev, struct ir_raw_event ev)
        enum rc_proto protocol;
 
        if (!is_timing_event(ev)) {
-               if (ev.reset)
+               if (ev.overflow)
                        data->state = STATE_INACTIVE;
                return 0;
        }
index fd9ec69a3718f6e90b7a5147cad7caed3a2ed08b..a8a34436fe85735a671c13939f358dc45501c788 100644 (file)
@@ -69,7 +69,7 @@ static int ir_rcmm_decode(struct rc_dev *dev, struct ir_raw_event ev)
                return 0;
 
        if (!is_timing_event(ev)) {
-               if (ev.reset)
+               if (ev.overflow)
                        data->state = STATE_INACTIVE;
                return 0;
        }
index bfc181be1044e3da4a9fef19119cec9961eb0a00..2bc98c3428829051ec277ab188fe780568bc95a0 100644 (file)
@@ -51,8 +51,8 @@ static int ir_sanyo_decode(struct rc_dev *dev, struct ir_raw_event ev)
        u8 command, not_command;
 
        if (!is_timing_event(ev)) {
-               if (ev.reset) {
-                       dev_dbg(&dev->dev, "SANYO event reset received. reset to state 0\n");
+               if (ev.overflow) {
+                       dev_dbg(&dev->dev, "SANYO event overflow received. reset to state 0\n");
                        data->state = STATE_INACTIVE;
                }
                return 0;
index d09c38c07dbdb8670f0dc9ccdbfe7038272b17da..3d8488c39c5615343c9f3e797a3d143a3604a0a3 100644 (file)
@@ -41,7 +41,7 @@ static int ir_sharp_decode(struct rc_dev *dev, struct ir_raw_event ev)
        u32 msg, echo, address, command, scancode;
 
        if (!is_timing_event(ev)) {
-               if (ev.reset)
+               if (ev.overflow)
                        data->state = STATE_INACTIVE;
                return 0;
        }
index d760d52abaa2f6d5e455da84bfa29170a50a88e4..bb25867ecb5ebe0649c348917841bffd5929917d 100644 (file)
@@ -39,7 +39,7 @@ static int ir_sony_decode(struct rc_dev *dev, struct ir_raw_event ev)
        u8 device, subdevice, function;
 
        if (!is_timing_event(ev)) {
-               if (ev.reset)
+               if (ev.overflow)
                        data->state = STATE_INACTIVE;
                return 0;
        }
index ff94f48bda32fd15905e0431236f43c155f25715..dc36b68739cb9ae19f91f5f07100fa91857bb98b 100644 (file)
@@ -37,7 +37,7 @@ static int ir_xmp_decode(struct rc_dev *dev, struct ir_raw_event ev)
        struct xmp_dec *data = &dev->raw->xmp;
 
        if (!is_timing_event(ev)) {
-               if (ev.reset)
+               if (ev.overflow)
                        data->state = STATE_INACTIVE;
                return 0;
        }
index 4f77d4ebacdc5343e095d169a63cb9e8a2f3597a..fcfadd7ea31cf300a451ca9be73547f7766f3f92 100644 (file)
@@ -238,7 +238,7 @@ static irqreturn_t ite_cir_isr(int irq, void *data)
        /* Check for RX overflow */
        if (iflags & ITE_IRQ_RX_FIFO_OVERRUN) {
                dev_warn(&dev->rdev->dev, "receive overflow\n");
-               ir_raw_event_reset(dev->rdev);
+               ir_raw_event_overflow(dev->rdev);
        }
 
        /* check for the receive interrupt */
index a080291c4b06985d261d895bb317be195340b460..fa4671fc92be2f1965cd0c3baf54b075ab9f1335 100644 (file)
@@ -41,17 +41,17 @@ void lirc_raw_event(struct rc_dev *dev, struct ir_raw_event ev)
        struct lirc_fh *fh;
        int sample;
 
-       /* Packet start */
-       if (ev.reset) {
+       /* Receiver overflow, data missing */
+       if (ev.overflow) {
                /*
                 * Userspace expects a long space event before the start of
                 * the signal to use as a sync.  This may be done with repeat
-                * packets and normal samples.  But if a reset has been sent
+                * packets and normal samples.  But if an overflow has been sent
                 * then we assume that a long time has passed, so we send a
                 * space with the maximum time value.
                 */
                sample = LIRC_SPACE(LIRC_VALUE_MASK);
-               dev_dbg(&dev->dev, "delivering reset sync space to lirc_dev\n");
+               dev_dbg(&dev->dev, "delivering overflow space to lirc_dev\n");
 
        /* Carrier reports */
        } else if (ev.carrier_report) {
index 8a37f083fe3d75c23e5f642303f739e02bdf49eb..2214d41ef57942ccd1c4bc3b7a45539dc402cccf 100644 (file)
@@ -742,7 +742,7 @@ static void nvt_handle_rx_fifo_overrun(struct nvt_dev *nvt)
 
        nvt->pkts = 0;
        nvt_clear_cir_fifo(nvt);
-       ir_raw_event_reset(nvt->rdev);
+       ir_raw_event_overflow(nvt->rdev);
 }
 
 /* copy data from hardware rx fifo into driver buffer */
index 62f032dffd33a09413c133a1bcbae5f5fb1f431f..ef1e95e1af7fcccda49324e375b940cc92c627f2 100644 (file)
@@ -190,7 +190,7 @@ static inline void decrease_duration(struct ir_raw_event *ev, unsigned duration)
 /* Returns true if event is normal pulse/space event */
 static inline bool is_timing_event(struct ir_raw_event ev)
 {
-       return !ev.carrier_report && !ev.reset;
+       return !ev.carrier_report && !ev.overflow;
 }
 
 #define TO_STR(is_pulse)               ((is_pulse) ? "pulse" : "space")
index c65bba4ec473907558554fab0e2f4d51e7b65d87..16e33d7eaaa2d8bb914e91d4807e57236718b1ff 100644 (file)
@@ -35,8 +35,6 @@ static int ir_raw_event_thread(void *data)
                                    !is_transition(&ev, &raw->prev_ev))
                                        dev_warn_once(&dev->dev, "two consecutive events of type %s",
                                                      TO_STR(ev.pulse));
-                               if (raw->prev_ev.reset && ev.pulse == 0)
-                                       dev_warn_once(&dev->dev, "timing event after reset should be pulse");
                        }
                        list_for_each_entry(handler, &ir_raw_handler_list, list)
                                if (dev->enabled_protocols &
index 4e419dbbacd32e7f019958f1717e0de7a805cab7..19e987a048ccb9080933cbc1addfcef5f28ad32e 100644 (file)
@@ -111,7 +111,7 @@ static irqreturn_t st_rc_rx_interrupt(int irq, void *data)
                int_status = readl(dev->rx_base + IRB_RX_INT_STATUS);
                if (unlikely(int_status & IRB_RX_OVERRUN_INT)) {
                        /* discard the entire collection in case of errors!  */
-                       ir_raw_event_reset(dev->rdev);
+                       ir_raw_event_overflow(dev->rdev);
                        dev_info(dev->dev, "IR RX overrun\n");
                        writel(IRB_RX_OVERRUN_INT,
                                        dev->rx_base + IRB_RX_INT_CLEAR);
index 391a591c1b7547b6f6f784df4af07773d5d4fe2e..b631a81e58bb1eea338587288212dc96175ed6ed 100644 (file)
@@ -126,7 +126,7 @@ static irqreturn_t sunxi_ir_irq(int irqno, void *dev_id)
        }
 
        if (status & REG_RXSTA_ROI) {
-               ir_raw_event_reset(ir->rc);
+               ir_raw_event_overflow(ir->rc);
        } else if (status & REG_RXSTA_RPE) {
                ir_raw_event_set_idle(ir->rc, true);
                ir_raw_event_handle(ir->rc);
index 94efb035d21b3ff99c670454637e6058513aabb0..25884a79985c8ad8bc7a5476444a071da509ac9a 100644 (file)
@@ -470,7 +470,7 @@ wbcir_irq_handler(int irqno, void *cookie)
                /* RX overflow? (read clears bit) */
                if (inb(data->sbase + WBCIR_REG_SP3_LSR) & WBCIR_RX_OVERRUN) {
                        data->rxstate = WBCIR_RXSTATE_ERROR;
-                       ir_raw_event_reset(data->dev);
+                       ir_raw_event_overflow(data->dev);
                }
 
                /* TX underflow? */
index 33b3f7fcf92e7bd3fce5642a45b4aa0688b12953..803349599c272bdd23c4ac50a7e1c7277b340458 100644 (file)
@@ -298,7 +298,7 @@ struct ir_raw_event {
        u8                      duty_cycle;
 
        unsigned                pulse:1;
-       unsigned                reset:1;
+       unsigned                overflow:1;
        unsigned                timeout:1;
        unsigned                carrier_report:1;
 };
@@ -321,9 +321,9 @@ int ir_raw_encode_scancode(enum rc_proto protocol, u32 scancode,
                           struct ir_raw_event *events, unsigned int max);
 int ir_raw_encode_carrier(enum rc_proto protocol);
 
-static inline void ir_raw_event_reset(struct rc_dev *dev)
+static inline void ir_raw_event_overflow(struct rc_dev *dev)
 {
-       ir_raw_event_store(dev, &((struct ir_raw_event) { .reset = true }));
+       ir_raw_event_store(dev, &((struct ir_raw_event) { .overflow = true }));
        dev->idle = true;
        ir_raw_event_handle(dev);
 }