V4L/DVB: ir-core: Add callbacks for input/evdev open/close on IR core

Especially when IR needs to do polling, it generates lots of wakeups per
second. This makes no sense, if the input event device is closed.

Adds a callback handler to the IR hardware driver, to allow registering
an open/close ops.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

3 files changed, 78 insertions, 17 deletions

diff --git a/drivers/media/video/saa7134/saa7134-core.c b/drivers/media/video/saa7134/saa7134-core.c
index a7ad7810fddc..68cda10e0783 100644
--- a/drivers/media/video/saa7134/saa7134-core.c
+++ b/drivers/media/video/saa7134/saa7134-core.c
@@ -1227,7 +1227,7 @@ static int saa7134_resume(struct pci_dev *pci_dev)
         if (card_has_mpeg(dev))
                 saa7134_ts_init_hw(dev);
         if (dev->remote)
-                saa7134_ir_start(dev, dev->remote);
+                saa7134_ir_start(dev);
         saa7134_hw_enable1(dev);
 
         msleep(100);
diff --git a/drivers/media/video/saa7134/saa7134-input.c b/drivers/media/video/saa7134/saa7134-input.c
index e278d7304b3a..ae17b853f957 100644
--- a/drivers/media/video/saa7134/saa7134-input.c
+++ b/drivers/media/video/saa7134/saa7134-input.c
@@ -400,7 +400,14 @@ static int get_key_pinnacle_color(struct IR_i2c *ir, u32 *ir_key, u32 *ir_raw)
 
 void saa7134_input_irq(struct saa7134_dev *dev)
 {
-        struct card_ir *ir = dev->remote;
+        struct card_ir *ir;
+
+        if (!dev || !dev->remote)
+                return;
+
+        ir = dev->remote;
+        if (!ir->running)
+                return;
 
         if (ir->nec_gpio) {
                 saa7134_nec_irq(dev);
@@ -432,10 +439,20 @@ void ir_raw_decode_timer_end(unsigned long data)
         ir->active = 0;
 }
 
-void saa7134_ir_start(struct saa7134_dev *dev, struct card_ir *ir)
+static int __saa7134_ir_start(void *priv)
 {
+        struct saa7134_dev *dev = priv;
+        struct card_ir *ir;
+
+        if (!dev)
+                return -EINVAL;
+
+        ir  = dev->remote;
+        if (!ir)
+                return -EINVAL;
+
         if (ir->running)
-                return;
+                return 0;
 
         ir->running = 1;
         if (ir->polling) {
@@ -467,11 +484,21 @@ void saa7134_ir_start(struct saa7134_dev *dev, struct card_ir *ir)
                 ir->timer_end.data = (unsigned long)dev;
                 ir->active = 0;
         }
+
+        return 0;
 }
 
-void saa7134_ir_stop(struct saa7134_dev *dev)
+static void __saa7134_ir_stop(void *priv)
 {
-        struct card_ir *ir = dev->remote;
+        struct saa7134_dev *dev = priv;
+        struct card_ir *ir;
+
+        if (!dev)
+                return;
+
+        ir  = dev->remote;
+        if (!ir)
+                return;
 
         if (!ir->running)
                 return;
@@ -487,8 +514,42 @@ void saa7134_ir_stop(struct saa7134_dev *dev)
         }
 
         ir->running = 0;
+
+        return;
+}
+
+int saa7134_ir_start(struct saa7134_dev *dev)
+{
+        if (dev->remote->users)
+                return __saa7134_ir_start(dev);
+
+        return 0;
+}
+
+void saa7134_ir_stop(struct saa7134_dev *dev)
+{
+        if (dev->remote->users)
+                __saa7134_ir_stop(dev);
+}
+
+static int saa7134_ir_open(void *priv)
+{
+        struct saa7134_dev *dev = priv;
+
+        dev->remote->users++;
+        return __saa7134_ir_start(dev);
+}
+
+static void saa7134_ir_close(void *priv)
+{
+        struct saa7134_dev *dev = priv;
+
+        dev->remote->users--;
+        if (!dev->remote->users)
+                __saa7134_ir_stop(dev);
 }
 
+
 int saa7134_ir_change_protocol(void *priv, u64 ir_type)
 {
         struct saa7134_dev *dev = priv;
@@ -513,7 +574,7 @@ int saa7134_ir_change_protocol(void *priv, u64 ir_type)
                 saa7134_ir_stop(dev);
                 ir->nec_gpio = nec_gpio;
                 ir->rc5_gpio = rc5_gpio;
-                saa7134_ir_start(dev, ir);
+                saa7134_ir_start(dev);
         } else {
                 ir->nec_gpio = nec_gpio;
                 ir->rc5_gpio = rc5_gpio;
@@ -788,9 +849,13 @@ int saa7134_input_init1(struct saa7134_dev *dev)
         snprintf(ir->phys, sizeof(ir->phys), "pci-%s/ir0",
                  pci_name(dev->pci));
 
+
+        ir->props.priv = dev;
+        ir->props.open = saa7134_ir_open;
+        ir->props.close = saa7134_ir_close;
+
         if (ir_codes->ir_type != IR_TYPE_OTHER && !raw_decode) {
                 ir->props.allowed_protos = IR_TYPE_RC5 | IR_TYPE_NEC;
-                ir->props.priv = dev;
                 ir->props.change_protocol = saa7134_ir_change_protocol;
 
                 /* Set IR protocol */
@@ -815,25 +880,21 @@ int saa7134_input_init1(struct saa7134_dev *dev)
 
         err = ir_input_register(ir->dev, ir_codes, &ir->props, MODULE_NAME);
         if (err)
-                goto err_out_stop;
+                goto err_out_free;
         if (ir_codes->ir_type != IR_TYPE_OTHER) {
                 err = ir_raw_event_register(ir->dev);
                 if (err)
-                        goto err_out_stop;
+                        goto err_out_free;
         }
 
-        saa7134_ir_start(dev, ir);
-
         /* the remote isn't as bouncy as a keyboard */
         ir->dev->rep[REP_DELAY] = repeat_delay;
         ir->dev->rep[REP_PERIOD] = repeat_period;
 
         return 0;
 
- err_out_stop:
-        saa7134_ir_stop(dev);
+err_out_free:
         dev->remote = NULL;
- err_out_free:
         kfree(ir);
         return err;
 }
diff --git a/drivers/media/video/saa7134/saa7134.h b/drivers/media/video/saa7134/saa7134.h
index c3a1ae0adca0..cad8aee98e53 100644
--- a/drivers/media/video/saa7134/saa7134.h
+++ b/drivers/media/video/saa7134/saa7134.h
@@ -20,7 +20,7 @@
  */
 
 #include <linux/version.h>
-#define SAA7134_VERSION_CODE KERNEL_VERSION(0,2,15)
+#define SAA7134_VERSION_CODE KERNEL_VERSION(0, 2, 16)
 
 #include <linux/pci.h>
 #include <linux/i2c.h>
@@ -810,7 +810,7 @@ int  saa7134_input_init1(struct saa7134_dev *dev);
 void saa7134_input_fini(struct saa7134_dev *dev);
 void saa7134_input_irq(struct saa7134_dev *dev);
 void saa7134_probe_i2c_ir(struct saa7134_dev *dev);
-void saa7134_ir_start(struct saa7134_dev *dev, struct card_ir *ir);
+int saa7134_ir_start(struct saa7134_dev *dev);
 void saa7134_ir_stop(struct saa7134_dev *dev);