This was found with a static checker and has not been tested, but it seems pretty clear that the mutex_lock() was supposed to be mutex_unlock() This is a 2.6.32 candidate. Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> --- orig/drivers/media/video/ov511.c 2009-11-29 14:44:46.000000000 +0200 +++ devel/drivers/media/video/ov511.c 2009-11-29 14:44:57.000000000 +0200 @@ -5878,7 +5878,7 @@ ov51x_probe(struct usb_interface *intf, goto error; } - mutex_lock(&ov->lock); + mutex_unlock(&ov->lock); return 0; -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html