On 10/12/07, Bryan Wu [off-list ref] wrote:

On Fri, 2007-10-12 at 10:49 -0400, Dmitry Torokhov wrote:

quoted

Hi Bryan,

On 10/12/07, Bryan Wu [off-list ref] wrote:

quoted

+

[snip]

quoted

quoted

+
+static void ad7142_close(struct input_dev *dev)
+{
+       free_irq(CONFIG_BFIN_JOYSTICK_IRQ_PFX, ad7142_interrupt);
+       kthread_stop(ad7142_task);

Don't you need to write something over i2c to shut the devoce off?
What stops it from continuing to generate interrupts?

Actually, I am going to use "completion" to replace the whole
wait_interrupt_xxx and intr_flag things which original from Aubrey. How
do you think of that?

I don't think it is a very good idea - for me completion is one-time
deal. You use it and then you are done. How about firing a work from
interrupt and either rely on the default workqueue (keventd) or create
your own to execute it?

Thank you.

-- 
Dmitry