On 9 November 2017 at 14:27, Geert Uytterhoeven <geert+renesas@xxxxxxxxx> wrote: > If a device is part of the CPG/MSTP Clock Domain and to be used as a > wakeup source, it must be kept active during system suspend. > > Currently this is handled in device-specific drivers by explicitly > increasing the use count of the module clock when the device is > configured as a wakeup source. However, the proper way to prevent the > device from being stopped is to inform this requirement to the genpd > core, by setting the GENPD_FLAG_ACTIVE_WAKEUP flag. > > Note that this will only affect devices configured as wakeup sources. > > Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> Reviewed-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx> Kind regards Uffe > --- > v2: > - Integrate "clk: renesas: mstp: Use GENPD_FLAG_ACTIVE_WAKEUP", > --- > drivers/clk/renesas/clk-mstp.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clk/renesas/clk-mstp.c b/drivers/clk/renesas/clk-mstp.c > index 500a9e4e03c48957..0b222197a4576fd7 100644 > --- a/drivers/clk/renesas/clk-mstp.c > +++ b/drivers/clk/renesas/clk-mstp.c > @@ -344,7 +344,7 @@ void __init cpg_mstp_add_clk_domain(struct device_node *np) > return; > > pd->name = np->name; > - pd->flags = GENPD_FLAG_PM_CLK; > + pd->flags = GENPD_FLAG_PM_CLK | GENPD_FLAG_ACTIVE_WAKEUP; > pd->attach_dev = cpg_mstp_attach_dev; > pd->detach_dev = cpg_mstp_detach_dev; > pm_genpd_init(pd, &pm_domain_always_on_gov, false); > -- > 2.7.4 >
