DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :cc:content-type:content-transfer-encoding;
        b=qVmxuVMptr+Hvy4o+y6ttoDgCTKnEZ7aZGqp7QPEYoktbHgLi83c6zjJHNaFX9N9rg
         /I3Pkfu4zavgPIq0BM/drUJ6xmreVwLpS0VWr9jV/+s602d4Hhs8BVEOC+rwM4fK812u
         EuN7pSzzRRPk7VBcWvJXyYRYJSIf+y4qtSwSs=
MIME-Version: 1.0
In-Reply-To: <201012142058.04469.rjw@sisk.pl>
References: <201012130128.31243.rjw@sisk.pl>
	<AANLkTi=UVOQb15EdpfSeD+Qh8x779pJpsSCTyGvf-gxG@mail.gmail.com>
	<AANLkTim+UoR+GeEK8T1_jgie3gwq6Pdt_=sRiSf8qx6K@mail.gmail.com>
	<201012142058.04469.rjw@sisk.pl>
Date: Wed, 15 Dec 2010 22:12:49 +0800
Message-ID: <AANLkTikT-8XjqBfSPAVRZRKp5BQq_D8DeUo2YN3c4ntF@mail.gmail.com>
Subject: Re: [RFC][PATCH 2/4] PM: Remove redundant checks from core device
 resume routines
From: Ming Lei <tom.leiming@gmail.com>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
        Alan Stern <stern@rowland.harvard.edu>,
        LKML <linux-kernel@vger.kernel.org>,
        Linux-pm mailing list <linux-pm@lists.linux-foundation.org>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 765
Lines: 22

2010/12/15 Rafael J. Wysocki <rjw@sisk.pl>:
> On Tuesday, December 14, 2010, Ming Lei wrote:

>> Seems it may apply in other cases(dpm_prepare/dpm_suspend
>> /dpm_suspend_noirq) too?
>
> I thought about that, but in these cases the status is changed after the
> callback has returned and only if it's succeeded. ?Moreover, if the callback
> hasn't been successful, the device is not moved to the new list, so I think
> it's better to leave that as is.

Yes, you are right, sorry for my noise, :-(


thanks,
-- 
Lei Ming
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/