RE: [PATCH v3 1/3] usb: udc: add usb_gadget_find_udc

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



 
> >
> > > > > An idea just struck me...  Instead of looping through all the
> > > > > udc's to find the right one, why not simply store a pointer to
> > > > > the udc in struct usb_gadget?
> > > > >
> > >
> > > We still have code to find usb_gadget_driver to iterate udc_list, it
> > > is better to change all, we need to have solution to consolidate
> > > struct usb_udc, struct usb_gadget, and struct usb_gadget_drver.
> > > I have no good solution now.
> >
> > I think it's okay to iterate through the list to find a
> > usb_gadget_driver.  But it's silly to iterate to find the usb_udc
> > corresponding to a usb_gadget.
> 
> Felipe, what's your opinion?
> 

Hi Felipe, would you have comments for it?

- Follow Alan's suggestion
- Adding a new API like this patch
- Drop this patch, and still iterate udc_list to get usb_gadget

Peter
 
--
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




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux