@@ -1264,6 +1264,15 @@ static int omap2_mcspi_prepare_message(struct spi_controller *ctlr,
struct spi_transfer *tr;
u8 bits_per_word;
+ /*
+ * When the last transfer used multi-mode, the CS is not keep down by the controller.
+ * The default transfer_one_message expect that the last transfer of a message keep the CS
+ * active.
+ * Ensure that the next call to spi_set_cs will work as expected if single-mode is used.
+ */
+ if (mcspi->use_multi_mode)
+ ctlr->last_cs[0] = -1;
+
/*
* The conditions are strict, it is mandatory to check each transfer of the list to see if
* multi-mode is applicable.
The OMAP2 support features a "multi" mode, where the controller automatically sets the Chip Select (CS) when necessary. This mode allows for shorter delays between CS selection and word content, but the CS is always deasserted at the end. The default transfer_one_message function expects the last transfer of a message to keep the CS enabled. However, when two consecutive messages use "multi" mode followed by "single" mode, the chipselect is not enabled during the second message. This discrepancy causes the driver to time out. To address this issue, the CS is now reasserted for each new message that follows a "multi"-mode message. This change ensures proper CS handling and prevents driver timeouts. Fixes: d153ff4056cb ("spi: omap2-mcspi: Add support for MULTI-mode") Signed-off-by: Louis Chauvet <louis.chauvet@bootlin.com> --- Changes in v2: - Fix build error, I used the old API (master instead of controller) - Link to v1: https://lore.kernel.org/r/20240409-fix-omap2-mcspi-v1-1-f2b881ae196d@bootlin.com --- drivers/spi/spi-omap2-mcspi.c | 9 +++++++++ 1 file changed, 9 insertions(+) --- base-commit: dfca7eb7615116b696ecdc85417c9e2df0b546dc change-id: 20240408-fix-omap2-mcspi-a6b41142ad9a Best regards,