Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755435Ab3H1Vvp (ORCPT ); Wed, 28 Aug 2013 17:51:45 -0400 Received: from mga02.intel.com ([134.134.136.20]:58909 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753464Ab3H1Vvn (ORCPT ); Wed, 28 Aug 2013 17:51:43 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.89,978,1367996400"; d="scan'208";a="369976612" Date: Wed, 28 Aug 2013 14:51:42 -0700 From: Sarah Sharp To: Paul Zimmerman 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 Message-ID: <20130828215142.GJ26483@xanatos> References: <1377040909-18428-1-git-send-email-jwerner@chromium.org> <20130821184042.GC3577@xanatos> <20130822004514.GB19747@xanatos> <20130826191411.GG5112@xanatos> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3282 Lines: 71 On Mon, Aug 26, 2013 at 07:37:56PM +0000, Paul Zimmerman wrote: > > 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 the 2.41a has BESL support, but may not set the BLC flag. What happens if we use the HIRD encoding instead? Will things break? It seems like we would need to disable USB 2.0 LPM on that host all together, if it expects BESL encoding, but advertises HIRD encoding. > So it may be safer to say that the feature is present starting with 2.50a. Is there a way we can tell the difference between a 2.41a xHCI host and a 2.50a host? If so, we can add a quirk to disable LPM on the 2.41a host. > 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. Can you please test Julius' patch on the 2.41a, 2.50a, and 2.51a hosts? Please test against usb-next, since that includes a fix for the BESL patches. I'm not going to merge this unless we know it's not going to break those earlier host controllers. Besides, at this point, it's too late to include this patch in 3.12. I already got bitten yesterday by trying to push patches this late, and I'm not doing that again. Sarah Sharp -- 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/