| Message ID | 20250326083954.3338597-1-quic_msavaliy@quicinc.com |
|---|---|
| State | New |
| Headers | show |
| Series | [V1] spi: Add support for Double Transfer Rate (DTR) mode | expand |
Thanks Mark ! On 3/27/2025 7:37 PM, Mark Brown wrote: > On Thu, Mar 27, 2025 at 03:33:15PM +0530, Mukesh Kumar Savaliya wrote: > >> IIUC, It comes to the point of first identifying if it's in context of QSPI >> controller or SPI controller right ? > >> Identify if SPI/QSPI controller has this capability using dtr_caps = >> true/false. Then check if it's supporting SDR/DDR mode. Can we then >> introduce below struct to first mark capability as true/false and then >> process dtr_mode ? > >> if not supported (dtr_caps = false), then don't care dtr_mode. > > We should complain if the device requsted anything the controller can't > support. > Yes, makes sense. i have added dtr_caps into controller data structure and validating it inside the __spi_validate(). >> struct spi_caps { >> bool dtr_mode; >> bool dtr_caps; >> }; > > The controller capabilities are currently mainly advertised via the > SPI_CONTROLLER_ flags but adding some bools also works, some things > already do use that. I'm not sure we need to wrap things in a further > struct though. Uploaded Patch V2. Please help review and suggest next. it may require still some fine stunning.
diff --git a/include/linux/spi/spi.h b/include/linux/spi/spi.h index 0ba5e49bace4..4e1152de82cd 100644 --- a/include/linux/spi/spi.h +++ b/include/linux/spi/spi.h @@ -998,6 +998,7 @@ struct spi_res { * processed the word, i.e. the "pre" timestamp should be taken before * transmitting the "pre" word, and the "post" timestamp after receiving * transmit confirmation from the controller for the "post" word. + * @dtr_mode: true if supports double transfer rate. * @timestamped: true if the transfer has been timestamped * @error: Error status logged by SPI controller driver. * @@ -1049,6 +1050,9 @@ struct spi_res { * two should both be set. User can set transfer mode with SPI_NBITS_SINGLE(1x) * SPI_NBITS_DUAL(2x) and SPI_NBITS_QUAD(4x) to support these three transfer. * + * User may also set dtr_mode to true to use dual transfer mode if desired. if + * not, default considered as single transfer mode. + * * The code that submits an spi_message (and its spi_transfers) * to the lower layers is responsible for managing its memory. * Zero-initialize every field you don't set up explicitly, to @@ -1083,6 +1087,7 @@ struct spi_transfer { unsigned tx_nbits:4; unsigned rx_nbits:4; unsigned timestamped:1; + bool dtr_mode; #define SPI_NBITS_SINGLE 0x01 /* 1-bit transfer */ #define SPI_NBITS_DUAL 0x02 /* 2-bit transfer */ #define SPI_NBITS_QUAD 0x04 /* 4-bit transfer */
There is no support for protocol drivers to specify whether a transfer should use single or dual transfer mode. As a result, the SPI controller has no way to determine this information from the user, leading to potential limitations in transfer capabilities. This change introduces a new field `dtr_mode` in the `spi_transfer` structure. The `dtr_mode` field allows protocol drivers to indicate if Double Transfer Rate (DTR) mode is supported for a given transfer. When `dtr_mode` is set to true, the SPI controller will use DTR mode otherwise, it will default to single transfer mode. The QSPI controller driver uses this flag and configures single or double transfer rate using the controller register. The existing spi-mem driver helps configure the DTR mode but is limited to memory devices. There is no support available to set DTR mode for non-memory devices, e.g., touch or any generic SPI sensor. This change is backward compatible and doesn't break existing SPI or QSPI drivers. Changes include: - Addition of `dtr_mode` field in `spi_transfer` structure. - Documentation updates to reflect the new `dtr_mode` field. Signed-off-by: Mukesh Kumar Savaliya <quic_msavaliy@quicinc.com> --- include/linux/spi/spi.h | 5 +++++ 1 file changed, 5 insertions(+)