spi: tegra210-quad: Fix X1_X2_X4 encoding and support x4 transfers

This patch corrects the QSPI_COMMAND_X1_X2_X4 and QSPI_ADDRESS_X1_X2_X4
macros to properly encode the bus width for x1, x2, and x4 transfers.
Although these macros were previously incorrect, they were not being
used in the driver, so no functionality was affected.

The patch updates tegra_qspi_cmd_config() and tegra_qspi_addr_config()
function calls to use the actual bus width from the transfer, instead of
hardcoding it to 0 (which implied x1 mode). This change enables proper
support for x1, x2, and x4 data transfers by correctly configuring the
interface width for commands and addresses.

These modifications improve the QSPI driver's flexibility and prepare it
for future use cases that may require different bus widths for commands
and addresses.

Fixes: 1b8342cc4a38 ("spi: tegra210-quad: combined sequence mode")
Signed-off-by: Vishwaroop A <va@nvidia.com>
Link: https://patch.msgid.link/20250416110606.2737315-2-va@nvidia.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/drivers/spi/spi-tegra210-quad.c b/drivers/spi/spi-tegra210-quad.c
index 08e49a8768943c1cf12fe2f2893e5eae1bf70d0f..4983b1f705b84a49ab62f3d521b67069595d3330 100644 (file)
--- a/drivers/spi/spi-tegra210-quad.c
+++ b/drivers/spi/spi-tegra210-quad.c
@@ -134,7 +134,7 @@
 #define QSPI_COMMAND_VALUE_SET(X)              (((x) & 0xFF) << 0)
 
 #define QSPI_CMB_SEQ_CMD_CFG                   0x1a0
-#define QSPI_COMMAND_X1_X2_X4(x)               (((x) & 0x3) << 13)
+#define QSPI_COMMAND_X1_X2_X4(x)               ((((x) >> 1) & 0x3) << 13)
 #define QSPI_COMMAND_X1_X2_X4_MASK             (0x03 << 13)
 #define QSPI_COMMAND_SDR_DDR                   BIT(12)
 #define QSPI_COMMAND_SIZE_SET(x)               (((x) & 0xFF) << 0)
@@ -147,7 +147,7 @@
 #define QSPI_ADDRESS_VALUE_SET(X)              (((x) & 0xFFFF) << 0)
 
 #define QSPI_CMB_SEQ_ADDR_CFG                  0x1ac
-#define QSPI_ADDRESS_X1_X2_X4(x)               (((x) & 0x3) << 13)
+#define QSPI_ADDRESS_X1_X2_X4(x)               ((((x) >> 1) & 0x3) << 13)
 #define QSPI_ADDRESS_X1_X2_X4_MASK             (0x03 << 13)
 #define QSPI_ADDRESS_SDR_DDR                   BIT(12)
 #define QSPI_ADDRESS_SIZE_SET(x)               (((x) & 0xFF) << 0)
@@ -1036,10 +1036,6 @@ static u32 tegra_qspi_addr_config(bool is_ddr, u8 bus_width, u8 len)
 {
        u32 addr_config = 0;
 
-       /* Extract Address configuration and value */
-       is_ddr = 0; //Only SDR mode supported
-       bus_width = 0; //X1 mode
-
        if (is_ddr)
                addr_config |= QSPI_ADDRESS_SDR_DDR;
        else
@@ -1079,13 +1075,13 @@ static int tegra_qspi_combined_seq_xfer(struct tegra_qspi *tqspi,
                switch (transfer_phase) {
                case CMD_TRANSFER:
                        /* X1 SDR mode */
-                       cmd_config = tegra_qspi_cmd_config(false, 0,
+                       cmd_config = tegra_qspi_cmd_config(false, xfer->tx_nbits,
                                                           xfer->len);
                        cmd_value = *((const u8 *)(xfer->tx_buf));
                        break;
                case ADDR_TRANSFER:
                        /* X1 SDR mode */
-                       addr_config = tegra_qspi_addr_config(false, 0,
+                       addr_config = tegra_qspi_addr_config(false, xfer->tx_nbits,
                                                             xfer->len);
                        address_value = *((const u32 *)(xfer->tx_buf));
                        break;