drm/tegra: Do not implement runtime PM
authorThierry Reding <treding@nvidia.com>
Mon, 2 Dec 2019 14:29:03 +0000 (15:29 +0100)
committerThierry Reding <treding@nvidia.com>
Fri, 10 Jan 2020 15:37:43 +0000 (16:37 +0100)
commitfd67e9c6ed5af223af0daee093593abe3dbb53d4
treeb6fa196007ee3b62453fdb5481679ed18185023e
parent608f43ad27ab26700677cc4d9ead3b2d9d338a21
drm/tegra: Do not implement runtime PM

The Tegra DRM driver heavily relies on the implementations for runtime
suspend/resume to be called at specific times. Unfortunately, there are
some cases where that doesn't work. One example is if the user disables
runtime PM for a given subdevice. Another example is that the PM core
acquires a reference to runtime PM during system sleep, effectively
preventing devices from going into low power modes. This is intentional
to avoid nasty race conditions, but it also causes system sleep to not
function properly on all Tegra systems.

Fix this by not implementing runtime PM at all. Instead, a minimal,
reference-counted suspend/resume infrastructure is added to the host1x
bus. This has the benefit that it can be used regardless of the system
power state (or any transitions we might be in), or whether or not the
user allows runtime PM.

Atomic modesetting guarantees that these functions will end up being
called at the right point in time, so the pitfalls for the more generic
runtime PM do not apply here.

Signed-off-by: Thierry Reding <treding@nvidia.com>
drivers/gpu/drm/tegra/dc.c
drivers/gpu/drm/tegra/dpaux.c
drivers/gpu/drm/tegra/drm.h
drivers/gpu/drm/tegra/dsi.c
drivers/gpu/drm/tegra/hdmi.c
drivers/gpu/drm/tegra/hub.c
drivers/gpu/drm/tegra/hub.h
drivers/gpu/drm/tegra/sor.c
drivers/gpu/host1x/bus.c
include/linux/host1x.h