On Fri, 11 Jan 2013, Roger Quadros wrote:

> > --- usb-3.7.orig/drivers/usb/host/ehci.h
> > +++ usb-3.7/drivers/usb/host/ehci.h
> > @@ -221,6 +221,8 @@ struct ehci_hcd {                       /* one per 
> > controlle
> >  #ifdef DEBUG
> >     struct dentry           *debug_dir;
> >  #endif
> > +
> > +   unsigned long           priv[0];        /* platform-specific data */
> 
> Why can't it be void * ?

It can.  I don't see that it makes any difference one way or the other, 
and I don't see any strong reason for changing it to void *.

> Also we need to put a comment there stating that priv must be the last
> member in this structure, else it won't work.

Good point.  I'll update the comment.

Alan Stern

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to