Messages in this thread | | | Date | Sat, 16 Feb 2019 21:29:59 -0800 | From | Dmitry Torokhov <> | Subject | Re: [PATCH] Input: synaptics_i2c - remove redundant spinlock |
| |
Hi Sven,
On Mon, Feb 11, 2019 at 08:34:42PM -0500, thesven73@gmail.com wrote: > @@ -372,13 +371,7 @@ static bool synaptics_i2c_get_input(struct synaptics_i2c *touch) > static void synaptics_i2c_reschedule_work(struct synaptics_i2c *touch, > unsigned long delay) > { > - unsigned long flags; > - > - spin_lock_irqsave(&touch->lock, flags); > - > mod_delayed_work(system_wq, &touch->dwork, delay); > - > - spin_unlock_irqrestore(&touch->lock, flags); > }
This makes synaptics_i2c_reschedule_work() a useless wrapper for mod_delayed_work(). Can we get rid of it?
Thanks.
-- Dmitry
| |