Re: [PATCH 2/2 v3] USB: dwc3-exynos: Add vbus setup function to theexynos dwc3 glue layer
From: Felipe Balbi
Date: Mon Jul 23 2012 - 11:27:14 EST
On Tue, Jul 24, 2012 at 12:09:47AM +0900, Vivek Gautam wrote:
> > > @@ -58,6 +81,8 @@ static int __devinit dwc3_exynos_probe(struct
> > platform_device *pdev)
> > > if (!pdev->dev.coherent_dma_mask)
> > > pdev->dev.coherent_dma_mask = DMA_BIT_MASK(32);
> > >
> > > + dwc3_setup_vbus_gpio(pdev);
> > I think this should be handled by the gpio-vbus transceiver. No ?
> > I can understand that we can use gpio-vbus transceiver. But i have one
> confusion here.
> We have ehci as well as this dwc3 which need gpio to be setup to
> control vbus. So do we have two instances of this gpio-vbus driver to
> facilitate both ehci and dwc3, because both may be on different gpio
Sure... one for dwc3 and one for ehci.
> Moreover, the gpio line right now is parsed from the DT node of the
> device, so while assigning the 'gpio_vbus' field in
> 'gpio_vbus_mach_info', how do we provide that. Please suggest.
gpio-vbus doesn't have support for DT today, just add it ;-)
> One more suggestion please. Now that we are moving to DT we should be
> making changes for gpio-vbus transceiver also for device tree support.
yes, it needs support for DT before you can use it then.
Description: Digital signature