diff options
author | Roger Quadros <rogerq@ti.com> | 2015-06-02 12:10:40 +0300 |
---|---|---|
committer | Kishon Vijay Abraham I <kishon@ti.com> | 2015-07-15 20:02:09 +0530 |
commit | 0a0830feb2adce8c7234b8c166a32fe9e7616788 (patch) | |
tree | 0a3677d1b6a3446334d144ca10c7038ffaa06999 /.mailmap | |
parent | d770e558e21961ad6cfdf0ff7df0eb5d7d4f0754 (diff) | |
download | op-kernel-dev-0a0830feb2adce8c7234b8c166a32fe9e7616788.zip op-kernel-dev-0a0830feb2adce8c7234b8c166a32fe9e7616788.tar.gz |
phy: ti-pipe3: fix suspend
Relying on PM-ops for shutting down PHY clocks was a
bad idea since the users (e.g. PCIe/SATA) might not
have been suspended by then.
The main culprit for not shutting down the clocks was
the stray pm_runtime_get() call in probe.
Fix the whole thing in the right way by getting rid
of that pm_runtime_get() call from probe and
removing all PM-ops. It is the sole responsibility
of the PHY user to properly turn OFF and de-initialize
the PHY as part of its suspend routine.
As PHY core serializes init/exit we don't need
to use a spinlock in this driver. So get rid of it.
Signed-off-by: Roger Quadros <rogerq@ti.com>
Signed-off-by: Sekhar Nori <nsekhar@ti.com>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
Diffstat (limited to '.mailmap')
0 files changed, 0 insertions, 0 deletions