drivers: i2c: Introduce I2C timeout for SAM0

Adds configurable timeout for I2C transactions for SAM0 SoCs.

Signed-off-by: Jan Kowalewski <[email protected]>

Author: kowalewskijan

drivers/i2c/Kconfig.sam0

@@ -18,3 +18,11 @@ config I2C_SAM0_DMA_DRIVEN
 	  DMA driven mode requires fewer interrupts to handle the
 	  transaction and ensures that high speed modes are not delayed
 	  by data reloading.
+
+config I2C_SAM0_TRANSFER_TIMEOUT
+	int "Transfer timeout [ms]"
+	default 500
+	help
+	  Timeout in milliseconds used for each I2C transfer.
+	  0 means that the driver should use the K_FOREVER value,
+	  i.e. it should wait as long as necessary.

drivers/i2c/i2c_sam0.c

@@ -24,6 +24,12 @@ LOG_MODULE_REGISTER(i2c_sam0, CONFIG_I2C_LOG_LEVEL);
 #define SERCOM_I2CM_CTRLA_MODE_I2C_MASTER SERCOM_I2CM_CTRLA_MODE(5)
 #endif
 
+#if CONFIG_I2C_SAM0_TRANSFER_TIMEOUT
+#define I2C_TRANSFER_TIMEOUT_MSEC K_MSEC(CONFIG_I2C_SAM0_TRANSFER_TIMEOUT)
+#else
+#define I2C_TRANSFER_TIMEOUT_MSEC K_FOREVER
+#endif
+
 struct i2c_sam0_dev_config {
 	SercomI2cm *regs;
 	const struct pinctrl_dev_config *pcfg;
@@ -504,7 +510,12 @@ static int i2c_sam0_transfer(const struct device *dev, struct i2c_msg *msgs,
 		irq_unlock(key);
 
 		/* Now wait for the ISR to handle everything */
-		k_sem_take(&data->sem, K_FOREVER);
+		ret = k_sem_take(&data->sem, I2C_TRANSFER_TIMEOUT_MSEC);
+
+		if (ret != 0) {
+			ret = -EIO;
+			goto unlock;
+		}
 
 		if (data->msg.status) {
 			if (data->msg.status & SERCOM_I2CM_STATUS_ARBLOST) {