From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755407AbcBGVid (ORCPT ); Sun, 7 Feb 2016 16:38:33 -0500 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:46292 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755220AbcBGVhX (ORCPT ); Sun, 7 Feb 2016 16:37:23 -0500 Date: Sun, 7 Feb 2016 22:37:20 +0100 From: Pavel Machek To: Tony Lindgren Cc: pali.rohar@gmail.com, sre@kernel.org, kernel list , linux-arm-kernel , linux-omap@vger.kernel.org, khilman@kernel.org, aaro.koskinen@iki.fi, ivo.g.dimitrov.75@gmail.com, patrikbachan@gmail.com, serge@hallyn.com Subject: Re: N900 sleep mode (in 4.5-rc0, if that matters) Message-ID: <20160207213720.GB6147@amd> References: <20160125163332.GT19432@atomide.com> <20160126140010.GA29723@amd> <20160126172527.GY3500@atomide.com> <20160130200244.GA30429@amd> <20160130201452.GC30429@amd> <20160130221509.GA28644@amd> <20160201181316.GM19432@atomide.com> <20160201211736.GA11283@amd> <20160201221111.GP19432@atomide.com> <20160204053518.GP19432@atomide.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160204053518.GP19432@atomide.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi! > > > No, sorry, that was with 4.4. As you hit "PM regression with commit > > > 5de85b9d57ab PM runtime re-init in v4.5-rc1", I thought I'd avoid > > > v4.5-rc. > > > > OK > > > > > (I assume I have to insmod and rmmod, right? Because powersave is not > > > entered if I simply compile-out usb). > > > > Depending on what the bootloader does and probably also if > > USB was used during the booting.. So yeah you may need to modprobe > > and then rmmod. > > > > > Would you have commit ids for those rmmod fixes? It might be good to > > > push them into stable, and I should try again with them applied... > > > > 055555fc459 ("usb: musb: core: Fix handling of the phy notifications") > > 03e43528ab68 ("usb: musb: Fix unbalanced pm_runtime_enable") > > Oh and looks like these two phy-twl4030-phy fixes never got > merged from thread "[PATCH 0/2] Two phy-twl4030-usb fixes for > unloading the module": > > phy: twl4030-usb: Relase usb phy on unload > phy: twl4030-usb: Fix unbalanced pm_runtime_enable on module reload > > I'll resend those. But without those again deeper idle states > are blocked.. And it could be that n900 needs similar additional > patches for it's USB PHY as I've tested things so far only with > phy-twl4030-usb PHY based systems. I don't get it. I was using +CONFIG_USB_MUSB_OMAP2PLUS=m +CONFIG_TWL4030_USB=m on the n900. Are you suggesting I should be using something else on the N900? If so, what? Pali, do you have deeper sleep states working on 4.4 kernel? (Or anything 4.X?) Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html