On Thu, Sep 28, 2006 at 12:09:59AM -0700, Andrew Morton wrote:
> On Thu, 28 Sep 2006 08:49:52 +0200
> Olaf Hering <[EMAIL PROTECTED]> wrote:
> 
> > On Wed, Sep 27, Greg KH wrote:
> > 
> > > @@ -372,6 +379,15 @@ struct usb_device {
> > >  
> > >   int maxchild;                   /* Number of ports if hub */
> > >   struct usb_device *children[USB_MAXCHILDREN];
> > > +
> > > +#ifdef CONFIG_PM
> > > + struct work_struct autosuspend; /* for delayed autosuspends */
> > > + struct mutex pm_mutex;          /* protects PM operations */
> > > + int pm_usage_cnt;               /* usage counter for autosuspend */
> > > +
> > > + unsigned auto_pm:1;             /* autosuspend/resume in progress */
> > > + unsigned do_remote_wakeup:1;    /* remote wakeup should be enabled */
> > > +#endif
> > 
> > This fails for CONFIG_PM=n. 
> > Why did -mm not catch this?
> 
> -mm did catch it, and -mm reported it loudly.  I had a nasty hack in there
> (fix-gregkh-usb-usbcore-add-autosuspend-autoresume-infrastructure.patch) to
> fix things up, but it got forgotten about.
> 
> Alan came up with an alternate patch this week (and then another one) which 
> I've
> queued, but Greg hasn't picked it up yet.

Just did... am catching up on things right now...

thanks,

greg k-h

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to