Re: [PATCH 35/44] m68k: Register with kernel poweroff handler

From: Geert Uytterhoeven
Date: Tue Oct 07 2014 - 05:19:46 EST


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
--
To unsubscribe from this list: send the line "unsubscribe linux-alpha" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html