Hi Lina, On Wed, Oct 5, 2016 at 10:31 PM, Lina Iyer <lina.iyer@xxxxxxxxxx> wrote: > This patch allows domains to define idle states in the DT. SoC's can > define domain idle states in DT using the "domain-idle-states" property > of the domain provider. Calling of_pm_genpd_init() will read the idle > states and initialize the genpd for the domain. > > In addition to the entry and exit latency for idle state, also add > residency_ns, param and of_node property to each state. A domain idling > in a state is only power effecient if it stays idle for a certain period > in that state. The residency provides this minimum time for the idle > state to provide power benefits. The param is a state specific u32 value > that the platform may use for that idle state. > > This patch is based on the original patch by Marc Titinger. > > Signed-off-by: Marc Titinger <mtitinger+renesas@xxxxxxxxxxxx> > Signed-off-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx> > Signed-off-by: Lina Iyer <lina.iyer@xxxxxxxxxx> > --- > drivers/base/power/domain.c | 103 ++++++++++++++++++++++++++++++++++++++++++++ > include/linux/pm_domain.h | 8 ++++ > 2 files changed, 111 insertions(+) > > diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c > index 740afa9..368a5b8 100644 > --- a/drivers/base/power/domain.c > +++ b/drivers/base/power/domain.c > @@ -1895,6 +1895,109 @@ out: > return ret ? -EPROBE_DEFER : 0; > } > EXPORT_SYMBOL_GPL(genpd_dev_pm_attach); > + > +static const struct of_device_id idle_state_match[] = { > + { .compatible = "arm,idle-state", }, Do we want ARM-specific compatible values without an #ifdef in drivers/base/? I know we already have "samsung,power-domain". Perhaps that should be protected by #ifdef, too. > + { } > +}; 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-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html