From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH] r8169: increase the delay parameter of pm_schedule_suspend Date: Tue, 08 Nov 2011 12:37:16 -0500 (EST) Message-ID: <20111108.123716.1108093580193144770.davem@davemloft.net> References: <1320734677-1372-1-git-send-email-hayeswang@realtek.com> <20111108090716.GA4885@electric-eye.fr.zoreil.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: hayeswang@realtek.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, rjw@sisk.pl To: romieu@fr.zoreil.com Return-path: Received: from shards.monkeyblade.net ([198.137.202.13]:37727 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756143Ab1KHRi2 (ORCPT ); Tue, 8 Nov 2011 12:38:28 -0500 In-Reply-To: <20111108090716.GA4885@electric-eye.fr.zoreil.com> Sender: netdev-owner@vger.kernel.org List-ID: From: Francois Romieu Date: Tue, 8 Nov 2011 10:07:16 +0100 > Hayes Wang : >> The link down would occur when reseting PHY. And it would take about 2 ~ 5 seconds >> from link down to link up. If the delay of pm_schedule_suspend is not long enough, >> the device would enter runtime_suspend before link up. After link up, the device >> would wake up and reset PHY again. Then, you would find the driver keep in a loop >> of runtime_suspend and rumtime_resume. > > Acked-by: Francois Romieu Applied.