Hi Matthew,

Thanks for the patch.

On Thursday 16 September 2010 20:00:04 Matthew Garrett wrote:
> We've been doing this for a while in Fedora without any complaints.

I'll trust you on that. If users start complaining I'll tell them who to blame 
;-)

> Signed-off-by: Matthew Garrett <m...@redhat.com>

Acked-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>

(although, if you need to resubmit the patch for any reason, I'd rather like 
the usb_enable_autosuspend call to be before the uvc_trace call).

> ---
>  drivers/media/video/uvc/uvc_driver.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/media/video/uvc/uvc_driver.c
> b/drivers/media/video/uvc/uvc_driver.c index 8bdd940..28ed5b4 100644
> --- a/drivers/media/video/uvc/uvc_driver.c
> +++ b/drivers/media/video/uvc/uvc_driver.c
> @@ -1814,6 +1814,7 @@ static int uvc_probe(struct usb_interface *intf,
>       }
> 
>       uvc_trace(UVC_TRACE_PROBE, "UVC device initialized.\n");
> +     usb_enable_autosuspend(udev);
>       return 0;
> 
>  error:

-- 
Regards,

Laurent Pinchart
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to