drivers: spi: cc23x0: Add power management

Add PM support to cc23x0 SPI module. This implies listing states which
cause power loss and enabling device runtime PM for the DMA in the DT.

Signed-off-by: Julien Panis <[email protected]>

Author: jpanisbl

drivers/spi/spi_cc23x0.c

@@ -14,6 +14,9 @@ LOG_MODULE_REGISTER(spi_cc23x0, CONFIG_SPI_LOG_LEVEL);
 #include <zephyr/drivers/pinctrl.h>
 #include <zephyr/drivers/spi.h>
 #include <zephyr/irq.h>
+#include <zephyr/pm/device.h>
+#include <zephyr/pm/device_runtime.h>
+#include <zephyr/pm/policy.h>
 #include <zephyr/sys/util.h>
 
 #include <driverlib/clkctl.h>
@@ -307,6 +310,8 @@ static int spi_cc23x0_transceive(const struct device *dev,
 	};
 #endif
 
+	pm_policy_device_power_lock_get(dev);
+
 	spi_context_lock(ctx, false, NULL, NULL, config);
 
 	ret = spi_cc23x0_configure(dev, config);
@@ -339,16 +344,22 @@ static int spi_cc23x0_transceive(const struct device *dev,
 	block_cfg_rx.dest_addr_adj = DMA_ADDR_ADJ_INCREMENT;
 	block_cfg_rx.block_size = SPI_CC23_DFS * data->tx_len_left;
 
-	ret = dma_config(cfg->dma_dev, cfg->dma_channel_tx, &dma_cfg_tx);
+	ret = pm_device_runtime_get(cfg->dma_dev);
 	if (ret) {
-		LOG_ERR("Failed to configure DMA TX channel");
+		LOG_ERR("Failed to resume DMA (%d)", ret);
 		goto int_disable;
 	}
 
+	ret = dma_config(cfg->dma_dev, cfg->dma_channel_tx, &dma_cfg_tx);
+	if (ret) {
+		LOG_ERR("Failed to configure DMA TX channel (%d)", ret);
+		goto dma_suspend;
+	}
+
 	ret = dma_config(cfg->dma_dev, cfg->dma_channel_rx, &dma_cfg_rx);
 	if (ret) {
-		LOG_ERR("Failed to configure DMA RX channel");
-		goto int_disable;
+		LOG_ERR("Failed to configure DMA RX channel (%d)", ret);
+		goto dma_suspend;
 	}
 
 	/* Disable DMA triggers */
@@ -364,14 +375,19 @@ static int spi_cc23x0_transceive(const struct device *dev,
 	ret = spi_context_wait_for_completion(&data->ctx);
 	if (ret) {
 		LOG_ERR("SPI transfer failed (%d)", ret);
-		goto int_disable;
+		goto dma_suspend;
 	}
 
 	spi_context_update_tx(ctx, SPI_CC23_DFS, data->tx_len_left);
 	spi_context_update_rx(ctx, SPI_CC23_DFS, data->tx_len_left);
 
 	LOG_DBG("SPI transfer completed");
 
+dma_suspend:
+	ret = pm_device_runtime_put(cfg->dma_dev);
+	if (ret) {
+		LOG_ERR("Failed to suspend DMA (%d)", ret);
+	}
 int_disable:
 	SPIDisableInt(cfg->base, SPI_CC23_INT_MASK);
 #else
@@ -388,6 +404,7 @@ static int spi_cc23x0_transceive(const struct device *dev,
 
 ctx_release:
 	spi_context_release(ctx, ret);
+	pm_policy_device_power_lock_put(dev);
 	return ret;
 }
 
@@ -446,6 +463,29 @@ static int spi_cc23x0_init(const struct device *dev)
 	return 0;
 }
 
+#ifdef CONFIG_PM_DEVICE
+
+static int spi_cc23x0_pm_action(const struct device *dev, enum pm_device_action action)
+{
+	const struct spi_cc23x0_config *cfg = dev->config;
+	struct spi_cc23x0_data *data = dev->data;
+
+	switch (action) {
+	case PM_DEVICE_ACTION_SUSPEND:
+		SPIDisable(cfg->base);
+		CLKCTLDisable(CLKCTL_BASE, CLKCTL_SPI0);
+		return 0;
+	case PM_DEVICE_ACTION_RESUME:
+		/* Force SPI to be reconfigured at next transfer */
+		data->ctx.config = NULL;
+		return 0;
+	default:
+		return -ENOTSUP;
+	}
+}
+
+#endif /* CONFIG_PM_DEVICE */
+
 #ifdef CONFIG_SPI_CC23X0_DMA_DRIVEN
 #define SPI_CC23X0_DMA_INIT(n)						\
 	.dma_dev = DEVICE_DT_GET(TI_CC23X0_DT_INST_DMA_CTLR(n, tx)),	\
@@ -459,6 +499,7 @@ static int spi_cc23x0_init(const struct device *dev)
 
 #define SPI_CC23X0_INIT(n)						\
 	PINCTRL_DT_INST_DEFINE(n);					\
+	PM_DEVICE_DT_INST_DEFINE(n, spi_cc23x0_pm_action);		\
 									\
 	static void spi_irq_config_func_##n(void)			\
 	{								\
@@ -484,7 +525,7 @@ static int spi_cc23x0_init(const struct device *dev)
 									\
 	DEVICE_DT_INST_DEFINE(n,					\
 			      spi_cc23x0_init,				\
-			      NULL,					\
+			      PM_DEVICE_DT_INST_GET(n),			\
 			      &spi_cc23x0_data_##n,			\
 			      &spi_cc23x0_config_##n,			\
 			      POST_KERNEL,				\

dts/arm/ti/cc23x0.dtsi

@@ -121,6 +121,7 @@
 			reg = <0x40026000 0x524>;
 			interrupts = <8 0>;
 			#dma-cells = <2>;
+			zephyr,pm-device-runtime-auto;
 			status = "disabled";
 		};
 
@@ -132,6 +133,7 @@
 			interrupts = <10 0>;
 			dmas = <&dma 0 0>, <&dma 1 1>;
 			dma-names = "tx", "rx";
+			zephyr,disabling-power-states = <&state0 &state1>;
 			status = "disabled";
 		};