RE: DWC2 and/or S3C-HSOTG for STA2X11 board
From: Paul Zimmerman
Date: Tue Jul 16 2013 - 14:36:27 EST
> From: Federico Vaga [mailto:federico.vaga@xxxxxxxxx]
> Sent: Tuesday, July 16, 2013 5:02 AM
>
> I have an x86 board made by STMicroelectronics (STA2X11) with the Synopsis
> USB-OTG DesignWare 2 on it and connected through the PCI-e bus.
>
> I know that there are two drivers for the same controller:
>
> (host) drivers/staging/dwc2/*
> (device) drivers/usb/gadget/s3c-hsotg.{c|h}
>
> So, at the moment I cannot have a board with both host/device working at the
> same time. I have to choose to use the block as device or host, right?
>
> I know that the plan is to merge the s3c-hsotg in the dwc2 driver
> (https://lwn.net/Articles/540283/). Are still accepted patch to s3c-hsotg? Or
> it is work in progress right now (soon), so it is better to wait after the
> merge?
>
> In order to use the s3c-hsotg I must implement a PCI wrapper that uses this
> driver. It will be accepted in the kernel even if it will be removed sooner or
> later because of the driver merge?
Hi Federico,
As part of the merge, we will need to develop a PCI wrapper for
s3c-hsotg anyway, so I think it would not be wasted effort.
Actually, as a POC I already did this as a quick hack, just to
make sure that the driver will work on our PCIe prototyping
platform (it does).
As Felipe says, currently s3c-hsotg does have too much knowledge
of Samsung platform. But it should be fairly easy to move that
knowledge from the core code to a platform-device wrapper,
similar to platform.c in the dwc2 driver. So if you would like
to work on that (creating a PCI wrapper and a platform wrapper)
I think it would be useful.
If you want, I can send you my hacked-up code for the PCI
version of the driver, to use as a starting point.
--
Paul
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/