Re: [PATCH] usb gadget: don't save bind callback in struct usb_gadget_driver

["Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>]

[stripped down Cc: list to annoy fewer people]

On Mon, Aug 02, 2010 at 03:54:40PM -0700, Greg KH wrote:
> On Mon, Aug 02, 2010 at 03:12:48PM +0200, Uwe Kleine-König wrote:
> > On Mon, Aug 02, 2010 at 02:51:36PM +0200, Julia Lawall wrote:
> > > > diff --git a/drivers/usb/gadget/amd5536udc.c b/drivers/usb/gadget/amd5536udc.c
> > > > index 731150d..c266c1e 100644
> > > > --- a/drivers/usb/gadget/amd5536udc.c
> > > > +++ b/drivers/usb/gadget/amd5536udc.c
> > > > @@ -1954,13 +1954,14 @@ static int setup_ep0(struct udc *dev)
> > > >  }
> > > >  
> > > >  /* Called by gadget driver to register itself */
> > > > -int usb_gadget_register_driver(struct usb_gadget_driver *driver)
> > > > +int usb_gadget_probe_driver(struct usb_gadget_driver *driver,
> > > > +		int (*bind)(struct usb_gadget *))
> > > >  {
> > > >  	struct udc		*dev = udc;
> > > >  	int			retval;
> > > >  	u32 tmp;
> > > >  
> > > > -	if (!driver || !driver->bind || !driver->setup
> > > > +	if (!driver || bind || !driver->setup
> > > 
> > > I have the impression that this should be !bind rather than bind.  
> > ah, now I see what you meant.  Obviously you're right.  I fixed it up
> > locally here for now.  Greg, just tell me when/if you need a fixed
> > patch.
> 
> Didn't you already post a "fixed" patch?
Yes, I did:

	http://article.gmane.org/gmane.linux.ports.arm.kernel/86640
	http://article.gmane.org/gmane.linux.kernel/1017575

If you append '/raw' to the URL they should be applicable by
git/quilt/whatever you use.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |