Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751661Ab3FZPp5 (ORCPT ); Wed, 26 Jun 2013 11:45:57 -0400 Received: from nat28.tlf.novell.com ([130.57.49.28]:52992 "EHLO nat28.tlf.novell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751349Ab3FZPp4 convert rfc822-to-8bit (ORCPT ); Wed, 26 Jun 2013 11:45:56 -0400 Message-Id: <51CB28D102000078000E0DF8@nat28.tlf.novell.com> X-Mailer: Novell GroupWise Internet Agent 12.0.2 Date: Wed, 26 Jun 2013 16:45:53 +0100 From: "Jan Beulich" To: "Ben Guthro" Cc: "Bob Moore" , , "Konrad Rzeszutek Wilk" , "Rafaell J . Wysocki" , , Subject: Re: [PATCH v3 1/3] acpi: Call acpi_os_prepare_sleep hook in reduced hardware sleep path References: <1372255575-29567-1-git-send-email-benjamin.guthro@citrix.com> <1372255575-29567-2-git-send-email-benjamin.guthro@citrix.com> <51CB19C602000078000E0D75@nat28.tlf.novell.com> In-Reply-To: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 8BIT Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2019 Lines: 51 >>> On 26.06.13 at 17:03, Ben Guthro wrote: > On Wed, Jun 26, 2013 at 10:41 AM, Jan Beulich wrote: >>>>> On 26.06.13 at 16:06, Ben Guthro wrote: >>> In version 3.4 acpi_os_prepare_sleep() got introduced in parallel with >>> reduced hardware sleep support, and the two changes didn't get >>> synchronized: The new code doesn't call the hook function (if so >>> requested). Fix this, requiring a parameter to be added to the >>> hook function to distinguish "extended" from "legacy" sleep. >>> >>> Signed-off-by: Ben Guthro >>> Signed-off-by: Jan Beulich >> >> I think these are intended to reflect the flow of things, so >> should be reversed (also in the other patches). >> >>> --- a/drivers/acpi/acpica/hwesleep.c >>> +++ b/drivers/acpi/acpica/hwesleep.c >>> @@ -43,6 +43,7 @@ >>> */ >>> >>> #include >>> +#include >> >> This also got complaints, so I'd be very surprised if they took it now. > > I did see these complaints in the last version. > However, the file drivers/acpi/acpica/hwsleep.c contains this include, > and has since > > commit 09f98a825a821f7a3f1b162f9ed023f37213a63b > Author: Tang Liang > Date: Fri Dec 9 10:05:54 2011 +0800 > > So since this is the extended sleep file, vs the standard one - I > don't see why such a restriction would be placed on the former, but > not the latter. In essence they said (in the same thread I pointed you to) that according to the current policy this include is wrong and should be dropped. Now, if you can get along without dropping it that'll likely be fine, but I doubt they'll allow you to add another instance of this. Jan -- 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/