tree d932a663f7b984d96be05a6ef18ff038845ce86a
parent 9cb2ff11171264d10be7ea9e31d9ee5d49ba84a5
author Marek Vasut <marex@denx.de> 1626459693 +0200
committer Mark Brown <broonie@kernel.org> 1626695924 +0100
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmD1aPQACgkQJNaLcl1U
 h9BGggf9FtPM7KQfsOdgPLuWAHymJEI/2eJEKR/Wnxhdu0FSGTOTlUQBSBDP9TMy
 6G6VXq9uvMz1LBWjcAdHj00tLm50fwNG2hPxpUFAVF23DxzJ8w1FYPkUCHd1rCuy
 0kceBn3//1s71gwgP3HvcBRogQhHn/htnU9xANGjEca7y5Dk12S+T0w1dO7kpYr3
 I7GQE2k9SiQt8cbL/AldZpO1T396jtx2p37UU/MdusXH+uv62GVpfeos6PdUyEH3
 qfWJ2660rcKhrnN6uL6nk/gb2xFfuPmgtLVlNU2XtXz3hWjpTqT+Ja6/TzenumDK
 BNVaKSpHBaROxeZRSDlY4Z5pVY8i9g==
 =4oYJ
 -----END PGP SIGNATURE-----

spi: cadence: Correct initialisation of runtime PM again

The original implementation of RPM handling in probe() was mostly
correct, except it failed to call pm_runtime_get_*() to activate the
hardware. The subsequent fix, 734882a8bf98 ("spi: cadence: Correct
initialisation of runtime PM"), breaks the implementation further,
to the point where the system using this hard IP on ZynqMP hangs on
boot, because it accesses hardware which is gated off.

Undo 734882a8bf98 ("spi: cadence: Correct initialisation of runtime
PM") and instead add missing pm_runtime_get_noresume() and move the
RPM disabling all the way to the end of probe(). That makes ZynqMP
not hang on boot yet again.

Fixes: 734882a8bf98 ("spi: cadence: Correct initialisation of runtime PM")
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Charles Keepax <ckeepax@opensource.cirrus.com>
Cc: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20210716182133.218640-1-marex@denx.de
Signed-off-by: Mark Brown <broonie@kernel.org>