[PATCH v13 13/35] drm/tegra: gr2d: Support generic power domain and runtime PM
Ulf Hansson
ulf.hansson at linaro.org
Fri Oct 1 13:39:10 UTC 2021
On Mon, 27 Sept 2021 at 00:42, Dmitry Osipenko <digetx at gmail.com> wrote:
>
> Add runtime power management and support generic power domains.
>
> Tested-by: Peter Geis <pgwipeout at gmail.com> # Ouya T30
> Tested-by: Paul Fertser <fercerpav at gmail.com> # PAZ00 T20
> Tested-by: Nicolas Chauvet <kwizart at gmail.com> # PAZ00 T20 and TK1 T124
> Tested-by: Matt Merhar <mattmerhar at protonmail.com> # Ouya T30
> Signed-off-by: Dmitry Osipenko <digetx at gmail.com>
> ---
> drivers/gpu/drm/tegra/gr2d.c | 155 +++++++++++++++++++++++++++++++++--
[...]
> static int gr2d_remove(struct platform_device *pdev)
> @@ -259,15 +312,101 @@ static int gr2d_remove(struct platform_device *pdev)
> return err;
> }
>
> + pm_runtime_dont_use_autosuspend(&pdev->dev);
> + pm_runtime_disable(&pdev->dev);
There is no guarantee that the ->runtime_suspend() has been invoked
here, which means that clock may be left prepared/enabled beyond this
point.
I suggest you call pm_runtime_force_suspend(), instead of
pm_runtime_disable(), to make sure that gets done.
[...]
Kind regards
Uffe
More information about the dri-devel
mailing list