Re: cx23885-input.c does in fact use a workqueue....

2011-02-15 Thread Tejun Heo
Hello, On Mon, Feb 14, 2011 at 07:54:36PM -0500, Andy Walls wrote: > > 1. Just flush the work items explicitly using flush_work_sync(). > > That will do for now. > > > 2. Create a dedicated workqueue to serve as flushing domain. > > I have gotten reports of the IR Rx FIFO overflows for the CX23

Re: cx23885-input.c does in fact use a workqueue....

2011-02-14 Thread Andy Walls
On Mon, 2011-02-14 at 12:03 +0100, Tejun Heo wrote: > Hello, > > On Sun, Feb 13, 2011 at 08:33:55PM -0800, Dmitry Torokhov wrote: > > > The cx23885 driver does in fact schedule work for IR input handling: > > > > > > Here's where it is scheduled for CX23888 chips: > > > > > > http://git.linuxtv.

Re: cx23885-input.c does in fact use a workqueue....

2011-02-14 Thread Tejun Heo
Hello, On Sun, Feb 13, 2011 at 08:33:55PM -0800, Dmitry Torokhov wrote: > > The cx23885 driver does in fact schedule work for IR input handling: > > > > Here's where it is scheduled for CX23888 chips: > > > > http://git.linuxtv.org/media_tree.git?a=blob;f=drivers/media/video/cx23885/cx23885-ir.c

Re: cx23885-input.c does in fact use a workqueue....

2011-02-13 Thread Dmitry Torokhov
On Sun, Feb 13, 2011 at 08:35:22PM -0500, Andy Walls wrote: > Tejun, > > I just noticed this commit: > > commit 8c71778cbf2c8beaefaa2dee5478aa0622d96682 > Author: Tejun Heo > Date: Fri Dec 24 16:14:20 2010 +0100 > > media/video: don't use flush_scheduled_work() > > This patch con

cx23885-input.c does in fact use a workqueue....

2011-02-13 Thread Andy Walls
Tejun, I just noticed this commit: commit 8c71778cbf2c8beaefaa2dee5478aa0622d96682 Author: Tejun Heo Date: Fri Dec 24 16:14:20 2010 +0100 media/video: don't use flush_scheduled_work() This patch converts the remaining users of flush_scheduled_work() in media/video. *