From mboxrd@z Thu Jan 1 00:00:00 1970 From: Borislav Petkov Subject: Re: [PATCH] PM: HIBERNATION: add resume_delay kernel param as well as resume_delay Date: Mon, 10 Oct 2011 09:00:28 +0200 Message-ID: <20111010070028.GA11577@liondog.tnic> References: <1318225452-10397-1-git-send-email-Barry.Song@csr.com> <20111010062653.GB4139@liondog.tnic> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mail.skyhub.de ([78.46.96.112]:50396 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751659Ab1JJHAc (ORCPT ); Mon, 10 Oct 2011 03:00:32 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Barry Song <21cnbao@gmail.com> Cc: Barry Song , pavel@ucw.cz, rjw@sisk.pl, linux-pm@lists.linux-foundation.org, linux-mmc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, workgroup.linux@csr.com, Barry Song On Mon, Oct 10, 2011 at 02:34:45PM +0800, Barry Song wrote: > >> diff --git a/kernel/power/hibernate.c b/kernel/power/hibernate.c > >> index b3da53c..f73a00e 100644 > >> --- a/kernel/power/hibernate.c > >> +++ b/kernel/power/hibernate.c > >> @@ -33,6 +33,7 @@ > >> =C2=A0static int nocompress =3D 0; > >> =C2=A0static int noresume =3D 0; > >> =C2=A0static int resume_wait =3D 0; > >> +static int resume_delay =3D 0; > > > > These static variables get zeroed needlessly and checkpatch actuall= y > > warns about that. Maybe fix that up while you're there? >=20 > yes. right. i initilized it to 0 to keep consistent coding style with > the context because there were other variants doing that. > i'd like to delete all initialization together as a seperate patch. O= r > do you want to do that? Nah, you go ahead :-). Thanks. --=20 Regards/Gruss, Boris.