Commit c3104e1b authored by David Härdeman's avatar David Härdeman Committed by Mauro Carvalho Chehab

[media] lirc_dev: remove sampling kthread

There are no drivers which use this functionality.
Signed-off-by: default avatarDavid Härdeman <david@hardeman.nu>
Signed-off-by: default avatarSean Young <sean@mess.org>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent 2c5a1f44
...@@ -28,7 +28,6 @@ ...@@ -28,7 +28,6 @@
#include <linux/mutex.h> #include <linux/mutex.h>
#include <linux/wait.h> #include <linux/wait.h>
#include <linux/unistd.h> #include <linux/unistd.h>
#include <linux/kthread.h>
#include <linux/bitops.h> #include <linux/bitops.h>
#include <linux/device.h> #include <linux/device.h>
#include <linux/cdev.h> #include <linux/cdev.h>
...@@ -57,9 +56,6 @@ struct irctl { ...@@ -57,9 +56,6 @@ struct irctl {
struct device dev; struct device dev;
struct cdev cdev; struct cdev cdev;
struct task_struct *task;
long jiffies_to_wait;
}; };
static DEFINE_MUTEX(lirc_dev_lock); static DEFINE_MUTEX(lirc_dev_lock);
...@@ -95,59 +91,6 @@ static void lirc_release(struct device *ld) ...@@ -95,59 +91,6 @@ static void lirc_release(struct device *ld)
kfree(ir); kfree(ir);
} }
/* helper function
* reads key codes from driver and puts them into buffer
* returns 0 on success
*/
static int lirc_add_to_buf(struct irctl *ir)
{
int res;
int got_data = -1;
if (!ir->d.add_to_buf)
return 0;
/*
* service the device as long as it is returning
* data and we have space
*/
do {
got_data++;
res = ir->d.add_to_buf(ir->d.data, ir->buf);
} while (!res);
if (res == -ENODEV)
kthread_stop(ir->task);
return got_data ? 0 : res;
}
/* main function of the polling thread
*/
static int lirc_thread(void *irctl)
{
struct irctl *ir = irctl;
do {
if (ir->open) {
if (ir->jiffies_to_wait) {
set_current_state(TASK_INTERRUPTIBLE);
schedule_timeout(ir->jiffies_to_wait);
}
if (kthread_should_stop())
break;
if (!lirc_add_to_buf(ir))
wake_up_interruptible(&ir->buf->wait_poll);
} else {
set_current_state(TASK_INTERRUPTIBLE);
schedule();
}
} while (!kthread_should_stop());
return 0;
}
static const struct file_operations lirc_dev_fops = { static const struct file_operations lirc_dev_fops = {
.owner = THIS_MODULE, .owner = THIS_MODULE,
.read = lirc_dev_fop_read, .read = lirc_dev_fop_read,
...@@ -252,18 +195,8 @@ static int lirc_allocate_driver(struct lirc_driver *d) ...@@ -252,18 +195,8 @@ static int lirc_allocate_driver(struct lirc_driver *d)
return -EBADRQC; return -EBADRQC;
} }
if (d->sample_rate) { if (!d->rbuf && !(d->fops && d->fops->read &&
if (2 > d->sample_rate || HZ < d->sample_rate) { d->fops->poll && d->fops->unlocked_ioctl)) {
dev_err(d->dev, "invalid %d sample rate\n",
d->sample_rate);
return -EBADRQC;
}
if (!d->add_to_buf) {
dev_err(d->dev, "add_to_buf not set\n");
return -EBADRQC;
}
} else if (!d->rbuf && !(d->fops && d->fops->read &&
d->fops->poll && d->fops->unlocked_ioctl)) {
dev_err(d->dev, "undefined read, poll, ioctl\n"); dev_err(d->dev, "undefined read, poll, ioctl\n");
return -EBADRQC; return -EBADRQC;
} }
...@@ -312,22 +245,6 @@ static int lirc_allocate_driver(struct lirc_driver *d) ...@@ -312,22 +245,6 @@ static int lirc_allocate_driver(struct lirc_driver *d)
dev_set_name(&ir->dev, "lirc%d", ir->d.minor); dev_set_name(&ir->dev, "lirc%d", ir->d.minor);
device_initialize(&ir->dev); device_initialize(&ir->dev);
if (d->sample_rate) {
ir->jiffies_to_wait = HZ / d->sample_rate;
/* try to fire up polling thread */
ir->task = kthread_run(lirc_thread, (void *)ir, "lirc_dev");
if (IS_ERR(ir->task)) {
dev_err(d->dev, "cannot run thread for minor = %d\n",
d->minor);
err = -ECHILD;
goto out_sysfs;
}
} else {
/* it means - wait for external event in task queue */
ir->jiffies_to_wait = 0;
}
err = lirc_cdev_add(ir); err = lirc_cdev_add(ir);
if (err) if (err)
goto out_sysfs; goto out_sysfs;
...@@ -404,10 +321,6 @@ int lirc_unregister_driver(int minor) ...@@ -404,10 +321,6 @@ int lirc_unregister_driver(int minor)
return -ENOENT; return -ENOENT;
} }
/* end up polling thread */
if (ir->task)
kthread_stop(ir->task);
dev_dbg(ir->d.dev, "lirc_dev: driver %s unregistered from minor = %d\n", dev_dbg(ir->d.dev, "lirc_dev: driver %s unregistered from minor = %d\n",
ir->d.name, ir->d.minor); ir->d.name, ir->d.minor);
...@@ -470,9 +383,6 @@ int lirc_dev_fop_open(struct inode *inode, struct file *file) ...@@ -470,9 +383,6 @@ int lirc_dev_fop_open(struct inode *inode, struct file *file)
if (ir->buf) if (ir->buf)
lirc_buffer_clear(ir->buf); lirc_buffer_clear(ir->buf);
if (ir->task)
wake_up_process(ir->task);
ir->open++; ir->open++;
error: error:
......
...@@ -1385,7 +1385,6 @@ static struct lirc_driver lirc_template = { ...@@ -1385,7 +1385,6 @@ static struct lirc_driver lirc_template = {
.minor = -1, .minor = -1,
.code_length = 13, .code_length = 13,
.buffer_size = BUFLEN / 2, .buffer_size = BUFLEN / 2,
.sample_rate = 0, /* tell lirc_dev to not start its own kthread */
.chunk_size = 2, .chunk_size = 2,
.fops = &lirc_fops, .fops = &lirc_fops,
.owner = THIS_MODULE, .owner = THIS_MODULE,
......
...@@ -133,12 +133,6 @@ static inline unsigned int lirc_buffer_write(struct lirc_buffer *buf, ...@@ -133,12 +133,6 @@ static inline unsigned int lirc_buffer_write(struct lirc_buffer *buf,
* @buffer_size: Number of FIFO buffers with @chunk_size size. If zero, * @buffer_size: Number of FIFO buffers with @chunk_size size. If zero,
* creates a buffer with BUFLEN size (16 bytes). * creates a buffer with BUFLEN size (16 bytes).
* *
* @sample_rate: if zero, the device will wait for an event with a new
* code to be parsed. Otherwise, specifies the sample
* rate for polling. Value should be between 0
* and HZ. If equal to HZ, it would mean one polling per
* second.
*
* @features: lirc compatible hardware features, like LIRC_MODE_RAW, * @features: lirc compatible hardware features, like LIRC_MODE_RAW,
* LIRC_CAN\_\*, as defined at include/media/lirc.h. * LIRC_CAN\_\*, as defined at include/media/lirc.h.
* *
...@@ -153,14 +147,6 @@ static inline unsigned int lirc_buffer_write(struct lirc_buffer *buf, ...@@ -153,14 +147,6 @@ static inline unsigned int lirc_buffer_write(struct lirc_buffer *buf,
* @max_timeout: Maximum timeout for record. Valid only if * @max_timeout: Maximum timeout for record. Valid only if
* LIRC_CAN_SET_REC_TIMEOUT is defined. * LIRC_CAN_SET_REC_TIMEOUT is defined.
* *
* @add_to_buf: add_to_buf will be called after specified period of the
* time or triggered by the external event, this behavior
* depends on value of the sample_rate this function will
* be called in user context. This routine should return
* 0 if data was added to the buffer and -ENODATA if none
* was available. This should add some number of bits
* evenly divisible by code_length to the buffer.
*
* @rbuf: if not NULL, it will be used as a read buffer, you will * @rbuf: if not NULL, it will be used as a read buffer, you will
* have to write to the buffer by other means, like irq's * have to write to the buffer by other means, like irq's
* (see also lirc_serial.c). * (see also lirc_serial.c).
...@@ -184,7 +170,6 @@ struct lirc_driver { ...@@ -184,7 +170,6 @@ struct lirc_driver {
int minor; int minor;
__u32 code_length; __u32 code_length;
unsigned int buffer_size; /* in chunks holding one code each */ unsigned int buffer_size; /* in chunks holding one code each */
int sample_rate;
__u32 features; __u32 features;
unsigned int chunk_size; unsigned int chunk_size;
...@@ -192,7 +177,6 @@ struct lirc_driver { ...@@ -192,7 +177,6 @@ struct lirc_driver {
void *data; void *data;
int min_timeout; int min_timeout;
int max_timeout; int max_timeout;
int (*add_to_buf)(void *data, struct lirc_buffer *buf);
struct lirc_buffer *rbuf; struct lirc_buffer *rbuf;
struct rc_dev *rdev; struct rc_dev *rdev;
const struct file_operations *fops; const struct file_operations *fops;
......
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