From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: Suspend broken on 3.3? Date: Fri, 06 Apr 2012 07:21:28 -0700 Message-ID: <87iphdeyaf.fsf@ti.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from na3sys009aog102.obsmtp.com ([74.125.149.69]:34700 "EHLO na3sys009aog102.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753916Ab2DFOVa (ORCPT ); Fri, 6 Apr 2012 10:21:30 -0400 Received: by pbcup15 with SMTP id up15so3184718pbc.14 for ; Fri, 06 Apr 2012 07:21:29 -0700 (PDT) In-Reply-To: (Paul Walmsley's message of "Thu, 5 Apr 2012 18:29:42 -0600 (MDT)") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Paul Walmsley Cc: "Raja, Govindraj" , Joe Woodward , linux-omap@vger.kernel.org, Felipe Balbi , neilb@suse.de Paul Walmsley writes: > On Wed, 4 Apr 2012, Raja, Govindraj wrote: > >> On Wed, Apr 4, 2012 at 8:26 PM, Paul Walmsley wrote: >> > On Mon, 2 Apr 2012, Raja, Govindraj wrote: >> > >> >> As of now what I can think of is to update qos reqest to prevent MPU >> >> from transitioning in case of port activity if wakeup capability for port >> >> is disabled. >> > >> > Haven't been following this thread closely, but this doesn't seem right. >> > Why would changing the UART driver's ability to wake from suspend via the >> > sysfs file prevent the driver from using hardware wakeups to wake from >> > dynamic idle? >> >> >> IIUC wakeup capability is needed in suspend path or in cpu idle path. >> >> once uart wakeup capability is disabled from sysfs the Module level >> wakeup from PM_WKEN1 reg on omap3 and pad wakeup from pin mux data given >> will be disabled.. > > As far as I know, that sysfs wakeup file is not meant to disable > all module-level wakeup. Kevin can correct me if I'm wrong, but I think > it's only supposed to control wakeup from suspend. In theory, sysfs control is meant for static suspend. ISTR though that we were using it for idle as well so there weren't unncessary UART wakeups from idle on UART activity that was not serial console. Kevin