]> git.proxmox.com Git - mirror_ubuntu-hirsute-kernel.git/commit
bus: ti-sysc: Fix am335x resume hang for usb otg module
authorTony Lindgren <tony@atomide.com>
Tue, 18 May 2021 06:47:23 +0000 (09:47 +0300)
committerStefan Bader <stefan.bader@canonical.com>
Wed, 30 Jun 2021 06:27:50 +0000 (08:27 +0200)
commit6ede563e7b0cf0c5e10dea6cbc4cd0058583a26d
tree0344d47c37da9fbe6be17d1a087568c4810c1b04
parentdcbf647e0fc7f327fdea6201dfa90127c384dd18
bus: ti-sysc: Fix am335x resume hang for usb otg module

BugLink: https://bugs.launchpad.net/bugs/1933691
[ Upstream commit 4d7b324e231366ea772ab10df46be31273ca39af ]

On am335x, suspend and resume only works once, and the system hangs if
suspend is attempted again. However, turns out suspend and resume works
fine multiple times if the USB OTG driver for musb controller is loaded.

The issue is caused my the interconnect target module losing context
during suspend, and it needs a restore on resume to be reconfigure again
as debugged earlier by Dave Gerlach <d-gerlach@ti.com>.

There are also other modules that need a restore on resume, like gpmc as
noted by Dave. So let's add a common way to restore an interconnect
target module based on a quirk flag. For now, let's enable the quirk for
am335x otg only to fix the suspend and resume issue.

As gpmc is not causing hangs based on tests with BeagleBone, let's patch
gpmc separately. For gpmc, we also need a hardware reset done before
restore according to Dave.

To reinit the modules, we decouple system suspend from PM runtime. We
replace calls to pm_runtime_force_suspend() and pm_runtime_force_resume()
with direct calls to internal functions and rely on the driver internal
state. There no point trying to handle complex system suspend and resume
quirks via PM runtime.

This is issue should have already been noticed with commit 1819ef2e2d12
("bus: ti-sysc: Use swsup quirks also for am335x musb") when quirk
handling was added for am335x otg for swsup. But the issue went unnoticed
as having musb driver loaded hides the issue, and suspend and resume works
once without the driver loaded.

Fixes: 1819ef2e2d12 ("bus: ti-sysc: Use swsup quirks also for am335x musb")
Suggested-by: Dave Gerlach <d-gerlach@ti.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Signed-off-by: Kelsey Skunberg <kelsey.skunberg@canonical.com>
drivers/bus/ti-sysc.c
include/linux/platform_data/ti-sysc.h