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--