mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-08 11:50:43 +09:00
UPSTREAM: [media] lirc_dev: remove double if ... else statement
There are two if ... else which check the same thing in different
part of the code, they can be merged in a single check.
Signed-off-by: Andi Shyti <andi.shyti@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
(cherry picked from commit 6ab86d2aa0)
Signed-off-by: Ziyuan Xu <xzy.xu@rock-chips.com>
This commit is contained in:
@@ -310,13 +310,6 @@ static int lirc_allocate_driver(struct lirc_driver *d)
|
||||
irctls[minor] = ir;
|
||||
d->minor = minor;
|
||||
|
||||
if (d->sample_rate) {
|
||||
ir->jiffies_to_wait = HZ / d->sample_rate;
|
||||
} else {
|
||||
/* it means - wait for external event in task queue */
|
||||
ir->jiffies_to_wait = 0;
|
||||
}
|
||||
|
||||
/* some safety check 8-) */
|
||||
d->name[sizeof(d->name)-1] = '\0';
|
||||
|
||||
@@ -330,6 +323,8 @@ static int lirc_allocate_driver(struct lirc_driver *d)
|
||||
"lirc%u", ir->d.minor);
|
||||
|
||||
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)) {
|
||||
@@ -338,6 +333,9 @@ static int lirc_allocate_driver(struct lirc_driver *d)
|
||||
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);
|
||||
|
||||
Reference in New Issue
Block a user