On Fri, Apr 25, 2014 at 02:07:35AM +0000, Peter Chen wrote: > > > > > On Wed, Apr 23, 2014 at 01:30:43PM +0800, Peter Chen wrote: > > > For internal PHY (like UTMI), the phy clock may from internal pll, it > > > is on/off on the fly, the access PORTSC.PTS will hang without phy > > > clock. So, the usb_phy_init which will open phy clock needs to be > > > called before hw_phymode_configure. > > > See: http://marc.info/?l=linux-arm-kernel&m=139350618732108&w=2 > > > > > > For external PHY (like ulpi), it needs to configure portsc.pts before > > > visit viewport, or the viewport can't be visited. so > > > phy_phymode_configure needs to be called before usb_phy_init. > > > See: cd0b42c2a6d2a74244f0053f8960f5dad5842278 > > > > > > It may not the best solution, but it can work for all situations. > > > > > > Cc: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> > > > Cc: Chris Ruehl <chris.ruehl@xxxxxxxxxxxx> > > > Cc: shc_work@xxxxxxx > > > Cc: denis@xxxxxxxxxx > > > Cc: festevam@xxxxxxxxx > > > Signed-off-by: Peter Chen <peter.chen@xxxxxxxxxxxxx> > > > --- > > > drivers/usb/chipidea/core.c | 37 ++++++++++++++++++++++++++++++++++- > > -- > > > 1 files changed, 34 insertions(+), 3 deletions(-) > > > > This should go into 3.14 as well, right? I've marked it for stable > > because of this. > > > > I have checked v3.14, it can't apply due to this patch is based on 3.15-rc1. > So, do not mark it for stable please. But it fixes a problem in 3.14, right? Anyway, it's too late, just ignore the "this does not apply" message you will get when it tries to go into the stable trees in a few weeks if you don't want it there. greg k-h -- 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
