]> git.ipfire.org Git - thirdparty/u-boot.git/commitdiff
zynq: qspi: Rename few names
authorJagannadha Sutradharudu Teki <jaganna@xilinx.com>
Mon, 20 May 2013 09:32:05 +0000 (15:02 +0530)
committerMichal Simek <michal.simek@xilinx.com>
Mon, 27 May 2013 11:21:20 +0000 (13:21 +0200)
Use proper name for the driver before we go to mainline.
- XQSPIPS -> ZYNQ_QSPI
- NORM_READ -> NR
- FAST_READ -> FR
- DUAL_READ -> DR
- QUAD_READ -> QR
- ERASE_SUS -> ES
- ERASE_RES -> ER
- xqspips -> zynq_qspi
- zynq_spi_slave -> zynq_qspi_slave

Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
drivers/spi/zynq_qspi.c

index 75939aceef728feae3cc9012ee2d0b7b85c5d154..3336b317543a50b1791da32b72713cab06117ac1 100644 (file)
 #include <asm/arch/sys_proto.h>
 
 /* QSPI Transmit Data Register */
-#define XQSPIPS_TXD_00_00_OFFSET       0x1C /* Transmit 4-byte inst, WO */
-#define XQSPIPS_TXD_00_01_OFFSET       0x80 /* Transmit 1-byte inst, WO */
-#define XQSPIPS_TXD_00_10_OFFSET       0x84 /* Transmit 2-byte inst, WO */
-#define XQSPIPS_TXD_00_11_OFFSET       0x88 /* Transmit 3-byte inst, WO */
+#define ZYNQ_QSPI_TXD_00_00_OFFSET     0x1C /* Transmit 4-byte inst, WO */
+#define ZYNQ_QSPI_TXD_00_01_OFFSET     0x80 /* Transmit 1-byte inst, WO */
+#define ZYNQ_QSPI_TXD_00_10_OFFSET     0x84 /* Transmit 2-byte inst, WO */
+#define ZYNQ_QSPI_TXD_00_11_OFFSET     0x88 /* Transmit 3-byte inst, WO */
 
 /*
  * QSPI Configuration Register bit Masks
  * This register contains various control bits that effect the operation
  * of the QSPI controller
  */
-#define XQSPIPS_CONFIG_MANSRT_MASK     0x00010000 /* Manual TX Start */
-#define XQSPIPS_CONFIG_CPHA_MASK       0x00000004 /* Clock Phase Control */
-#define XQSPIPS_CONFIG_CPOL_MASK       0x00000002 /* Clock Polarity Control */
-#define XQSPIPS_CONFIG_SSCTRL_MASK     0x00003C00 /* Slave Select Mask */
+#define ZYNQ_QSPI_CONFIG_MANSRT_MASK   0x00010000 /* Manual TX Start */
+#define ZYNQ_QSPI_CONFIG_CPHA_MASK     0x00000004 /* Clock Phase Control */
+#define ZYNQ_QSPI_CONFIG_CPOL_MASK     0x00000002 /* Clock Polarity Control */
+#define ZYNQ_QSPI_CONFIG_SSCTRL_MASK   0x00003C00 /* Slave Select Mask */
 
 /*
  * QSPI Interrupt Registers bit Masks
  * All the four interrupt registers (Status/Mask/Enable/Disable) have the same
  * bit definitions.
  */
-#define XQSPIPS_IXR_TXNFULL_MASK       0x00000004 /* QSPI TX FIFO Overflow */
-#define XQSPIPS_IXR_TXFULL_MASK                0x00000008 /* QSPI TX FIFO is full */
-#define XQSPIPS_IXR_RXNEMTY_MASK       0x00000010 /* QSPI RX FIFO Not Empty */
-#define XQSPIPS_IXR_ALL_MASK           (XQSPIPS_IXR_TXNFULL_MASK | \
-                                       XQSPIPS_IXR_RXNEMTY_MASK)
+#define ZYNQ_QSPI_IXR_TXNFULL_MASK     0x00000004 /* QSPI TX FIFO Overflow */
+#define ZYNQ_QSPI_IXR_TXFULL_MASK      0x00000008 /* QSPI TX FIFO is full */
+#define ZYNQ_QSPI_IXR_RXNEMTY_MASK     0x00000010 /* QSPI RX FIFO Not Empty */
+#define ZYNQ_QSPI_IXR_ALL_MASK         (ZYNQ_QSPI_IXR_TXNFULL_MASK | \
+                                       ZYNQ_QSPI_IXR_RXNEMTY_MASK)
 
 /*
  * QSPI Enable Register bit Masks
  *
  * This register is used to enable or disable the QSPI controller
  */
-#define XQSPIPS_ENABLE_ENABLE_MASK     0x00000001 /* QSPI Enable Bit Mask */
+#define ZYNQ_QSPI_ENABLE_ENABLE_MASK   0x00000001 /* QSPI Enable Bit Mask */
 
 /*
  * QSPI Linear Configuration Register
  * It is named Linear Configuration but it controls other modes when not in
  * linear mode also.
  */
-#define XQSPIPS_LCFG_TWO_MEM_MASK      0x40000000 /* QSPI Enable Bit Mask */
-#define XQSPIPS_LCFG_SEP_BUS_MASK      0x20000000 /* QSPI Enable Bit Mask */
-#define XQSPIPS_LCFG_U_PAGE            0x10000000 /* QSPI Upper memory set */
+#define ZYNQ_QSPI_LCFG_TWO_MEM_MASK    0x40000000 /* QSPI Enable Bit Mask */
+#define ZYNQ_QSPI_LCFG_SEP_BUS_MASK    0x20000000 /* QSPI Enable Bit Mask */
+#define ZYNQ_QSPI_LCFG_U_PAGE          0x10000000 /* QSPI Upper memory set */
 
-#define XQSPIPS_LCFG_DUMMY_SHIFT       8
+#define ZYNQ_QSPI_LCFG_DUMMY_SHIFT     8
 
-#define XQSPIPS_FAST_READ_QOUT_CODE    0x6B    /* read instruction code */
+#define ZYNQ_QSPI_FR_QOUT_CODE 0x6B    /* read instruction code */
 
 /*
  * The modebits configurable by the driver to make the SPI support different
 #define MODEBITS                       (SPI_CPOL | SPI_CPHA)
 
 /* Definitions for the status of queue */
-#define XQSPIPS_QUEUE_STOPPED          0
-#define XQSPIPS_QUEUE_RUNNING          1
+#define ZYNQ_QSPI_QUEUE_STOPPED                0
+#define ZYNQ_QSPI_QUEUE_RUNNING                1
 
 /* QSPI MIO's count for different connection topologies */
-#define XQSPIPS_MIO_NUM_QSPI0          6
-#define XQSPIPS_MIO_NUM_QSPI1          5
-#define XQSPIPS_MIO_NUM_QSPI1_CS       1
+#define ZYNQ_QSPI_MIO_NUM_QSPI0                6
+#define ZYNQ_QSPI_MIO_NUM_QSPI1                5
+#define ZYNQ_QSPI_MIO_NUM_QSPI1_CS     1
 
 /* Definitions of the flash commands - Flash opcodes in ascending order */
