From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH v2 01/47] kernel: Add support for poweroff handler call chain Date: Tue, 21 Oct 2014 11:30:07 +0100 Message-ID: <20141021103007.GI26842@x1> References: <1413864783-3271-1-git-send-email-linux@roeck-us.net> <1413864783-3271-2-git-send-email-linux@roeck-us.net> <20141021093405.GA2113@localhost> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mail-ie0-f172.google.com ([209.85.223.172]:52848 "EHLO mail-ie0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751283AbaJUKaO (ORCPT ); Tue, 21 Oct 2014 06:30:14 -0400 Received: by mail-ie0-f172.google.com with SMTP id rl12so872103iec.17 for ; Tue, 21 Oct 2014 03:30:14 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20141021093405.GA2113@localhost> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Johan Hovold Cc: Guenter Roeck , linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, Alan Cox , Alexander Graf , Andrew Morton , Geert Uytterhoeven , Heiko Stuebner , Len Brown , Pavel Machek , Philippe =?iso-8859-1?Q?R=E9tornaz?= , "Rafael J. Wysocki" , Romain Perier On Tue, 21 Oct 2014, Johan Hovold wrote: > On Mon, Oct 20, 2014 at 09:12:17PM -0700, Guenter Roeck wrote: >=20 > > v2: > > - poweroff -> power_off >=20 > Do you want to stick to power[-_ ]off consistently throughout? I guess you'll want to fix all of these before I review the MFD patches? > > - Add defines for default priorities > > - Use raw notifiers protected by spinlocks instead of atomic notifi= ers > > - Add register_poweroff_handler_simple > > - Add devm_register_power_off_handler > >=20 > > include/linux/pm.h | 22 ++++ > > kernel/power/Makefile | 1 + > > kernel/power/poweroff_handler.c | 252 ++++++++++++++++++++++++++++= ++++++++++++ > > 3 files changed, 275 insertions(+) > > create mode 100644 kernel/power/poweroff_handler.c > >=20 > > diff --git a/include/linux/pm.h b/include/linux/pm.h > > index 383fd68..7e0cb36 100644 > > --- a/include/linux/pm.h > > +++ b/include/linux/pm.h > > @@ -34,7 +34,29 @@ > > extern void (*pm_power_off)(void); > > extern void (*pm_power_off_prepare)(void); > > =20 > > +/* > > + * Callbacks to manage poweroff handlers >=20 > power-off power_off, no? --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog