On Mon, Jun 25, 2012 at 12:33:58PM +0200, Marc Kleine-Budde wrote: > On 06/25/2012 08:56 AM, Richard Zhao wrote: > > This patch supports only the host-mode functionality so far. > > > > Signed-off-by: Richard Zhao <richard.zhao@xxxxxxxxxxxxx> > > Signed-off-by: Marek Vasut <marex@xxxxxxx> > > Cc: Peter Chen <peter.chen@xxxxxxxxxxxxx> > > Cc: Alexander Shishkin <alexander.shishkin@xxxxxxxxxxxxxxx> > > Cc: Felipe Balbi <balbi@xxxxxx> > > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > > Tested-by: Subodh Nijsure <snijsure@xxxxxxxxxxxx> > > --- > > .../devicetree/bindings/usb/ci13xxx-imx.txt | 20 ++ > > drivers/usb/chipidea/Makefile | 3 + > > drivers/usb/chipidea/ci13xxx_imx.c | 197 ++++++++++++++++++++ > > 3 files changed, 220 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/usb/ci13xxx-imx.txt > > create mode 100644 drivers/usb/chipidea/ci13xxx_imx.c > > > > diff --git a/Documentation/devicetree/bindings/usb/ci13xxx-imx.txt b/Documentation/devicetree/bindings/usb/ci13xxx-imx.txt > > new file mode 100644 > > index 0000000..8bcd071 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/usb/ci13xxx-imx.txt > > @@ -0,0 +1,20 @@ > > +* Freescale i.MX ci13xxx usb controllers > > + > > +Required properties: > > +- compatible: Should be "fsl,imx27-usb" > > +- reg: Should contain registers location and length > > +- interrupts: Should contain controller interrupt > > + > > +Optional properties: > > +- fsl,usbphy: phandler of usb phy that connects to the only one port > > > +- fsl,hub-reset-gpios: gpio used to reset on-board usb hub > > +- fsl,vbus-power-gpios: gpio used to set vbus power of the only one port > > These two properties are not implemented, are they? > vbus is handled via a fixed regulator. I forgot to update the doc. > > Further I'm looking into the phy and otg implementation. I think the > handling of the regulator should go into mxs-phy.c, not into the > ci13xxx_imx.c Normally, we don't put it in phy driver. For example, most host driver don't call phy drivers. Thanks Richard > > Marc > -- > Pengutronix e.K. | Marc Kleine-Budde | > Industrial Linux Solutions | Phone: +49-231-2826-924 | > Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | > Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | > > > -- 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