[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