RE: [PATCH] usb: xhci-plat: Enable USB 2.0 hardware LPM support forplatform xHCs

From: Paul Zimmerman
Date: Mon Aug 26 2013 - 15:38:07 EST


> From: Sarah Sharp [mailto:sarah.a.sharp@xxxxxxxxxxxxxxx]
> Sent: Monday, August 26, 2013 12:14 PM
>
> On Thu, Aug 22, 2013 at 05:11:49AM +0000, Paul Zimmerman wrote:
> > > From: Julius Werner
> > > Sent: Wednesday, August 21, 2013 9:22 PM
> > >
> > > > You need the USB 2.0 spec errata from 2011-11 that describes the changes
> > > > made for BESL as well. It's in the USB2-LPM-Errata-final.pdf and
> > > > USB2_LinkPowerMangement_ECN[final].pdf files in this zip file:
> > > >
> > > > http://www.usb.org/developers/docs/usb_20_070113.zip
> > > >
> > > > I agree though, it's all a confusing mess for documentation on USB 2.0
> > > > Link PM.
> > >
> > > Thanks, I hadn't found that first one yet. Do you also have a link for
> > > the updated XHCI specification or a separate erratum document
> > > describing the PORTHLPMC register referenced in the BESL patches (they
> > > don't exist on DWC3, but I'm still curious)?
> >
> > Just to set the record straight :)
> >
> > The PORTHLPMC registers do exist on DWC3, starting with the 1.90a version
> > of the core or thereabouts. They only supported the HIRD flavor of LPM,
> > though. Only fairly recently has support for BESL been added, around
> > version 2.41a or so.
>
> And the 2.41a version that supports BESL properly sets the BLC flag in
> the USB 2.0 Protocol extended capabilities for all the USB 2.0 ports?
> Has this functionality been well-tested?

In 2.41a it is described as an "early adopter" feature in the databook,
and no mention is made of the BLC flag. So the answer there is "maybe".
Starting with 2.50a it is a full-fledged feature and the databook does
describe the BLC flag.

So it may be safer to say that the feature is present starting with 2.50a.

In 2.51a it has been well-tested in simulation. In actual hardware, it
has only been briefly tested in an ad-hoc sort of way, since none of the
standard drivers in the market support the feature yet, as far as we know.

Once the support is fully working in the Linux driver we can try testing
it there.

--
Paul

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/