| Message ID | 20160707222114.1673-7-stephen.boyd@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
Quoting Peter Chen (2016-07-08 02:25:35) > On Thu, Jul 07, 2016 at 03:20:57PM -0700, Stephen Boyd wrote: > > The ULPI phy on qcom platforms needs to be initialized and > > powered on after a USB reset and before we toggle the run/stop > > bit. Otherwise, the phy locks up and doesn't work properly. > > Therefore, add a flag to skip any phy power management in the > > core layer, leaving it up to the glue driver to manage. > > > > Cc: Peter Chen <peter.chen@nxp.com> > > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > Signed-off-by: Stephen Boyd <stephen.boyd@linaro.org> > > --- > > drivers/usb/chipidea/core.c | 6 ++++++ > > include/linux/usb/chipidea.h | 1 + > > 2 files changed, 7 insertions(+) > > > > diff --git a/drivers/usb/chipidea/core.c b/drivers/usb/chipidea/core.c > > index 01390e02ee53..532085a096d9 100644 > > --- a/drivers/usb/chipidea/core.c > > +++ b/drivers/usb/chipidea/core.c > > @@ -361,6 +361,9 @@ static int _ci_usb_phy_init(struct ci_hdrc *ci) > > */ > > static void ci_usb_phy_exit(struct ci_hdrc *ci) > > { > > + if (ci->platdata->flags & CI_HDRC_OVERRIDE_PHY_CONTROL) > > + return; > > + > > if (ci->phy) { > > phy_power_off(ci->phy); > > phy_exit(ci->phy); > > @@ -379,6 +382,9 @@ static int ci_usb_phy_init(struct ci_hdrc *ci) > > { > > int ret; > > > > + if (ci->platdata->flags & CI_HDRC_OVERRIDE_PHY_CONTROL) > > + return 0; > > + > > How you handle the code for PHY getting at probe? > Which probe? Glue layer probe? In my case I'm not using the phy until the RESET/STOPPED event so I just rely on the chipidea core to get the phy and then control the phy manually in the glue layer. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Quoting Peter Chen (2016-08-06 00:59:25) > On Fri, Aug 05, 2016 at 02:46:00PM -0700, Stephen Boyd wrote: > > Quoting Peter Chen (2016-07-08 02:25:35) > > > > > > How you handle the code for PHY getting at probe? > > > > > > > Which probe? Glue layer probe? In my case I'm not using the phy until > > the RESET/STOPPED event so I just rely on the chipidea core to get the > > phy and then control the phy manually in the glue layer. > > I have no idea about it after reading your patch 10/22. > Sorry, you lost me now. What is the question? _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
diff --git a/drivers/usb/chipidea/core.c b/drivers/usb/chipidea/core.c index 01390e02ee53..532085a096d9 100644 --- a/drivers/usb/chipidea/core.c +++ b/drivers/usb/chipidea/core.c @@ -361,6 +361,9 @@ static int _ci_usb_phy_init(struct ci_hdrc *ci) */ static void ci_usb_phy_exit(struct ci_hdrc *ci) { + if (ci->platdata->flags & CI_HDRC_OVERRIDE_PHY_CONTROL) + return; + if (ci->phy) { phy_power_off(ci->phy); phy_exit(ci->phy); @@ -379,6 +382,9 @@ static int ci_usb_phy_init(struct ci_hdrc *ci) { int ret; + if (ci->platdata->flags & CI_HDRC_OVERRIDE_PHY_CONTROL) + return 0; + switch (ci->platdata->phy_mode) { case USBPHY_INTERFACE_MODE_UTMI: case USBPHY_INTERFACE_MODE_UTMIW: diff --git a/include/linux/usb/chipidea.h b/include/linux/usb/chipidea.h index f9be467d6695..d07b162073f7 100644 --- a/include/linux/usb/chipidea.h +++ b/include/linux/usb/chipidea.h @@ -57,6 +57,7 @@ struct ci_hdrc_platform_data { #define CI_HDRC_OVERRIDE_AHB_BURST BIT(9) #define CI_HDRC_OVERRIDE_TX_BURST BIT(10) #define CI_HDRC_OVERRIDE_RX_BURST BIT(11) +#define CI_HDRC_OVERRIDE_PHY_CONTROL BIT(12) /* Glue layer manages phy */ enum usb_dr_mode dr_mode; #define CI_HDRC_CONTROLLER_RESET_EVENT 0 #define CI_HDRC_CONTROLLER_STOPPED_EVENT 1
The ULPI phy on qcom platforms needs to be initialized and powered on after a USB reset and before we toggle the run/stop bit. Otherwise, the phy locks up and doesn't work properly. Therefore, add a flag to skip any phy power management in the core layer, leaving it up to the glue driver to manage. Cc: Peter Chen <peter.chen@nxp.com> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Stephen Boyd <stephen.boyd@linaro.org> --- drivers/usb/chipidea/core.c | 6 ++++++ include/linux/usb/chipidea.h | 1 + 2 files changed, 7 insertions(+) -- 2.9.0.rc2.8.ga28705d _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel