Messages in this thread | | | Date | Wed, 14 Mar 2012 08:48:28 -0700 | From | Dmitry Torokhov <> | Subject | Re: [PATCH v3 2/2] input: add driver support for MAX8997-haptic |
| |
On Wed, Mar 14, 2012 at 11:47:42PM +0900, Chanwoo Choi wrote: > Hi Dmitry, > > > This is not proper locking between playing effect and suspend. Now that > > you are using workqueue you need to synchronize access with it, not with > > play_effect method. > > > > Does the following patch work for you? > > > > Your patch is worked on TRATS board based on EXYNOS4210 but, you should > apply following patch because 'chip->enabled' is used on > max8997_haptic_configure() function to enable/disable haptic motor.
OK, excellent. I'll pick up this fix and apply the driver for 3.4 queue, thanks!
-- Dmitry
| |