[U-Boot] [PATCH] spi: cadence_qspi: Add quad write support
Vignesh R
vigneshr at ti.com
Wed Feb 27 05:08:57 UTC 2019
On 26/02/19 1:59 PM, Ley Foon Tan wrote:
> Use quad write if SPI_TX_QUAD flag is set.
>
How was the patch tested? Could you add that info to commit msg?
> Signed-off-by: Ley Foon Tan <ley.foon.tan at intel.com>
> ---
> drivers/spi/cadence_qspi.c | 2 +-
> drivers/spi/cadence_qspi.h | 2 +-
> drivers/spi/cadence_qspi_apb.c | 7 ++++++-
> 3 files changed, 8 insertions(+), 3 deletions(-)
> mode change 100644 => 100755 drivers/spi/cadence_qspi.c
> mode change 100644 => 100755 drivers/spi/cadence_qspi.h
> mode change 100644 => 100755 drivers/spi/cadence_qspi_apb.c
>
Why are the permissions being changed? Please keep it as is
> diff --git a/drivers/spi/cadence_qspi.c b/drivers/spi/cadence_qspi.c
> old mode 100644
> new mode 100755
> index 11fce9c4fe..efdb178450
> --- a/drivers/spi/cadence_qspi.c
> +++ b/drivers/spi/cadence_qspi.c
> @@ -256,7 +256,7 @@ static int cadence_spi_xfer(struct udevice *dev, unsigned int bitlen,
> break;
> case CQSPI_INDIRECT_WRITE:
> err = cadence_qspi_apb_indirect_write_setup
> - (plat, priv->cmd_len, cmd_buf);
> + (plat, priv->cmd_len, dm_plat->mode, cmd_buf);
> if (!err) {
> err = cadence_qspi_apb_indirect_write_execute
> (plat, data_bytes, dout);
> diff --git a/drivers/spi/cadence_qspi.h b/drivers/spi/cadence_qspi.h
> old mode 100644
> new mode 100755
> index 055900def0..b491407130
> --- a/drivers/spi/cadence_qspi.h
> +++ b/drivers/spi/cadence_qspi.h
> @@ -60,7 +60,7 @@ int cadence_qspi_apb_indirect_read_setup(struct cadence_spi_platdata *plat,
> int cadence_qspi_apb_indirect_read_execute(struct cadence_spi_platdata *plat,
> unsigned int rxlen, u8 *rxbuf);
> int cadence_qspi_apb_indirect_write_setup(struct cadence_spi_platdata *plat,
> - unsigned int cmdlen, const u8 *cmdbuf);
> + unsigned int cmdlen, unsigned int tx_width, const u8 *cmdbuf);
> int cadence_qspi_apb_indirect_write_execute(struct cadence_spi_platdata *plat,
> unsigned int txlen, const u8 *txbuf);
>
> diff --git a/drivers/spi/cadence_qspi_apb.c b/drivers/spi/cadence_qspi_apb.c
> old mode 100644
> new mode 100755
> index a8af352030..55a7501913
> --- a/drivers/spi/cadence_qspi_apb.c
> +++ b/drivers/spi/cadence_qspi_apb.c
> @@ -77,6 +77,7 @@
>
> #define CQSPI_REG_WR_INSTR 0x08
> #define CQSPI_REG_WR_INSTR_OPCODE_LSB 0
> +#define CQSPI_REG_WR_INSTR_TYPE_DATA_LSB 16
>
> #define CQSPI_REG_DELAY 0x0C
> #define CQSPI_REG_DELAY_TSLCH_LSB 0
> @@ -686,7 +687,7 @@ failrd:
>
> /* Opcode + Address (3/4 bytes) */
> int cadence_qspi_apb_indirect_write_setup(struct cadence_spi_platdata *plat,
> - unsigned int cmdlen, const u8 *cmdbuf)
> + unsigned int cmdlen, unsigned int tx_width, const u8 *cmdbuf)
> {
> unsigned int reg;
> unsigned int addr_bytes = cmdlen > 4 ? 4 : 3;
> @@ -702,6 +703,10 @@ int cadence_qspi_apb_indirect_write_setup(struct cadence_spi_platdata *plat,
>
> /* Configure the opcode */
> reg = cmdbuf[0] << CQSPI_REG_WR_INSTR_OPCODE_LSB;
> +
> + if (tx_width & SPI_TX_QUAD)
> + reg |= CQSPI_INST_TYPE_QUAD << CQSPI_REG_WR_INSTR_TYPE_DATA_LSB;
> +
> writel(reg, plat->regbase + CQSPI_REG_WR_INSTR);
>
> /* Setup write address. */
>
--
Regards
Vignesh
More information about the U-Boot
mailing list