Hi Kevin, On Tue, Sep 9, 2014 at 6:11 PM, Kevin Hilman <khilman@xxxxxxxxxx> 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. >> + */ >> + return -EBUSY; >> +} > > > Do you have plans to use pm_genpd_attach_cpuidle() for this eventually, > maybe by checking if the enclosing node is a CPU? If not, are there > other plans to handle idle for the power domains that contain CPUs? Yes, the plan is to add more power saving features, and use DT topology analysis to get rid of the hardcoded domain logic. 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 devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html