Hi Kevin, On Tue, Sep 9, 2014 at 5:40 PM, Kevin Hilman <khil...@kernel.org> wrote: >> +static int r8a7740_pd_a4s_suspend(void) >> +{ >> + /* >> + * The A4S domain contains the CPU core and therefore it should >> + * only be turned off if the CPU is in use. > > I think you meant "if the CPU is *not* in use." ? :)
Ah, this was copied from pm-r8a7740.c, which was copied from pm-sh7372.c. The thinko has been fixed in the latter, when more advanced suspend support was added. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org 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-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/