On Wed, Jun 10, 2009 at 07:17:48PM +0200, Valentin Longchamp wrote: >> +static inline int usb_xcvr_set_vbus(struct usb_xcvr *xcvr, bool en) >> +{ >> + if (xcvr->set_vbus) >> + return xcvr->set_vbus(xcvr, en); >> + >> + if (xcvr->driver && xcvr->driver->set_vbus) >> + xcvr->driver->set_vbus(xcvr, en); > > I think that this should be > return xcvr->driver->set_vbus(xcvr, en); > Otherwise you return -EINVAL Erm. Sure. Thanks for noticing. Daniel -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html