From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 5D075B7B76 for ; Fri, 6 Nov 2009 01:57:56 +1100 (EST) Subject: Re: [PATCH 1/3] powerpc/83xx/suspend: Clear deep_sleeping after devices resume Mime-Version: 1.0 (Apple Message framework v1076) Content-Type: text/plain; charset=us-ascii; format=flowed From: Kumar Gala In-Reply-To: <20090923190113.GA19932@oksana.dev.rtsoft.ru> Date: Thu, 5 Nov 2009 08:57:50 -0600 Message-Id: <043A102D-9F70-4287-B34E-17FF9AE90EE1@kernel.crashing.org> References: <20090923190041.GA18944@oksana.dev.rtsoft.ru> <20090923190113.GA19932@oksana.dev.rtsoft.ru> To: Scott Wood Cc: linux-ppc list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Sep 23, 2009, at 2:01 PM, Anton Vorontsov wrote: > Currently 83xx PMC driver clears deep_sleeping variable very early, > before devices are resumed. This makes fsl_deep_sleep() unusable in > drivers' resume() callback. > > Sure, drivers can store fsl_deep_sleep() value on suspend and use > the stored value on resume. But a better solution is to postpone > clearing the deep_sleeping variable, i.e. move it into finish() > callback. > > Signed-off-by: Anton Vorontsov > --- > arch/powerpc/platforms/83xx/suspend.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) Scott, any comments or an ack? - k