From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from muru.com ([72.249.23.125]:34242 "EHLO muru.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750857AbcBLWkV (ORCPT ); Fri, 12 Feb 2016 17:40:21 -0500 Date: Fri, 12 Feb 2016 14:40:19 -0800 From: Tony Lindgren To: Javier Martinez Canillas Cc: Wolfram Sang , linux-i2c@vger.kernel.org, Mauro Carvalho Chehab , Linux Media Mailing List , linux-pm@vger.kernel.org, Alan Stern Subject: Re: tvp5150 regression after commit 9f924169c035 Message-ID: <20160212224018.GZ3500@atomide.com> References: <56B204CB.60602@osg.samsung.com> <20160208105417.GD2220@tetsubishi> <56BE57FC.3020407@osg.samsung.com> <20160212221352.GY3500@atomide.com> <56BE5C97.9070607@osg.samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <56BE5C97.9070607@osg.samsung.com> Sender: linux-media-owner@vger.kernel.org List-ID: * Javier Martinez Canillas [160212 14:29]: > On 02/12/2016 07:13 PM, Tony Lindgren wrote: > >Hmm yeah I wonder if this canned solution helps here too: > > > >1. Check if the driver(s) are using pm_runtime_use_autosuspend() > > > > By driver do you mean the OMAP GPIO driver or the tvp5150 I2C driver? > The latter does not have runtime PM support. Sounds like OMAP GPIO then. > >2. If so, you must use pm_runtime_dont_use_autosuspend() before > > pm_runtime_put_sync() to make sure that pm_runtime_put_sync() > > works. > > > >3. Or you can use pm_runtime_put_sync_suspend() instead of > > pm_runtime_put_sync() for sections of code where the clocks > > need to be stopped. > > > > I can check if the OMAP GPIO is following these and give a try but > don't have access to the board right now so I'll do it on Monday. It does not seem to be using pm_runtime_autosuspend(). Did you try reverting commit de85b9d57ab ("PM / runtime: Re-init runtime PM states at probe error and driver unbind") and see if that helps? If it does, then sounds like we may have some other regression as well. Regards, Tony