On Tue, Oct 7, 2014 at 7:28 AM, Guenter Roeck <linux@xxxxxxxxxxxx> wrote: > Register with kernel poweroff handler instead of setting pm_power_off > directly. > > Cc: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > Cc: Joshua Thompson <funaho@xxxxxxxxx> > Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx> As someone already mentioned, having #defines instead of hardcoded numbers for the priorities would be nice. Apart from that: Acked-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > + register_poweroff_handler_simple(nf_poweroff, 128); > + register_poweroff_handler_simple(mac_poweroff, 128); Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds