Hi, Oliver:

Quoting Oliver Neukum <[EMAIL PROTECTED]>:

> this patch removes sleep_on from usbvideo.
> I see no MAINTAINERS entry, thus it goes to the whole list.

Should be me, I guess... will add later.

> Two remarks: Using sleep_on just for a dely is not funny.

That was just a quick hack for deep debugging. I used a construct that was
known to work; it was I believe even suggested in the book (about 2.2
drivers). This procedure is not called from any production code.

> Checking a queue for activity is a bad idea.

Legacy code from cpia. The sleeping dogs again. It worked.

Good that it is now improved! I haven't seen this patch in Greg's patches
this morning, so I guess I have a chance to test it.

Dmitri

-- 
I question the practice of ending posts with a signature file in order
to thoughtlessly finish off the sorry things.

Attachment: msg03032/pgp00000.pgp
Description: PGP signature

Reply via email to