Re: [PATCH 1/2] pmdomain: ti-sci: set the GENPD_FLAG_ACTIVE_WAKEUP flag for all PM domains

From: Ulf Hansson
Date: Wed Oct 09 2024 - 09:18:38 EST


On Tue, 8 Oct 2024 at 11:34, Thomas Richard <thomas.richard@xxxxxxxxxxx> wrote:
>
> With this flag, if a device is marked on the wakeup path, the corresponding
> PM domain is kept powered on.

Perhaps extend this with some more information that it fixes a problem
for the uart console too?

>
> Suggested-by: Théo Lebrun <theo.lebrun@xxxxxxxxxxx>
> Signed-off-by: Thomas Richard <thomas.richard@xxxxxxxxxxx>

Should we add a stable/fixes tag?

Kind regards
Uffe

> ---
> drivers/pmdomain/ti/ti_sci_pm_domains.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/pmdomain/ti/ti_sci_pm_domains.c b/drivers/pmdomain/ti/ti_sci_pm_domains.c
> index 1510d5ddae3d..38448b4a035a 100644
> --- a/drivers/pmdomain/ti/ti_sci_pm_domains.c
> +++ b/drivers/pmdomain/ti/ti_sci_pm_domains.c
> @@ -186,6 +186,7 @@ static int ti_sci_pm_domain_probe(struct platform_device *pdev)
>
> pd->pd.power_off = ti_sci_pd_power_off;
> pd->pd.power_on = ti_sci_pd_power_on;
> + pd->pd.flags |= GENPD_FLAG_ACTIVE_WAKEUP;
> pd->idx = args.args[0];
> pd->parent = pd_provider;
>
>
> --
> 2.39.5
>