On Tue, 2006-08-29 at 14:15 -0700, Sukadev Bhattiprolu wrote: > @@ -1004,15 +1002,16 @@ int saa7134_tvaudio_init2(struct saa7134 > break; > } > > - dev->thread.pid = -1; > + dev->thread.task = NULL; > if (my_thread) { ... This is _really_ minor, but I think dev is kzmalloc()'d. I haven't examined it closely enough to tell if these devices get reused, but this one _might_ be unnecessary. Certainly no big deal either way, and it certainly doesn't make anything worse. -- Dave