On Mon, 27 Sept 2021 at 00:42, Dmitry Osipenko <dig...@gmail.com> wrote: > > Add runtime power management and support generic power domains. > > Tested-by: Peter Geis <pgwipe...@gmail.com> # Ouya T30 > Tested-by: Paul Fertser <fercer...@gmail.com> # PAZ00 T20 > Tested-by: Nicolas Chauvet <kwiz...@gmail.com> # PAZ00 T20 and TK1 T124 > Tested-by: Matt Merhar <mattmer...@protonmail.com> # Ouya T30 > Signed-off-by: Dmitry Osipenko <dig...@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