-#define XQSPIPS_FLASH_OPCODE_WRSR      0x01    /* Write status register */
-#define XQSPIPS_FLASH_OPCODE_PP                0x02    /* Page program */
-#define XQSPIPS_FLASH_OPCODE_NORM_READ 0x03    /* Normal read data bytes */
-#define XQSPIPS_FLASH_OPCODE_WRDS      0x04    /* Write disable */
-#define XQSPIPS_FLASH_OPCODE_RDSR1     0x05    /* Read status register 1 */
-#define XQSPIPS_FLASH_OPCODE_WREN      0x06    /* Write enable */
-#define XQSPIPS_FLASH_OPCODE_FAST_READ 0x0B    /* Fast read data bytes */
-#define XQSPIPS_FLASH_OPCODE_BRRD      0x16    /* Bank address reg read */
-#define XQSPIPS_FLASH_OPCODE_BRWR      0x17    /* Bank address reg write */
-#define XQSPIPS_FLASH_OPCODE_BE_4K     0x20    /* Erase 4KiB block */
-#define XQSPIPS_FLASH_OPCODE_RDSR2     0x35    /* Read status register 2 */
-#define XQSPIPS_FLASH_OPCODE_DUAL_READ 0x3B    /* Dual read data bytes */
-#define XQSPIPS_FLASH_OPCODE_BE_32K    0x52    /* Erase 32KiB block */
-#define XQSPIPS_FLASH_OPCODE_QUAD_READ 0x6B    /* Quad read data bytes */
-#define XQSPIPS_FLASH_OPCODE_ERASE_SUS 0x75    /* Erase suspend */
-#define XQSPIPS_FLASH_OPCODE_ERASE_RES 0x7A    /* Erase resume */
-#define XQSPIPS_FLASH_OPCODE_RDID      0x9F    /* Read JEDEC ID */
-#define XQSPIPS_FLASH_OPCODE_WREAR     0xC5    /* Extended address reg write */
-#define XQSPIPS_FLASH_OPCODE_RDEAR     0xC8    /* Extended address reg read */
-#define XQSPIPS_FLASH_OPCODE_BE                0xC7    /* Erase whole flash block */
-#define XQSPIPS_FLASH_OPCODE_SE                0xD8    /* Sector erase (usually 64KB)*/
+#define ZYNQ_QSPI_FLASH_OPCODE_WRSR    0x01    /* Write status register */
+#define ZYNQ_QSPI_FLASH_OPCODE_PP      0x02    /* Page program */
+#define ZYNQ_QSPI_FLASH_OPCODE_NR      0x03    /* Normal read data bytes */
+#define ZYNQ_QSPI_FLASH_OPCODE_WRDS    0x04    /* Write disable */
+#define ZYNQ_QSPI_FLASH_OPCODE_RDSR1   0x05    /* Read status register 1 */
+#define ZYNQ_QSPI_FLASH_OPCODE_WREN    0x06    /* Write enable */
+#define ZYNQ_QSPI_FLASH_OPCODE_FR      0x0B    /* Fast read data bytes */
+#define ZYNQ_QSPI_FLASH_OPCODE_BRRD    0x16    /* Bank address reg read */
+#define ZYNQ_QSPI_FLASH_OPCODE_BRWR    0x17    /* Bank address reg write */
+#define ZYNQ_QSPI_FLASH_OPCODE_BE_4K   0x20    /* Erase 4KiB block */
+#define ZYNQ_QSPI_FLASH_OPCODE_RDSR2   0x35    /* Read status register 2 */
+#define ZYNQ_QSPI_FLASH_OPCODE_DR      0x3B    /* Dual read data bytes */
+#define ZYNQ_QSPI_FLASH_OPCODE_BE_32K  0x52    /* Erase 32KiB block */
+#define ZYNQ_QSPI_FLASH_OPCODE_QR      0x6B    /* Quad read data bytes */
+#define ZYNQ_QSPI_FLASH_OPCODE_ES      0x75    /* Erase suspend */
+#define ZYNQ_QSPI_FLASH_OPCODE_ER      0x7A    /* Erase resume */
+#define ZYNQ_QSPI_FLASH_OPCODE_RDID    0x9F    /* Read JEDEC ID */
+#define ZYNQ_QSPI_FLASH_OPCODE_WREAR   0xC5    /* Extended address reg write */
+#define ZYNQ_QSPI_FLASH_OPCODE_RDEAR   0xC8    /* Extended address reg read */
+#define ZYNQ_QSPI_FLASH_OPCODE_BE      0xC7    /* Erase whole flash block */
+#define ZYNQ_QSPI_FLASH_OPCODE_SE      0xD8    /* Sector erase (usually 64KB)*/
 
 /* Few mtd flash functions */
 extern int spi_flash_cmd(struct spi_slave *spi, u8 cmd,
@@ -117,7 +117,7 @@ extern int spi_flash_cmd_read(struct spi_slave *spi, const u8 *cmd,
                size_t cmd_len, void *data, size_t data_len);
 
 /* QSPI register offsets */
-struct xqspips_regs {
+struct zynq_qspi_regs {
        u32 confr;      /* 0x00 */
        u32 isr;        /* 0x04 */
        u32 ier;        /* 0x08 */
@@ -141,16 +141,16 @@ struct xqspips_regs {
        u32 midr;       /* 0xFC */
 };
 
-#define xqspips_base ((struct xqspips_regs *)ZYNQ_QSPI_BASEADDR)
+#define zynq_qspi_base ((struct zynq_qspi_regs *)ZYNQ_QSPI_BASEADDR)
 
-struct xqspips {
+struct zynq_qspi {
        u32 input_clk_hz;
        u32 speed_hz;
        const void *txbuf;
        void *rxbuf;
        int bytes_to_transfer;
        int bytes_to_receive;
-       struct xqspips_inst_format *curr_inst;
+       struct zynq_qspi_inst_format *curr_inst;
        u8 inst_response;
        unsigned int is_inst;
        unsigned int is_dual;
@@ -158,7 +158,7 @@ struct xqspips {
 };
 
 struct spi_device {
-       struct xqspips master;
+       struct zynq_qspi master;
        u32 max_speed_hz;
        u8 chip_select;
        u8 mode;
@@ -175,52 +175,52 @@ struct spi_transfer {
        u32 speed_hz;
 };
 
-struct zynq_spi_slave {
+struct zynq_qspi_slave {
        struct spi_slave slave;
        struct spi_device qspi;
 };
-#define to_zynq_spi_slave(s) container_of(s, struct zynq_spi_slave, slave)
+#define to_zynq_qspi_slave(s) container_of(s, struct zynq_qspi_slave, slave)
 
 /*
- * struct xqspips_inst_format - Defines qspi flash instruction format
+ * struct zynq_qspi_inst_format - Defines qspi flash instruction format
  * @opcode:            Operational code of instruction
  * @inst_size:         Size of the instruction including address bytes
  * @offset:            Register address where instruction has to be written
  */
-struct xqspips_inst_format {
+struct zynq_qspi_inst_format {
        u8 opcode;
        u8 inst_size;
        u8 offset;
 };
 
 /* List of all the QSPI instructions and its format */
-static struct xqspips_inst_format flash_inst[] = {
-       { XQSPIPS_FLASH_OPCODE_WREN, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_WRDS, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_RDSR1, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_RDSR2, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_WRSR, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_PP, 4, XQSPIPS_TXD_00_00_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_SE, 4, XQSPIPS_TXD_00_00_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_BE_32K, 4, XQSPIPS_TXD_00_00_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_BE_4K, 4, XQSPIPS_TXD_00_00_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_BE, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_ERASE_SUS, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_ERASE_RES, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_RDID, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_NORM_READ, 4, XQSPIPS_TXD_00_00_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_FAST_READ, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_DUAL_READ, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_QUAD_READ, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_BRWR, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_BRRD, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_WREAR, 1, XQSPIPS_TXD_00_01_OFFSET },
-       { XQSPIPS_FLASH_OPCODE_RDEAR, 1, XQSPIPS_TXD_00_01_OFFSET },
+static struct zynq_qspi_inst_format flash_inst[] = {
+       { ZYNQ_QSPI_FLASH_OPCODE_WREN, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_WRDS, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_RDSR1, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_RDSR2, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_WRSR, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_PP, 4, ZYNQ_QSPI_TXD_00_00_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_SE, 4, ZYNQ_QSPI_TXD_00_00_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_BE_32K, 4, ZYNQ_QSPI_TXD_00_00_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_BE_4K, 4, ZYNQ_QSPI_TXD_00_00_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_BE, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_ES, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_ER, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_RDID, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_NR, 4, ZYNQ_QSPI_TXD_00_00_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_FR, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_DR, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_QR, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_BRWR, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_BRRD, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_WREAR, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
+       { ZYNQ_QSPI_FLASH_OPCODE_RDEAR, 1, ZYNQ_QSPI_TXD_00_01_OFFSET },
        /* Add all the instructions supported by the flash device */
 };
 
 /*
- * xqspips_init_hw - Initialize the hardware
+ * zynq_qspi_init_hw - Initialize the hardware
  * @is_dual:           Indicates whether dual memories are used
  * @cs:                        Indicates which chip select is used in dual stacked
  *
@@ -240,121 +240,122 @@ static struct xqspips_inst_format flash_inst[] = {
  *     - Set the little endian mode of TX FIFO and
  *     - Enable the QSPI controller
  */
-static void xqspips_init_hw(int is_dual, unsigned int cs)
+static void zynq_qspi_init_hw(int is_dual, unsigned int cs)
 {
        u32 config_reg;
 
-       writel(~XQSPIPS_ENABLE_ENABLE_MASK, &xqspips_base->enbr);
-       writel(0x7F, &xqspips_base->idisr);
+       writel(~ZYNQ_QSPI_ENABLE_ENABLE_MASK, &zynq_qspi_base->enbr);
+       writel(0x7F, &zynq_qspi_base->idisr);
 
        /* Disable linear mode as the boot loader may have used it */
-       writel(0x0, &xqspips_base->lcr);
+       writel(0x0, &zynq_qspi_base->lcr);
 
        /* Clear the RX FIFO */
-       while (readl(&xqspips_base->isr) & XQSPIPS_IXR_RXNEMTY_MASK)
-               readl(&xqspips_base->drxr);
+       while (readl(&zynq_qspi_base->isr) & ZYNQ_QSPI_IXR_RXNEMTY_MASK)
+               readl(&zynq_qspi_base->drxr);
 
-       writel(0x7F, &xqspips_base->isr);
-       config_reg = readl(&xqspips_base->confr);
+       writel(0x7F, &zynq_qspi_base->isr);
+       config_reg = readl(&zynq_qspi_base->confr);
        config_reg &= 0xFBFFFFFF; /* Set little endian mode of TX FIFO */
        config_reg |= 0x8000FCC1;
        if (is_dual == MODE_DUAL_STACKED)
                config_reg |= 0x10;
-       writel(config_reg, &xqspips_base->confr);
+       writel(config_reg, &zynq_qspi_base->confr);
 
        if (is_dual == MODE_DUAL_PARALLEL)
                /* Enable two memories on seperate buses */
-               writel((XQSPIPS_LCFG_TWO_MEM_MASK |
-                       XQSPIPS_LCFG_SEP_BUS_MASK |
-                       (1 << XQSPIPS_LCFG_DUMMY_SHIFT) |
-                       XQSPIPS_FAST_READ_QOUT_CODE),
-                       &xqspips_base->lcr);
+               writel((ZYNQ_QSPI_LCFG_TWO_MEM_MASK |
+                       ZYNQ_QSPI_LCFG_SEP_BUS_MASK |
+                       (1 << ZYNQ_QSPI_LCFG_DUMMY_SHIFT) |
+                       ZYNQ_QSPI_FR_QOUT_CODE),
+                       &zynq_qspi_base->lcr);
        else if (is_dual == MODE_DUAL_STACKED)
                /* Configure two memories on shared bus by enabling lower mem */
-               writel((XQSPIPS_LCFG_TWO_MEM_MASK |
-                       (1 << XQSPIPS_LCFG_DUMMY_SHIFT) |
-                       XQSPIPS_FAST_READ_QOUT_CODE),
-                       &xqspips_base->lcr);
+               writel((ZYNQ_QSPI_LCFG_TWO_MEM_MASK |
+                       (1 << ZYNQ_QSPI_LCFG_DUMMY_SHIFT) |
+                       ZYNQ_QSPI_FR_QOUT_CODE),
+                       &zynq_qspi_base->lcr);
 
-       writel(XQSPIPS_ENABLE_ENABLE_MASK, &xqspips_base->enbr);
+       writel(ZYNQ_QSPI_ENABLE_ENABLE_MASK, &zynq_qspi_base->enbr);
 }
 
 /*
- * xqspips_copy_read_data - Copy data to RX buffer
- * @xqspi:     Pointer to the xqspips structure
+ * zynq_qspi_copy_read_data - Copy data to RX buffer
+ * @zqspi:     Pointer to the zynq_qspi structure
  * @data:      The 32 bit variable where data is stored
  * @size:      Number of bytes to be copied from data to RX buffer
  */
-static void xqspips_copy_read_data(struct xqspips *xqspi, u32 data, u8 size)
+static void zynq_qspi_copy_read_data(struct zynq_qspi *zqspi, u32 data, u8 size)
 {
        u8 byte3;
 
        debug("%s: data 0x%04x rxbuf addr: 0x%08x size %d\n", __func__ ,
-             data, (unsigned)(xqspi->rxbuf), size);
+             data, (unsigned)(zqspi->rxbuf), size);
 
-       if (xqspi->rxbuf) {
+       if (zqspi->rxbuf) {
                switch (size) {
                case 1:
-                       *((u8 *)xqspi->rxbuf) = data;
-                       xqspi->rxbuf += 1;
+                       *((u8 *)zqspi->rxbuf) = data;
+                       zqspi->rxbuf += 1;
                        break;
                case 2:
-                       *((u16 *)xqspi->rxbuf) = data;
-                       xqspi->rxbuf += 2;
+                       *((u16 *)zqspi->rxbuf) = data;
+                       zqspi->rxbuf += 2;
                        break;
                case 3:
-                       *((u16 *)xqspi->rxbuf) = data;
-                       xqspi->rxbuf += 2;
+                       *((u16 *)zqspi->rxbuf) = data;
+                       zqspi->rxbuf += 2;
                        byte3 = (u8)(data >> 16);
-                       *((u8 *)xqspi->rxbuf) = byte3;
-                       xqspi->rxbuf += 1;
+                       *((u8 *)zqspi->rxbuf) = byte3;
+                       zqspi->rxbuf += 1;
                        break;
                case 4:
                        /* Can not assume word aligned buffer */
-                       memcpy(xqspi->rxbuf, &data, size);
-                       xqspi->rxbuf += 4;
+                       memcpy(zqspi->rxbuf, &data, size);
+                       zqspi->rxbuf += 4;
                        break;
                default:
                        /* This will never execute */
                        break;
                }
        }
-       xqspi->bytes_to_receive -= size;
-       if (xqspi->bytes_to_receive < 0)
-               xqspi->bytes_to_receive = 0;
+       zqspi->bytes_to_receive -= size;
+       if (zqspi->bytes_to_receive < 0)
+               zqspi->bytes_to_receive = 0;
 }
 
 /*
- * xqspips_copy_write_data - Copy data from TX buffer
- * @xqspi:     Pointer to the xqspips structure
+ * zynq_qspi_copy_write_data - Copy data from TX buffer
+ * @zqspi:     Pointer to the zynq_qspi structure
  * @data:      Pointer to the 32 bit variable where data is to be copied
  * @size:      Number of bytes to be copied from TX buffer to data
  */
-static void xqspips_copy_write_data(struct xqspips *xqspi, u32 *data, u8 size)
+static void zynq_qspi_copy_write_data(struct zynq_qspi *zqspi,
+               u32 *data, u8 size)
 {
-       if (xqspi->txbuf) {
+       if (zqspi->txbuf) {
                switch (size) {
                case 1:
-                       *data = *((u8 *)xqspi->txbuf);
-                       xqspi->txbuf += 1;
+                       *data = *((u8 *)zqspi->txbuf);
+                       zqspi->txbuf += 1;
                        *data |= 0xFFFFFF00;
                        break;
                case 2:
-                       *data = *((u16 *)xqspi->txbuf);
-                       xqspi->txbuf += 2;
+                       *data = *((u16 *)zqspi->txbuf);
+                       zqspi->txbuf += 2;
                        *data |= 0xFFFF0000;
                        break;
                case 3:
-                       *data = *((u16 *)xqspi->txbuf);
-                       xqspi->txbuf += 2;
-                       *data |= (*((u8 *)xqspi->txbuf) << 16);
-                       xqspi->txbuf += 1;
+                       *data = *((u16 *)zqspi->txbuf);
+                       zqspi->txbuf += 2;
+                       *data |= (*((u8 *)zqspi->txbuf) << 16);
+                       zqspi->txbuf += 1;
                        *data |= 0xFF000000;
                        break;
                case 4:
                        /* Can not assume word aligned buffer */
-                       memcpy(data, xqspi->txbuf, size);
-                       xqspi->txbuf += 4;
+                       memcpy(data, zqspi->txbuf, size);
+                       zqspi->txbuf += 4;
                        break;
                default:
                        /* This will never execute */
@@ -365,40 +366,40 @@ static void xqspips_copy_write_data(struct xqspips *xqspi, u32 *data, u8 size)
        }
 
        debug("%s: data 0x%08x txbuf addr: 0x%08x size %d\n", __func__,
-             *data, (u32)xqspi->txbuf, size);
+             *data, (u32)zqspi->txbuf, size);
 
-       xqspi->bytes_to_transfer -= size;
-       if (xqspi->bytes_to_transfer < 0)
-               xqspi->bytes_to_transfer = 0;
+       zqspi->bytes_to_transfer -= size;
+       if (zqspi->bytes_to_transfer < 0)
+               zqspi->bytes_to_transfer = 0;
 }
 
 /*
- * xqspips_chipselect - Select or deselect the chip select line
+ * zynq_qspi_chipselect - Select or deselect the chip select line
  * @qspi:      Pointer to the spi_device structure
  * @is_on:     Select(1) or deselect (0) the chip select line
  */
-static void xqspips_chipselect(struct spi_device *qspi, int is_on)
+static void zynq_qspi_chipselect(struct spi_device *qspi, int is_on)
 {
        u32 config_reg;
 
        debug("%s: is_on: %d\n", __func__, is_on);
 
-       config_reg = readl(&xqspips_base->confr);
+       config_reg = readl(&zynq_qspi_base->confr);
 
        if (is_on) {
                /* Select the slave */
-               config_reg &= ~XQSPIPS_CONFIG_SSCTRL_MASK;
+               config_reg &= ~ZYNQ_QSPI_CONFIG_SSCTRL_MASK;
                config_reg |= (((~(0x0001 << qspi->chip_select)) << 10) &
-                               XQSPIPS_CONFIG_SSCTRL_MASK);
+                               ZYNQ_QSPI_CONFIG_SSCTRL_MASK);
        } else
                /* Deselect the slave */
-               config_reg |= XQSPIPS_CONFIG_SSCTRL_MASK;
+               config_reg |= ZYNQ_QSPI_CONFIG_SSCTRL_MASK;
 
-       writel(config_reg, &xqspips_base->confr);
+       writel(config_reg, &zynq_qspi_base->confr);
 }
 
 /*
- * xqspips_setup_transfer - Configure QSPI controller for specified transfer
+ * zynq_qspi_setup_transfer - Configure QSPI controller for specified transfer
  * @qspi:      Pointer to the spi_device structure
  * @transfer:  Pointer to the spi_transfer structure which provides information
  *             about next transfer setup parameters
@@ -415,10 +416,10 @@ static void xqspips_chipselect(struct spi_device *qspi, int is_on)
  * controller the driver will set the highest or lowest frequency supported by
  * controller.
  */
-static int xqspips_setup_transfer(struct spi_device *qspi,
+static int zynq_qspi_setup_transfer(struct spi_device *qspi,
                struct spi_transfer *transfer)
 {
-       struct xqspips *xqspi = &qspi->master;
+       struct zynq_qspi *zqspi = &qspi->master;
        u8 bits_per_word;
        u32 config_reg;
        u32 req_hz;
@@ -440,71 +441,71 @@ static int xqspips_setup_transfer(struct spi_device *qspi,
        if (bits_per_word != 32)
                bits_per_word = 32;
 
-       config_reg = readl(&xqspips_base->confr);
+       config_reg = readl(&zynq_qspi_base->confr);
 
        /* Set the QSPI clock phase and clock polarity */
-       config_reg &= (~XQSPIPS_CONFIG_CPHA_MASK) &
-                               (~XQSPIPS_CONFIG_CPOL_MASK);
+       config_reg &= (~ZYNQ_QSPI_CONFIG_CPHA_MASK) &
+                               (~ZYNQ_QSPI_CONFIG_CPOL_MASK);
        if (qspi->mode & SPI_CPHA)
-               config_reg |= XQSPIPS_CONFIG_CPHA_MASK;
+               config_reg |= ZYNQ_QSPI_CONFIG_CPHA_MASK;
        if (qspi->mode & SPI_CPOL)
-               config_reg |= XQSPIPS_CONFIG_CPOL_MASK;
+               config_reg |= ZYNQ_QSPI_CONFIG_CPOL_MASK;
 
        /* Set the clock frequency */
-       if (xqspi->speed_hz != req_hz) {
+       if (zqspi->speed_hz != req_hz) {
                baud_rate_val = 0;
                while ((baud_rate_val < 8) &&
-                       (xqspi->input_clk_hz / (2 << baud_rate_val)) > req_hz) {
+                       (zqspi->input_clk_hz / (2 << baud_rate_val)) > req_hz) {
                                baud_rate_val++;
                }
                config_reg &= 0xFFFFFFC7;
                config_reg |= (baud_rate_val << 3);
-               xqspi->speed_hz = req_hz;
+               zqspi->speed_hz = req_hz;
        }
 
-       writel(config_reg, &xqspips_base->confr);
+       writel(config_reg, &zynq_qspi_base->confr);
 
        debug("%s: mode %d, %u bits/w, %u clock speed\n", __func__,
-             qspi->mode & MODEBITS, qspi->bits_per_word, xqspi->speed_hz);
+             qspi->mode & MODEBITS, qspi->bits_per_word, zqspi->speed_hz);
 
        return 0;
 }
 
 /*
- * xqspips_fill_tx_fifo - Fills the TX FIFO with as many bytes as possible
- * @xqspi:     Pointer to the xqspips structure
+ * zynq_qspi_fill_tx_fifo - Fills the TX FIFO with as many bytes as possible
+ * @zqspi:     Pointer to the zynq_qspi structure
  */
-static void xqspips_fill_tx_fifo(struct xqspips *xqspi)
+static void zynq_qspi_fill_tx_fifo(struct zynq_qspi *zqspi)
 {
        u32 data = 0;
        unsigned len, offset;
        static const unsigned offsets[4] = {
-               XQSPIPS_TXD_00_00_OFFSET, XQSPIPS_TXD_00_01_OFFSET,
-               XQSPIPS_TXD_00_10_OFFSET, XQSPIPS_TXD_00_11_OFFSET };
+               ZYNQ_QSPI_TXD_00_00_OFFSET, ZYNQ_QSPI_TXD_00_01_OFFSET,
+               ZYNQ_QSPI_TXD_00_10_OFFSET, ZYNQ_QSPI_TXD_00_11_OFFSET };
 
-       while ((!(readl(&xqspips_base->isr) &
-                       XQSPIPS_IXR_TXFULL_MASK)) &&
-                       (xqspi->bytes_to_transfer > 0)) {
-               if (xqspi->bytes_to_transfer < 4) {
+       while ((!(readl(&zynq_qspi_base->isr) &
+                       ZYNQ_QSPI_IXR_TXFULL_MASK)) &&
+                       (zqspi->bytes_to_transfer > 0)) {
+               if (zqspi->bytes_to_transfer < 4) {
                        /* Write TXD1, TXD2, TXD3 only if TxFIFO is empty. */
-                       if (!(readl(&xqspips_base->isr)
-                                       & XQSPIPS_IXR_TXNFULL_MASK) &&
-                                       !xqspi->rxbuf)
+                       if (!(readl(&zynq_qspi_base->isr)
+                                       & ZYNQ_QSPI_IXR_TXNFULL_MASK) &&
+                                       !zqspi->rxbuf)
                                return;
-                       len = xqspi->bytes_to_transfer;
-                       xqspips_copy_write_data(xqspi, &data, len);
-                       offset = (xqspi->rxbuf) ? offsets[0] : offsets[len];
-                       writel(data, &xqspips_base->confr + (offset / 4));
+                       len = zqspi->bytes_to_transfer;
+                       zynq_qspi_copy_write_data(zqspi, &data, len);
+                       offset = (zqspi->rxbuf) ? offsets[0] : offsets[len];
+                       writel(data, &zynq_qspi_base->confr + (offset / 4));
                } else {
-                       xqspips_copy_write_data(xqspi, &data, 4);
-                       writel(data, &xqspips_base->txd0r);
+                       zynq_qspi_copy_write_data(zqspi, &data, 4);
+                       writel(data, &zynq_qspi_base->txd0r);
                }
        }
 }
 
 /*
- * xqspips_irq_poll - Interrupt service routine of the QSPI controller
- * @xqspi:     Pointer to the xqspips structure
+ * zynq_qspi_irq_poll - Interrupt service routine of the QSPI controller
+ * @zqspi:     Pointer to the zynq_qspi structure
  *
  * This function handles TX empty and Mode Fault interrupts only.
  * On TX empty interrupt this function reads the received data from RX FIFO and
@@ -516,17 +517,17 @@ static void xqspips_fill_tx_fifo(struct xqspips *xqspi)
  * returns:    0 for poll timeout
  *             1 transfer operation complete
  */
-static int xqspips_irq_poll(struct xqspips *xqspi)
+static int zynq_qspi_irq_poll(struct zynq_qspi *zqspi)
 {
        int max_loop;
        u32 intr_status;
 
-       debug("%s: xqspi: 0x%08x\n", __func__, (u32)xqspi);
+       debug("%s: zqspi: 0x%08x\n", __func__, (u32)zqspi);
 
        /* Poll until any of the interrupt status bits are set */
        max_loop = 0;
        do {
-               intr_status = readl(&xqspips_base->isr);
+               intr_status = readl(&zynq_qspi_base->isr);
                max_loop++;
        } while ((intr_status == 0) && (max_loop < 100000));
 
@@ -535,12 +536,12 @@ static int xqspips_irq_poll(struct xqspips *xqspi)
                return 0;
        }
 
-       writel(intr_status, &xqspips_base->isr);
+       writel(intr_status, &zynq_qspi_base->isr);
 
        /* Disable all interrupts */
-       writel(XQSPIPS_IXR_ALL_MASK, &xqspips_base->idisr);
-       if ((intr_status & XQSPIPS_IXR_TXNFULL_MASK) ||
-           (intr_status & XQSPIPS_IXR_RXNEMTY_MASK)) {
+       writel(ZYNQ_QSPI_IXR_ALL_MASK, &zynq_qspi_base->idisr);
+       if ((intr_status & ZYNQ_QSPI_IXR_TXNFULL_MASK) ||
+           (intr_status & ZYNQ_QSPI_IXR_RXNEMTY_MASK)) {
                /*
                 * This bit is set when Tx FIFO has < THRESHOLD entries. We have
                 * the THRESHOLD value set to 1, so this bit indicates Tx FIFO
@@ -549,47 +550,47 @@ static int xqspips_irq_poll(struct xqspips *xqspi)
                u32 config_reg;
 
                /* Read out the data from the RX FIFO */
-               while (readl(&xqspips_base->isr) &
-                               XQSPIPS_IXR_RXNEMTY_MASK) {
+               while (readl(&zynq_qspi_base->isr) &
+                               ZYNQ_QSPI_IXR_RXNEMTY_MASK) {
                        u32 data;
 
-                       data = readl(&xqspips_base->drxr);
-
-                       if ((xqspi->inst_response) &&
-                           (!((xqspi->curr_inst->opcode ==
-                               XQSPIPS_FLASH_OPCODE_RDSR1) ||
-                              (xqspi->curr_inst->opcode ==
-                               XQSPIPS_FLASH_OPCODE_RDSR2)))) {
-                               xqspi->inst_response = 0;
-                               xqspips_copy_read_data(xqspi, data,
-                                               xqspi->curr_inst->inst_size);
-                       } else if (xqspi->bytes_to_receive < 4) {
-                               xqspips_copy_read_data(xqspi, data,
-                                                      xqspi->bytes_to_receive);
+                       data = readl(&zynq_qspi_base->drxr);
+
+                       if ((zqspi->inst_response) &&
+                           (!((zqspi->curr_inst->opcode ==
+                               ZYNQ_QSPI_FLASH_OPCODE_RDSR1) ||
+                              (zqspi->curr_inst->opcode ==
+                               ZYNQ_QSPI_FLASH_OPCODE_RDSR2)))) {
+                               zqspi->inst_response = 0;
+                               zynq_qspi_copy_read_data(zqspi, data,
+                                               zqspi->curr_inst->inst_size);
+                       } else if (zqspi->bytes_to_receive < 4) {
+                               zynq_qspi_copy_read_data(zqspi, data,
+                                                      zqspi->bytes_to_receive);
                        } else {
-                               xqspips_copy_read_data(xqspi, data, 4);
+                               zynq_qspi_copy_read_data(zqspi, data, 4);
                        }
                }
 
-               if (xqspi->bytes_to_transfer) {
+               if (zqspi->bytes_to_transfer) {
                        /* There is more data to send */
-                       xqspips_fill_tx_fifo(xqspi);
+                       zynq_qspi_fill_tx_fifo(zqspi);
 
-                       writel(XQSPIPS_IXR_ALL_MASK, &xqspips_base->ier);
+                       writel(ZYNQ_QSPI_IXR_ALL_MASK, &zynq_qspi_base->ier);
 
-                       config_reg = readl(&xqspips_base->confr);
+                       config_reg = readl(&zynq_qspi_base->confr);
 
-                       config_reg |= XQSPIPS_CONFIG_MANSRT_MASK;
-                       writel(config_reg, &xqspips_base->confr);
+                       config_reg |= ZYNQ_QSPI_CONFIG_MANSRT_MASK;
+                       writel(config_reg, &zynq_qspi_base->confr);
                } else {
                        /*
                         * If transfer and receive is completed then only send
                         * complete signal
                         */
-                       if (!xqspi->bytes_to_receive) {
+                       if (!zqspi->bytes_to_receive) {
                                /* return operation complete */
-                               writel(XQSPIPS_IXR_ALL_MASK,
-                                      &xqspips_base->idisr);
+                               writel(ZYNQ_QSPI_IXR_ALL_MASK,
+                                      &zynq_qspi_base->idisr);
                                return 1;
                        }
                }
@@ -599,7 +600,7 @@ static int xqspips_irq_poll(struct xqspips *xqspi)
 }
 
 /*
- * xqspips_start_transfer - Initiates the QSPI transfer
+ * zynq_qspi_start_transfer - Initiates the QSPI transfer
  * @qspi:      Pointer to the spi_device structure
  * @transfer:  Pointer to the spi_transfer structure which provide information
  *             about next transfer parameters
@@ -609,10 +610,10 @@ static int xqspips_irq_poll(struct xqspips *xqspi)
  *
  * returns:    Number of bytes transferred in the last transfer
  */
-static int xqspips_start_transfer(struct spi_device *qspi,
+static int zynq_qspi_start_transfer(struct spi_device *qspi,
                        struct spi_transfer *transfer)
 {
-       struct xqspips *xqspi = &qspi->master;
+       struct zynq_qspi *zqspi = &qspi->master;
        static u8 current_u_page;
        u32 config_reg;
        u32 data = 0;
@@ -622,15 +623,15 @@ static int xqspips_start_transfer(struct spi_device *qspi,
        debug("%s: qspi: 0x%08x transfer: 0x%08x len: %d\n", __func__,
              (u32)qspi, (u32)transfer, transfer->len);
 
-       xqspi->txbuf = transfer->tx_buf;
-       xqspi->rxbuf = transfer->rx_buf;
-       xqspi->bytes_to_transfer = transfer->len;
-       xqspi->bytes_to_receive = transfer->len;
+       zqspi->txbuf = transfer->tx_buf;
+       zqspi->rxbuf = transfer->rx_buf;
+       zqspi->bytes_to_transfer = transfer->len;
+       zqspi->bytes_to_receive = transfer->len;
 
-       if (xqspi->txbuf)
-               instruction = *(u8 *)xqspi->txbuf;
+       if (zqspi->txbuf)
+               instruction = *(u8 *)zqspi->txbuf;
 
-       if (instruction && xqspi->is_inst) {
+       if (instruction && zqspi->is_inst) {
                for (index = 0; index < ARRAY_SIZE(flash_inst); index++)
                        if (instruction == flash_inst[index].opcode)
                                break;
@@ -642,37 +643,37 @@ static int xqspips_start_transfer(struct spi_device *qspi,
                if (index == ARRAY_SIZE(flash_inst))
                        goto xfer_data;
 
-               xqspi->curr_inst = &flash_inst[index];
-               xqspi->inst_response = 1;
+               zqspi->curr_inst = &flash_inst[index];
+               zqspi->inst_response = 1;
 
-               if ((xqspi->is_dual == MODE_DUAL_STACKED) &&
-                               (current_u_page != xqspi->u_page)) {
-                       if (xqspi->u_page) {
+               if ((zqspi->is_dual == MODE_DUAL_STACKED) &&
+                               (current_u_page != zqspi->u_page)) {
+                       if (zqspi->u_page) {
                                /* Configure two memories on shared bus
                                 * by enabling upper mem
                                 */
-                               writel((XQSPIPS_LCFG_TWO_MEM_MASK |
-                                       XQSPIPS_LCFG_U_PAGE |
-                                       (1 << XQSPIPS_LCFG_DUMMY_SHIFT) |
-                                       XQSPIPS_FAST_READ_QOUT_CODE),
-                                       &xqspips_base->lcr);
+                               writel((ZYNQ_QSPI_LCFG_TWO_MEM_MASK |
+                                       ZYNQ_QSPI_LCFG_U_PAGE |
+                                       (1 << ZYNQ_QSPI_LCFG_DUMMY_SHIFT) |
+                                       ZYNQ_QSPI_FR_QOUT_CODE),
+                                       &zynq_qspi_base->lcr);
                        } else {
                                /* Configure two memories on shared bus
                                 * by enabling lower mem
                                 */
-                               writel((XQSPIPS_LCFG_TWO_MEM_MASK |
-                                       (1 << XQSPIPS_LCFG_DUMMY_SHIFT) |
-                                       XQSPIPS_FAST_READ_QOUT_CODE),
-                                       &xqspips_base->lcr);
+                               writel((ZYNQ_QSPI_LCFG_TWO_MEM_MASK |
+                                       (1 << ZYNQ_QSPI_LCFG_DUMMY_SHIFT) |
+                                       ZYNQ_QSPI_FR_QOUT_CODE),
+                                       &zynq_qspi_base->lcr);
                        }
 
-                       current_u_page = xqspi->u_page;
+                       current_u_page = zqspi->u_page;
                }
 
                /* Get the instruction */
                data = 0;
-               xqspips_copy_write_data(xqspi, &data,
-                                       xqspi->curr_inst->inst_size);
+               zynq_qspi_copy_write_data(zqspi, &data,
+                                       zqspi->curr_inst->inst_size);
 
                /*
                 * Write the instruction to LSB of the FIFO. The core is
@@ -680,23 +681,23 @@ static int xqspips_start_transfer(struct spi_device *qspi,
                 * write FIFO is full before writing. However, write would be
                 * delayed if the user tries to write when write FIFO is full
                 */
-               writel(data, &xqspips_base->confr +
-                               (xqspi->curr_inst->offset / 4));
+               writel(data, &zynq_qspi_base->confr +
+                               (zqspi->curr_inst->offset / 4));
 
                /*
                 * Read status register and Read ID instructions don't require
                 * to ignore the extra bytes in response of instruction as
                 * response contains the value
                 */
-               if ((instruction == XQSPIPS_FLASH_OPCODE_RDSR1) ||
-                   (instruction == XQSPIPS_FLASH_OPCODE_RDSR2) ||
-                   (instruction == XQSPIPS_FLASH_OPCODE_RDID) ||
-                   (instruction == XQSPIPS_FLASH_OPCODE_BRRD) ||
-                   (instruction == XQSPIPS_FLASH_OPCODE_RDEAR)) {
-                       if (xqspi->bytes_to_transfer < 4)
-                               xqspi->bytes_to_transfer = 0;
+               if ((instruction == ZYNQ_QSPI_FLASH_OPCODE_RDSR1) ||
+                   (instruction == ZYNQ_QSPI_FLASH_OPCODE_RDSR2) ||
+                   (instruction == ZYNQ_QSPI_FLASH_OPCODE_RDID) ||
+                   (instruction == ZYNQ_QSPI_FLASH_OPCODE_BRRD) ||
+                   (instruction == ZYNQ_QSPI_FLASH_OPCODE_RDEAR)) {
+                       if (zqspi->bytes_to_transfer < 4)
+                               zqspi->bytes_to_transfer = 0;
                        else
-                               xqspi->bytes_to_transfer -= 3;
+                               zqspi->bytes_to_transfer -= 3;
                }
        }
 
@@ -706,30 +707,31 @@ xfer_data:
         * Address and dummy byte should be transmitted after instruction
         * is transmitted
         */
-       if (((xqspi->is_inst == 0) && (xqspi->bytes_to_transfer)) ||
-           ((xqspi->bytes_to_transfer) &&
-            (instruction != XQSPIPS_FLASH_OPCODE_FAST_READ) &&
-            (instruction != XQSPIPS_FLASH_OPCODE_DUAL_READ) &&
-            (instruction != XQSPIPS_FLASH_OPCODE_QUAD_READ)))
-               xqspips_fill_tx_fifo(xqspi);
-
-       writel(XQSPIPS_IXR_ALL_MASK, &xqspips_base->ier);
+       if (((zqspi->is_inst == 0) && (zqspi->bytes_to_transfer)) ||
+           ((zqspi->bytes_to_transfer) &&
+            (instruction != ZYNQ_QSPI_FLASH_OPCODE_FR) &&
+            (instruction != ZYNQ_QSPI_FLASH_OPCODE_DR) &&
+            (instruction != ZYNQ_QSPI_FLASH_OPCODE_QR)))
+               zynq_qspi_fill_tx_fifo(zqspi);
+
+       writel(ZYNQ_QSPI_IXR_ALL_MASK, &zynq_qspi_base->ier);
        /* Start the transfer by enabling manual start bit */
-       config_reg = readl(&xqspips_base->confr) | XQSPIPS_CONFIG_MANSRT_MASK;
-       writel(config_reg, &xqspips_base->confr);
+       config_reg = readl(&zynq_qspi_base->confr) |
+                       ZYNQ_QSPI_CONFIG_MANSRT_MASK;
+       writel(config_reg, &zynq_qspi_base->confr);
 
        /* wait for completion */
        do {
-               data = xqspips_irq_poll(xqspi);
+               data = zynq_qspi_irq_poll(zqspi);
        } while (data == 0);
 
-       return (transfer->len) - (xqspi->bytes_to_transfer);
+       return (transfer->len) - (zqspi->bytes_to_transfer);
 }
 
-static int xqspips_transfer(struct spi_device *qspi,
+static int zynq_qspi_transfer(struct spi_device *qspi,
                struct spi_transfer *transfer)
 {
-       struct xqspips *xqspi = &qspi->master;
+       struct zynq_qspi *zqspi = &qspi->master;
        unsigned cs_change = 1;
        int status = 0;
 
@@ -737,14 +739,14 @@ static int xqspips_transfer(struct spi_device *qspi,
 
        while (1) {
                if (transfer->bits_per_word || transfer->speed_hz) {
-                       status = xqspips_setup_transfer(qspi, transfer);
+                       status = zynq_qspi_setup_transfer(qspi, transfer);
                        if (status < 0)
                                break;
                }
 
                /* Select the chip if required */
                if (cs_change)
-                       xqspips_chipselect(qspi, 1);
+                       zynq_qspi_chipselect(qspi, 1);
 
                cs_change = transfer->cs_change;
 
@@ -755,8 +757,8 @@ static int xqspips_transfer(struct spi_device *qspi,
 
                /* Request the transfer */
                if (transfer->len) {
-                       status = xqspips_start_transfer(qspi, transfer);
-                       xqspi->is_inst = 0;
+                       status = zynq_qspi_start_transfer(qspi, transfer);
+                       zqspi->is_inst = 0;
                }
 
                if (status != transfer->len) {
@@ -771,18 +773,18 @@ static int xqspips_transfer(struct spi_device *qspi,
 
                if (cs_change)
                        /* Deselect the chip */
-                       xqspips_chipselect(qspi, 0);
+                       zynq_qspi_chipselect(qspi, 0);
 
                break;
        }
 
-       xqspips_setup_transfer(qspi, NULL);
+       zynq_qspi_setup_transfer(qspi, NULL);
 
        return 0;
 }
 
 /*
- * xqspips_check_is_dual_flash - checking for dual or single qspi
+ * zynq_qspi_check_is_dual_flash - checking for dual or single qspi
  *
  * This function will check the type of the flash whether it supports
  * single or dual qspi based on the MIO configuration done by FSBL.
@@ -793,58 +795,59 @@ static int xqspips_transfer(struct spi_device *qspi,
  * function will return -1, if there is no MIO configuration for
  * qspi flash.
  */
-static int xqspips_check_is_dual_flash(void)
+static int zynq_qspi_check_is_dual_flash(void)
 {
        int is_dual = MODE_UNKNOWN;
        int lower_mio = 0, upper_mio = 0, upper_mio_cs1 = 0;
 
        lower_mio = zynq_slcr_get_mio_pin_status("qspi0");
-       if (lower_mio == XQSPIPS_MIO_NUM_QSPI0)
+       if (lower_mio == ZYNQ_QSPI_MIO_NUM_QSPI0)
                is_dual = MODE_SINGLE;
 
        upper_mio_cs1 = zynq_slcr_get_mio_pin_status("qspi1_cs");
-       if ((lower_mio == XQSPIPS_MIO_NUM_QSPI0) &&
-           (upper_mio_cs1 == XQSPIPS_MIO_NUM_QSPI1_CS))
+       if ((lower_mio == ZYNQ_QSPI_MIO_NUM_QSPI0) &&
+           (upper_mio_cs1 == ZYNQ_QSPI_MIO_NUM_QSPI1_CS))
                is_dual = MODE_DUAL_STACKED;
 
        upper_mio = zynq_slcr_get_mio_pin_status("qspi1");
-       if ((lower_mio == XQSPIPS_MIO_NUM_QSPI0) &&
-           (upper_mio_cs1 == XQSPIPS_MIO_NUM_QSPI1_CS) &&
-           (upper_mio == XQSPIPS_MIO_NUM_QSPI1))
+       if ((lower_mio == ZYNQ_QSPI_MIO_NUM_QSPI0) &&
+           (upper_mio_cs1 == ZYNQ_QSPI_MIO_NUM_QSPI1_CS) &&
+           (upper_mio == ZYNQ_QSPI_MIO_NUM_QSPI1))
                is_dual = MODE_DUAL_PARALLEL;
 
        return is_dual;
 }
 
 /*
- * xqspips_write_quad_bit - Write 1 to QUAD bit on flash
+ * zynq_qspi_write_quad_bit - Write 1 to QUAD bit on flash
  *
  * This function will write a 1 to quad bit in flash
  * using QSPI controller and supports only spansion flash.
  *
  * @regs_base: base address of QSPI controller
  */
-static void xqspips_write_quad_bit(void __iomem *regs_base)
+static void zynq_qspi_write_quad_bit(void __iomem *regs_base)
 {
        u32 config_reg, intr_status;
 
        /* enable the QSPI controller */
-       writel(XQSPIPS_ENABLE_ENABLE_MASK, &xqspips_base->enbr);
+       writel(ZYNQ_QSPI_ENABLE_ENABLE_MASK, &zynq_qspi_base->enbr);
 
        /* Write QUAD bit with 3-byte instruction */
-       writel(0x20001, &xqspips_base->txd3r);
+       writel(0x20001, &zynq_qspi_base->txd3r);
 
        /* Enable manual start command */
-       config_reg = readl(&xqspips_base->confr) | XQSPIPS_CONFIG_MANSRT_MASK;
-       writel(config_reg, &xqspips_base->confr);
+       config_reg = readl(&zynq_qspi_base->confr) |
+                       ZYNQ_QSPI_CONFIG_MANSRT_MASK;
+       writel(config_reg, &zynq_qspi_base->confr);
 
        /* Wait for the transfer to finish by polling Tx fifo status */
        do {
-               intr_status = readl(&xqspips_base->isr);
+               intr_status = readl(&zynq_qspi_base->isr);
        } while ((intr_status & 0x04) == 0);
 
        /* Read data receive register */
-       config_reg = readl(&xqspips_base->drxr);
+       config_reg = readl(&zynq_qspi_base->drxr);
 }
 
 int spi_cs_is_valid(unsigned int bus, unsigned int cs)
@@ -915,7 +918,7 @@ void spi_enable_quad_bit(struct spi_slave *spi)
                        }
 
                        /* Write QUAD bit */
-                       xqspips_write_quad_bit((void *)ZYNQ_QSPI_BASEADDR);
+                       zynq_qspi_write_quad_bit((void *)ZYNQ_QSPI_BASEADDR);
 
                        /* Read RDSR */
                        count = 0;
@@ -948,7 +951,7 @@ struct spi_slave *spi_setup_slave(unsigned int bus, unsigned int cs,
                unsigned int max_hz, unsigned int mode)
 {
        int is_dual;
-       struct zynq_spi_slave *pspi;
+       struct zynq_qspi_slave *qspi;
 
        debug("%s: bus: %d cs: %d max_hz: %d mode: %d\n",
              __func__, bus, cs, max_hz, mode);
@@ -956,7 +959,7 @@ struct spi_slave *spi_setup_slave(unsigned int bus, unsigned int cs,
        if (!spi_cs_is_valid(bus, cs))
                return NULL;
 
-       is_dual = xqspips_check_is_dual_flash();
+       is_dual = zynq_qspi_check_is_dual_flash();
 
        if (is_dual == MODE_UNKNOWN) {
                printf("%s: No QSPI device detected based on MIO settings\n",
@@ -964,37 +967,37 @@ struct spi_slave *spi_setup_slave(unsigned int bus, unsigned int cs,
                return NULL;
        }
 
-       xqspips_init_hw(is_dual, cs);
+       zynq_qspi_init_hw(is_dual, cs);
 
-       pspi = spi_alloc_slave(struct zynq_spi_slave, bus, cs);
-       if (!pspi) {
-               printf("%s: Fail to allocate zynq_spi_slave\n", __func__);
+       qspi = spi_alloc_slave(struct zynq_qspi_slave, bus, cs);
+       if (!qspi) {
+               printf("%s: Fail to allocate zynq_qspi_slave\n", __func__);
                return NULL;
        }
 
-       pspi->slave.is_dual = is_dual;
-       pspi->qspi.master.input_clk_hz = 100000000;
-       pspi->qspi.master.speed_hz = pspi->qspi.master.input_clk_hz / 2;
-       pspi->qspi.max_speed_hz = pspi->qspi.master.speed_hz;
-       pspi->qspi.master.is_dual = is_dual;
-       pspi->qspi.mode = mode;
-       pspi->qspi.chip_select = 0;
-       pspi->qspi.bits_per_word = 32;
-       xqspips_setup_transfer(&pspi->qspi, NULL);
+       qspi->slave.is_dual = is_dual;
+       qspi->qspi.master.input_clk_hz = 100000000;
+       qspi->qspi.master.speed_hz = qspi->qspi.master.input_clk_hz / 2;
+       qspi->qspi.max_speed_hz = qspi->qspi.master.speed_hz;
+       qspi->qspi.master.is_dual = is_dual;
+       qspi->qspi.mode = mode;
+       qspi->qspi.chip_select = 0;
+       qspi->qspi.bits_per_word = 32;
+       zynq_qspi_setup_transfer(&qspi->qspi, NULL);
 
-       spi_enable_quad_bit(&pspi->slave);
+       spi_enable_quad_bit(&qspi->slave);
 
-       return &pspi->slave;
+       return &qspi->slave;
 }
 
 void spi_free_slave(struct spi_slave *slave)
 {
-       struct zynq_spi_slave *pspi;
+       struct zynq_qspi_slave *qspi;
 
        debug("%s: slave: 0x%08x\n", __func__, (u32)slave);
 
-       pspi = to_zynq_spi_slave(slave);
-       free(pspi);
+       qspi = to_zynq_qspi_slave(slave);
+       free(qspi);
 }
 
 int spi_claim_bus(struct spi_slave *slave)
@@ -1011,14 +1014,14 @@ void spi_release_bus(struct spi_slave *slave)
 int spi_xfer(struct spi_slave *slave, unsigned int bitlen, const void *dout,
                void *din, unsigned long flags)
 {
-       struct zynq_spi_slave *pspi;
+       struct zynq_qspi_slave *qspi;
        struct spi_transfer transfer;
 
        debug("%s: slave: 0x%08x bitlen: %d dout: 0x%08x ", __func__,
              (u32)slave, bitlen, (u32)dout);
        debug("din: 0x%08x flags: 0x%lx\n", (u32)din, flags);
 
-       pspi = (struct zynq_spi_slave *)slave;
+       qspi = (struct zynq_qspi_slave *)slave;
        transfer.tx_buf = dout;
        transfer.rx_buf = din;
        transfer.len = bitlen / 8;
@@ -1029,9 +1032,9 @@ int spi_xfer(struct spi_slave *slave, unsigned int bitlen, const void *dout,
         * transmit must contain a device command.
         */
        if (dout && flags & SPI_XFER_BEGIN)
-               pspi->qspi.master.is_inst = 1;
+               qspi->qspi.master.is_inst = 1;
        else
-               pspi->qspi.master.is_inst = 0;
+               qspi->qspi.master.is_inst = 0;
 
        if (flags & SPI_XFER_END)
                transfer.cs_change = 1;
@@ -1039,15 +1042,15 @@ int spi_xfer(struct spi_slave *slave, unsigned int bitlen, const void *dout,
                transfer.cs_change = 0;
 
        if (flags & SPI_FLASH_U_PAGE)
-               pspi->qspi.master.u_page = 1;
+               qspi->qspi.master.u_page = 1;
        else
-               pspi->qspi.master.u_page = 0;
+               qspi->qspi.master.u_page = 0;
 
        transfer.delay_usecs = 0;
        transfer.bits_per_word = 32;
-       transfer.speed_hz = pspi->qspi.max_speed_hz;
+       transfer.speed_hz = qspi->qspi.max_speed_hz;
 
-       xqspips_transfer(&pspi->qspi, &transfer);
+       zynq_qspi_transfer(&qspi->qspi, &transfer);
 
        return 0;
 }