Commit ee03e3f0 authored by Kees Cook's avatar Kees Cook Committed by Dmitry Torokhov

Input: touchsceen - convert timers to use timer_setup()

In preparation for unconditionally passing the struct timer_list pointer to
all timer callbacks, switch to using the new timer_setup() and from_timer()
to pass the timer pointer explicitly.
Signed-off-by: default avatarKees Cook <keescook@chromium.org>
Signed-off-by: default avatarDmitry Torokhov <dmitry.torokhov@gmail.com>
parent 4ea40278
...@@ -237,9 +237,9 @@ static void ad7879_ts_event_release(struct ad7879 *ts) ...@@ -237,9 +237,9 @@ static void ad7879_ts_event_release(struct ad7879 *ts)
input_sync(input_dev); input_sync(input_dev);
} }
static void ad7879_timer(unsigned long handle) static void ad7879_timer(struct timer_list *t)
{ {
struct ad7879 *ts = (void *)handle; struct ad7879 *ts = from_timer(ts, t, timer);
ad7879_ts_event_release(ts); ad7879_ts_event_release(ts);
} }
...@@ -570,7 +570,7 @@ int ad7879_probe(struct device *dev, struct regmap *regmap, ...@@ -570,7 +570,7 @@ int ad7879_probe(struct device *dev, struct regmap *regmap,
ts->irq = irq; ts->irq = irq;
ts->regmap = regmap; ts->regmap = regmap;
setup_timer(&ts->timer, ad7879_timer, (unsigned long) ts); timer_setup(&ts->timer, ad7879_timer, 0);
snprintf(ts->phys, sizeof(ts->phys), "%s/input0", dev_name(dev)); snprintf(ts->phys, sizeof(ts->phys), "%s/input0", dev_name(dev));
input_dev->name = "AD7879 Touchscreen"; input_dev->name = "AD7879 Touchscreen";
......
...@@ -208,9 +208,12 @@ static void atmel_wm97xx_acc_pen_up(struct wm97xx *wm) ...@@ -208,9 +208,12 @@ static void atmel_wm97xx_acc_pen_up(struct wm97xx *wm)
} }
} }
static void atmel_wm97xx_pen_timer(unsigned long data) static void atmel_wm97xx_pen_timer(struct timer_list *t)
{ {
atmel_wm97xx_acc_pen_up((struct wm97xx *)data); struct atmel_wm97xx *atmel_wm97xx = from_timer(atmel_wm97xx, t,
pen_timer);
atmel_wm97xx_acc_pen_up(atmel_wm97xx->wm);
} }
static int atmel_wm97xx_acc_startup(struct wm97xx *wm) static int atmel_wm97xx_acc_startup(struct wm97xx *wm)
...@@ -348,8 +351,7 @@ static int __init atmel_wm97xx_probe(struct platform_device *pdev) ...@@ -348,8 +351,7 @@ static int __init atmel_wm97xx_probe(struct platform_device *pdev)
atmel_wm97xx->gpio_pen = atmel_gpio_line; atmel_wm97xx->gpio_pen = atmel_gpio_line;
atmel_wm97xx->gpio_irq = gpio_to_irq(atmel_wm97xx->gpio_pen); atmel_wm97xx->gpio_irq = gpio_to_irq(atmel_wm97xx->gpio_pen);
setup_timer(&atmel_wm97xx->pen_timer, atmel_wm97xx_pen_timer, timer_setup(&atmel_wm97xx->pen_timer, atmel_wm97xx_pen_timer, 0);
(unsigned long)wm);
ret = request_irq(atmel_wm97xx->ac97c_irq, ret = request_irq(atmel_wm97xx->ac97c_irq,
atmel_wm97xx_channel_b_interrupt, atmel_wm97xx_channel_b_interrupt,
......
...@@ -1237,9 +1237,9 @@ static void cyttsp4_stop_wd_timer(struct cyttsp4 *cd) ...@@ -1237,9 +1237,9 @@ static void cyttsp4_stop_wd_timer(struct cyttsp4 *cd)
del_timer_sync(&cd->watchdog_timer); del_timer_sync(&cd->watchdog_timer);
} }
static void cyttsp4_watchdog_timer(unsigned long handle) static void cyttsp4_watchdog_timer(struct timer_list *t)
{ {
struct cyttsp4 *cd = (struct cyttsp4 *)handle; struct cyttsp4 *cd = from_timer(cd, t, watchdog_timer);
dev_vdbg(cd->dev, "%s: Watchdog timer triggered\n", __func__); dev_vdbg(cd->dev, "%s: Watchdog timer triggered\n", __func__);
...@@ -2074,8 +2074,7 @@ struct cyttsp4 *cyttsp4_probe(const struct cyttsp4_bus_ops *ops, ...@@ -2074,8 +2074,7 @@ struct cyttsp4 *cyttsp4_probe(const struct cyttsp4_bus_ops *ops,
} }
/* Setup watchdog timer */ /* Setup watchdog timer */
setup_timer(&cd->watchdog_timer, cyttsp4_watchdog_timer, timer_setup(&cd->watchdog_timer, cyttsp4_watchdog_timer, 0);
(unsigned long)cd);
/* /*
* call startup directly to ensure that the device * call startup directly to ensure that the device
......
...@@ -202,9 +202,9 @@ static irqreturn_t tsc200x_irq_thread(int irq, void *_ts) ...@@ -202,9 +202,9 @@ static irqreturn_t tsc200x_irq_thread(int irq, void *_ts)
return IRQ_HANDLED; return IRQ_HANDLED;
} }
static void tsc200x_penup_timer(unsigned long data) static void tsc200x_penup_timer(struct timer_list *t)
{ {
struct tsc200x *ts = (struct tsc200x *)data; struct tsc200x *ts = from_timer(ts, t, penup_timer);
unsigned long flags; unsigned long flags;
spin_lock_irqsave(&ts->lock, flags); spin_lock_irqsave(&ts->lock, flags);
...@@ -506,7 +506,7 @@ int tsc200x_probe(struct device *dev, int irq, const struct input_id *tsc_id, ...@@ -506,7 +506,7 @@ int tsc200x_probe(struct device *dev, int irq, const struct input_id *tsc_id,
mutex_init(&ts->mutex); mutex_init(&ts->mutex);
spin_lock_init(&ts->lock); spin_lock_init(&ts->lock);
setup_timer(&ts->penup_timer, tsc200x_penup_timer, (unsigned long)ts); timer_setup(&ts->penup_timer, tsc200x_penup_timer, 0);
INIT_DELAYED_WORK(&ts->esd_work, tsc200x_esd_work); INIT_DELAYED_WORK(&ts->esd_work, tsc200x_esd_work);
......
...@@ -146,9 +146,9 @@ static irqreturn_t w90p910_ts_interrupt(int irq, void *dev_id) ...@@ -146,9 +146,9 @@ static irqreturn_t w90p910_ts_interrupt(int irq, void *dev_id)
return IRQ_HANDLED; return IRQ_HANDLED;
} }
static void w90p910_check_pen_up(unsigned long data) static void w90p910_check_pen_up(struct timer_list *t)
{ {
struct w90p910_ts *w90p910_ts = (struct w90p910_ts *) data; struct w90p910_ts *w90p910_ts = from_timer(w90p910_ts, t, timer);
unsigned long flags; unsigned long flags;
spin_lock_irqsave(&w90p910_ts->lock, flags); spin_lock_irqsave(&w90p910_ts->lock, flags);
...@@ -232,8 +232,7 @@ static int w90x900ts_probe(struct platform_device *pdev) ...@@ -232,8 +232,7 @@ static int w90x900ts_probe(struct platform_device *pdev)
w90p910_ts->input = input_dev; w90p910_ts->input = input_dev;
w90p910_ts->state = TS_IDLE; w90p910_ts->state = TS_IDLE;
spin_lock_init(&w90p910_ts->lock); spin_lock_init(&w90p910_ts->lock);
setup_timer(&w90p910_ts->timer, w90p910_check_pen_up, timer_setup(&w90p910_ts->timer, w90p910_check_pen_up, 0);
(unsigned long)w90p910_ts);
res = platform_get_resource(pdev, IORESOURCE_MEM, 0); res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
if (!res) { if (!res) {
......
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