[U-Boot] DM for ci_udc

[Lukasz Majewski]

Hi Sven, Marek, Jean-Jacques,

> On 5/6/19 12:12 PM, Sven Schwermer wrote:
> > Hi,
> > 
> > I’m running into linking issues when enabling CONFIG_DM_USB_GADGET
> > with CONFIG_CI_UDC because dm_usb_gadget_handle_interrupts is not
> > available. That made me look into what’s required to port ci_udc to
> > driver model. I have a few conceptual questions about this:
> > 
> > 1. Some other USB gadget drivers (dwc3-generic, ti-musb) seem to
> > create a wrapper driver that binds either the gadget or the
> > host-mode driver to the USB device, depending on the configured
> > mode. Is this the desired approach?

I think that we shall follow the already in-tree pattern.
Jean-Jacques Hiblot (CC'ed) did a great job on porting / developing
this code (moreover there is a u-boot command - bind/unbind which allows
binding usb devices)

> > 
> > 2. I’m working with a i.MX6 (7) device so the host-mode driver is
> > ehci-mx6, but I’ve seen that ci_udc is also used with other
> > host-mode drivers. How would a wrapper work in this case where
> > there are multiple possible host-mode drivers but one gadget driver?
> > 

As fair as I understood the "bind/unbind" command would allow you to do
it [1]. Jean-Jacques am I right here ?

> > 3. Any tips and hints are welcome :)  

I can offer myself as a tester for imx6q (as I do have a few of them
queued for the DM/DTS conversion).

> 
> +CC Lukasz, UDC is his stuff.
> 

Note:
[1] - https://lists.denx.de/pipermail/u-boot/2018-June/333105.html


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190506/5717df60/attachment.sig>