From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH 2/2] Input: gpio_keys - replace timer and workqueue with delayed workqueue Date: Sat, 13 Dec 2014 11:09:26 -0800 Message-ID: <20141213190926.GD22702@dtor-ws> References: <20141208071930.GA20802@dtor-ws> <20141208072124.GA21031@dtor-ws> <1418236376.17201.57.camel@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1418236376.17201.57.camel@linux.intel.com> Sender: linux-input-owner@vger.kernel.org To: Andy Shevchenko Cc: linux-input@vger.kernel.org, Alexander Stein , Linus Walleij , Ezequiel Garcia , Alexander Shiyan , Laxman Dewangan , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: devicetree@vger.kernel.org Hi Andy, On Wed, Dec 10, 2014 at 08:32:56PM +0200, Andy Shevchenko wrote: > On Sun, 2014-12-07 at 23:21 -0800, Dmitry Torokhov wrote: > > We do not need to roll our own implementation of delayed work now that we > > have proper implementation of mod_delayed_work. > > > > For interrupt-only driven buttons we retain the timer, but we rename > > it to release_timer to better reflect its purpose. > > At least it doesn't break the driver on Intel Medfield device. > > Tested-by: Andy Shevchenko Thank you for testing it. Assume it is for both patches, right? -- Dmitry