From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755599AbcHBBbl (ORCPT ); Mon, 1 Aug 2016 21:31:41 -0400 Received: from mail-pf0-f194.google.com ([209.85.192.194]:34795 "EHLO mail-pf0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755543AbcHBBbb (ORCPT ); Mon, 1 Aug 2016 21:31:31 -0400 Date: Tue, 2 Aug 2016 09:08:13 +0800 From: Peter Chen To: Joshua Clayton Cc: Fabio Estevam , Peter Chen , Greg Kroah-Hartman , Alan Stern , Ulf Hansson , Mark Brown , Sebastian Reichel , "robh+dt@kernel.org" , Shawn Guo , Dmitry Eremin-Solenikov , David Woodhouse , Krzysztof =?utf-8?Q?Koz=C5=82owski?= , "linux-arm-kernel@lists.infradead.org" , Philipp Zabel , "devicetree@vger.kernel.org" , Pawel Moll , Mark Rutland , USB list , Arnd Bergmann , Sascha Hauer , "Maciej S. Szmigiero" , Troy Kisky , Oscar Curero , stephen.boyd@linaro.org, "linux-pm@vger.kernel.org" , linux-kernel , Fabio Estevam Subject: Re: [PATCH v3 0/6] power: add power sequence library Message-ID: <20160802010812.GA26831@shlinux2> References: <1469007629-31757-1-git-send-email-peter.chen@nxp.com> <579A2B38.3000202@gmail.com> <579F62CE.1050900@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <579F62CE.1050900@gmail.com> 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 On Mon, Aug 01, 2016 at 07:55:10AM -0700, Joshua Clayton wrote: > > > On 07/28/2016 09:41 AM, Fabio Estevam wrote: > > Hi Joshua, > > > > On Thu, Jul 28, 2016 at 12:56 PM, Joshua Clayton > > wrote: > > > >> I assume there is a v4 coming due to rmk's comments on patch 5. > >> I couldn't figure out where to null the of_node in error paths, but in testing > >> we did put a line of code to null the of_node on release. > >> > >> but... > >> As an aside, > >> I was hoping this series would magically fix a problem > >> with the imx6q-evi which has forced us to disable > >> runtime power management. But it did not. :( > > I also see a similar problem on a mx28 board with a hub. > > > > Does it help in your case if you pass 'usbcore.autosuspend=-1' in the > > kernel command line? > > > > Regards, > > > > Fabio Estevam > Thanks a million, Fabio! > > 'usbcore.autosuspend=-1' quiets the errors. > > ~joshua > If you are using imx6q, there are some patches which is not upstreamed to workaround one SoC problem. http://www.spinics.net/lists/linux-usb/msg98682.html Try to implement the design for notify_suspend/resume at drivers/usb/phy/phy-mxs-usb.c and drivers/usb/chipidea/host.c You can refer to the code at: http://git.freescale.com/git/cgit.cgi/imx/linux-2.6-imx.git/tree/?h=imx_4.1.15_1.0.0_ga -- Best Regards, Peter Chen