lkml.org 
[lkml]   [2014]   [Oct]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v2 01/47] kernel: Add support for poweroff handler call chain
    On Tue, 21 Oct 2014, Johan Hovold wrote:

    > On Mon, Oct 20, 2014 at 09:12:17PM -0700, Guenter Roeck wrote:
    >
    > > v2:
    > > - poweroff -> power_off
    >
    > 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 notifiers
    > > - Add register_poweroff_handler_simple
    > > - Add devm_register_power_off_handler
    > >
    > > 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
    > >
    > > 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);
    > >
    > > +/*
    > > + * Callbacks to manage poweroff handlers
    >
    > power-off

    power_off, no?

    --
    Lee Jones
    Linaro STMicroelectronics Landing Team Lead
    Linaro.org │ Open source software for ARM SoCs
    Follow Linaro: Facebook | Twitter | Blog
    --
    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/

    \
     
     \ /
      Last update: 2014-10-21 13:01    [W:6.038 / U:0.020 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site