2024-02-05 15:25:24

by Théo Lebrun

[permalink] [raw]
Subject: [PATCH v2 0/4] spi: cadence-qspi: Fix runtime PM and system-wide suspend

Hi,

This V2 got a false start, hoping you got a good laugh out of it. :-)
Sorry for the spam.

This fixes runtime PM and system-wide suspend for the cadence-qspi
driver. Seeing how runtime PM and autosuspend are enabled by default, I
believe this affects all users of the driver.

The V1 has been split into three commits and a fourth commit is added to
add system-wide suspend/resume callbacks as discussed under V1.

The MIPS platform at hand, used for debugging and testing, is currently
not supported by the driver. It is the Mobileye EyeQ5 [0]. No code
changes are required for support, only a new compatible and appropriate
match data + flags. That will come later, with some performance-related
patches.

This series has been tested on both Mobileye EyeQ5 hardware and the TI
J7200 EVM board, under s2idle.

Thanks all,
Théo

[0]: https://lore.kernel.org/lkml/[email protected]/

Signed-off-by: Théo Lebrun <[email protected]>
---
Changes in v2:
- Split the initial change into three separate commits, to make intents
clearer.
- Mark controller as suspended during the system-wide suspend.
- Link to v1: https://lore.kernel.org/r/[email protected]

---
Théo Lebrun (4):
spi: cadence-qspi: put runtime in runtime PM hooks names
spi: cadence-qspi: fix pointer reference in runtime PM hooks
spi: cadence-qspi: remove system-wide suspend helper calls from runtime PM hooks
spi: cadence-qspi: add system-wide suspend and resume callbacks

drivers/spi/spi-cadence-quadspi.c | 33 +++++++++++++++++++++------------
1 file changed, 21 insertions(+), 12 deletions(-)
---
base-commit: 13acce918af915278e49980a3038df31845dbf39
change-id: 20240202-cdns-qspi-pm-fix-29600cc6d7bf

Best regards,
--
Théo Lebrun <[email protected]>