All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: linux-media@vger.kernel.org, linux-usb@vger.kernel.org,
	Wolfram Sang <wsa@the-dreams.de>
Subject: Re: [PATCH/RFC] uvcvideo: Disable USB autosuspend for Creative Live! Cam Optia AF
Date: Thu, 04 Apr 2013 17:28:01 +0200	[thread overview]
Message-ID: <1675297.SBVpg0UeLp@avalon> (raw)
In-Reply-To: <Pine.LNX.4.44L0.1303281043140.1652-100000@iolanthe.rowland.org>

Hi Alan,

On Thursday 28 March 2013 10:45:27 Alan Stern wrote:
> On Thu, 28 Mar 2013, Laurent Pinchart wrote:
> > The camera fails to start video streaming after having been autosuspend.
> > Add a new quirk to selectively disable autosuspend for devices that
> > don't support it.
> > 
> > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > ---
> > 
> >  drivers/media/usb/uvc/uvc_driver.c | 14 +++++++++++++-
> >  drivers/media/usb/uvc/uvcvideo.h   |  1 +
> >  2 files changed, 14 insertions(+), 1 deletion(-)
> > 
> > I've tried to set the reset resume quirk for this device in the USB core
> > but the camera still failed to start video streaming after having been
> > autosuspended. Regardless of whether the reset resume quirk was set, it
> > would respond to control messages but wouldn't send video data.
> 
> Presumably the camera won't work after a system suspend, either.

That was my expectation as well, but the device has survived system suspend 
without being reenumerated. I don't know if the USB port power got cut off 
during system suspend.

> > This solution below is a hack, but I'm not sure what else I can try. Crazy
> > ideas are welcome.
> 
> It's not a hack; it's a normal use for a quirk flag.  Of course, if you
> can figure out what's wrong with the camera and see how to fix it, that
> would be best.

I've tried to but I can't figure out what goes wrong exactly.

> How does the camera perform on a Windows system after being put to
> sleep and then woken up?

I don't know, I have no Windows system to test the camera on.

-- 
Regards,

Laurent Pinchart


      reply	other threads:[~2013-04-04 15:27 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-28 11:53 [PATCH/RFC] uvcvideo: Disable USB autosuspend for Creative Live! Cam Optia AF Laurent Pinchart
2013-03-28 14:45 ` Alan Stern
2013-04-04 15:28   ` Laurent Pinchart [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1675297.SBVpg0UeLp@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=stern@rowland.harvard.edu \
    --cc=wsa@the-dreams.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.