Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752196Ab3HZTiE (ORCPT ); Mon, 26 Aug 2013 15:38:04 -0400 Received: from us02smtp2.synopsys.com ([198.182.60.77]:33337 "EHLO alvesta.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751820Ab3HZTiC convert rfc822-to-8bit (ORCPT ); Mon, 26 Aug 2013 15:38:02 -0400 From: Paul Zimmerman To: Sarah Sharp CC: Julius Werner , LKML , "linux-usb@vger.kernel.org" , Greg Kroah-Hartman , Vincent Palatin , "Benson Leung" , Felipe Balbi , "mathias.nyman@linux.intel.com" Subject: RE: [PATCH] usb: xhci-plat: Enable USB 2.0 hardware LPM support for platform xHCs Thread-Topic: [PATCH] usb: xhci-plat: Enable USB 2.0 hardware LPM support for platform xHCs Thread-Index: AQHOnfwYI9Np33Plqkq7vUNbfD4u75mgdUEAgAAzMICAADKpAIAAPKeA//+VVhCAB60tgP//i2hQ Date: Mon, 26 Aug 2013 19:37:56 +0000 Message-ID: References: <1377040909-18428-1-git-send-email-jwerner@chromium.org> <20130821184042.GC3577@xanatos> <20130822004514.GB19747@xanatos> <20130826191411.GG5112@xanatos> In-Reply-To: <20130826191411.GG5112@xanatos> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.9.64.240] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2291 Lines: 54 > From: Sarah Sharp [mailto:sarah.a.sharp@linux.intel.com] > 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/