From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tero Kristo Subject: Re: [PATCHv7 07/12] ARM: OMAP4: PM: put all domains to OSWR during suspend Date: Fri, 20 Jul 2012 11:37:06 +0300 Message-ID: <1342773426.4672.173.camel@sokoban> References: <1342704392-23657-1-git-send-email-t-kristo@ti.com> <1342704392-23657-8-git-send-email-t-kristo@ti.com> <1342711907.4672.153.camel@sokoban> Reply-To: Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from devils.ext.ti.com ([198.47.26.153]:38166 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751892Ab2GTIh1 (ORCPT ); Fri, 20 Jul 2012 04:37:27 -0400 In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Paul Walmsley Cc: linux-omap@vger.kernel.org, nm@ti.com, khilman@ti.com, rnayak@ti.com, linux-arm-kernel@lists.infradead.org On Thu, 2012-07-19 at 17:30 -0600, Paul Walmsley wrote: > Hi > > On Thu, 19 Jul 2012, Tero Kristo wrote: > > > Subject: [PATCHv7 07/12] ARM: OMAP4: PM: put all domains to OSWR during > > suspend > > > > Currently OMAP4 suspend puts all power domains to CSWR. OSWR is a deeper > > state that saves more power, but has higher latencies also. As suspend > > is considered a high-latency operation, OSWR is appropriate here. > > > > Signed-off-by: Tero Kristo > > --- > > > > I'll update this to next rev if one is requested. > > No need, I'll add it in the local copy here. Thanks, thats what I thought. :) > > > (Kind of hoping this set would be reaching maturity already.) > > It kind of looks to me like there are two or three separate sets within > the series. My feeling is that Kevin should take the first two, then I > should take the rest other than 6 and 7. Then once those are queued, > we can pull in 6 and 7. Does that make sense to you? Yea, that looks good to me. Patches up from 6+ should only be pulled once the pre-reqs for this set are in also (io-chain + Jean's func pwrst stuff.) I haven't actually tried these patches without the pre-reqs lately, but I think they should be fine. -Tero