Re: [PATCH v2 6/6] memory: renesas-rpc-if: Use runtime PM autosuspend after transfers

Wolfram Sang <[email protected]> Mon, 3 Aug 2026 09:33:20 +0200
Newsgroups org.kernel.vger.linux-renesas-soc,org.kernel.vger.linux-kernel
Message-ID <anBEQJ4VAaZfsUTw@ninjato>
--d3Mhk3lKqygFcu9s
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Jul 27, 2026 at 11:34:49AM +0100, Prabhakar wrote:
> From: Lad Prabhakar <[email protected]>
>=20
> Replace pm_runtime_put() with pm_runtime_put_autosuspend() after manual
> transfers and direct memory-mapped read/write operations.
>=20
> Flash page programming may be performed as a sequence of closely spaced
> transfer operations. Calling pm_runtime_put() after each operation allows
> the runtime PM core to suspend the controller between successive
> transfers, introducing unnecessary suspend/resume cycles and potentially
> interrupting multi-step transfer sequences.
>=20
> Use pm_runtime_put_autosuspend() instead so the controller remains
> active while transfers continue, allowing it to suspend only after an
> idle period.
>=20
> Signed-off-by: Lad Prabhakar <[email protected]>

I tested this series with my SparrowHawk but it doesn't use the xSPI
code paths, so only this patch is only affected. No improvements, but
neither regressions.

Tested-by: Wolfram Sang <[email protected]>

> Note corresponding pm_runtime_set_autosuspend_delay()/pm_runtime_use_auto=
suspend()
> calls are already present in SPI driver.

Unless I am missing it, there is no pm_runtime_set_autosuspend_delay()
in the driver nor in this patchset?


--d3Mhk3lKqygFcu9s
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmpwRD0ACgkQFA3kzBSg
KbZNKA/+M8ti7cYOvVQOWc7jd3tv3gM7luc8Lt5C95zmR+dO3ycLlxcu8Tgvvac+
JS9mJT7p7PpPpVOfyg91AxywpHli5GD99LeiDl2JOWNQPoAybhVkzYVQIxjyJI2H
TZZ8CzAivqvChP4hYmlPcM09UWTzqAki+WGQw/XyQP4d6z+x4soz7VmaQFANhStl
QmovIBvmnU8Ln5p7GagIaiAWbeJfbIA22FzTQWwRLgVvsruhXUrNMd6S/ogeVQik
rYDjEy0mgug7dFm7pyFOc6mvBeKvx7dud6WQbdjk2JBMvjiByKpzWx1vJ98RrttQ
web/JxUGthcarHHTBilGkCQZZoEW+ICgoVHeSRkP80B7ADut1yB1bRo362RF49Nh
arwe5jnuZphh4GQtFBGr2g7I6DbwFC7IlpxgZRPoOVgYB1vyYHiGnS87uRqB7Tzg
PyHYlWPa4XjSE6GxVNU9U95H1zXGYLj0hqeuBiiug88x3dhJNgpaHdQP6Ov878z0
4XCU2sKjiBJTBaoFTvNwMhpcRcmS5wREqv7f/n7+xVPU7ykU8DYlqX7u9vU0f+e/
rEibQNBEk9LtjjpaCbm9mu91Q8ZnVdUoYjnjZnzOWio+hlbsHbkhaLB8AVKDCg6n
0Z9wzbc4x2EBDuOyIf4hfxeZtO8B20FxO/9l39D/706XvYWLTto=
=r0Yh
-----END PGP SIGNATURE-----

--d3Mhk3lKqygFcu9s--