On Wed, Feb 04, 2015 at 05:04:18AM -0800, Greg Kroah-Hartman wrote:
> On Wed, Feb 04, 2015 at 10:35:22AM +0100, Maxime Ripard wrote:
> > Hi Mathias, Greg,
> > 
> > On Mon, Jan 19, 2015 at 02:01:12PM +0100, Maxime Ripard wrote:
> > > The commit 973747928514 ("usb: host: xhci-plat: add support for the Armada
> > > 375/38x XHCI controllers") extended the xhci-plat driver to support the 
> > > Armada
> > > 375/38x SoCs, mostly by adding a quirk configuring the MBUS window.
> > > 
> > > However, that quirk was run before the clock the controllers needs has 
> > > been
> > > enabled. This usually worked because the clock was first enabled by the
> > > bootloader, and left as such until the driver is probe, where it tries to
> > > access the MBUS configuration registers before enabling the clock.
> > > 
> > > Things get messy when EPROBE_DEFER is involved during the probe, since as 
> > > part
> > > of its error path, the driver will rightfully disable the clock. When the
> > > driver will be reprobed, it will retry to access the MBUS registers, but 
> > > this
> > > time with the clock disabled, which hangs forever.
> > > 
> > > Fix this by running the quirks after the clock has been enabled by the 
> > > driver.
> > > 
> > > Signed-off-by: Maxime Ripard <maxime.rip...@free-electrons.com>
> > > Cc: <sta...@vger.kernel.org> # v3.16+
> > 
> > Any chance for this to go in 3.19?
> 
> For 3.19?  It's too late, I'm not sending anything else to Linus as 3.19
> will be out in a few days.

Ok. I'll resend it when 3.20-rc1 is out then.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Attachment: signature.asc
Description: Digital signature

Reply via email to