zynq spi控制器挂在flash高16M空间的使用问题

zynq挂载32M qspi flash后不能复位的问题

今天遇到一个奇怪的现象，zynq 平台搭载了一个32M qspi flash，我将其中高16M作为一个分区放了一个jffs2文件系统，在系统启动后mount到根文件系统ramdisk中。
在设备树分好区后，到uboot下将jffs2写到了flash的1000000~2000000空间，随后reset，却卡死了，按复位键也没用。
断电重启后系统正常起来，进入了linux。手动mount 了新的分区，文件系统读写正常。随后reboot，又卡死了。。。按复位键依旧没用，还是断电重启才行。
一开始以为是fsbl阶段有问题，跟踪代码后发现连ps7_init都没走到，又跟踪了uboot的reset代码，确定已经写入了cpu的复位寄存器，uboot reset流程正常走完。
加上linux中reboot也失败，捋了一下启动流程：
1、

• uboot
  reset
• x
• fsbl
  2、
• linux
  reboot
• x
• fsbl
• uboot
  问题很可能出在 x 处。这个 x 只能是bootROM了啊。
  根据这个线索搜了一下，果然有：使用四字节命令读写256Mb QSPI Flash
  全文如下：

在手册中，Zynq QSPI flash 控制器只支持3字节的地址，因此只支持最大128M bit的QSPI flash。 如果要支持更大的flash，需要使用QSPI flash中的bank寄存器。SDK 14.6的驱动就是采用这种模式。 使用bank寄存器，会带来副作用。Zynq内部的BootRom不会去操作bank寄存器。如果在操作非0的bank时复位，BootRom从QSPI Flash读取FSBL就会失败。这种情况，可以使用复位信号对QSPI Flash进行复位，使其内部的bank寄存器也复位为0，或者直接复位bank寄存器。另外一种办法，是使用新的QSPI Flash命令，使用四字节地址读写Flash。 本文使用Zed板上的Spansion S25FL256S 器件做测试。“12h” 是支持四字节地址的写命令； “13h”是支持四字节地址的读命令。在Zed板上，这两个命令，都工作正常，可以读写数据，而且不影响Zynq启动。

在uboot下验证了一下：sf 读取1000000 以上， 后reset，卡死；sf 读取1000000 以上，再读取1000000以下，然后reset，没卡死。的确是这个原因。

kernel中解决方法：
查看了 s25fl256s 的手册，控制读写高低16M空间的BAR寄存器在硬件复位和软件复位后都会清0 ，对应内核中最简单的处理办法就是在flash驱动中的 shutdown方法里复位芯片。
修改代码如下：

static void m25p_s25fl256s_reset(struct spi_device *spi)
{
    struct m25p *flash = spi_get_drvdata(spi);
        flash->command[0] = 0xf0;  //reset command
        spi_write(flash->spi, flash->command, 1); 
        printk(KERN_INFO "reset qspi flash\n");
        return;
}
static struct spi_driver m25p80_driver = {
    .driver = {
        .name   = "m25p80",
        .owner  = THIS_MODULE,
    },
    .id_table   = m25p_ids,
    .probe  = m25p_probe,
    .remove = m25p_remove,
        .shutdown = m25p_s25fl256s_reset,
    /* REVISIT: many of these chips have deep power-down modes, which
     * should clearly be entered on suspend() to minimize power use.
     * And also when they're otherwise idle...
     */
};

这样在执行reboot时，flash芯片就会被复位，BAR寄存器清除。

上面提到采用4字节来读写flash的解决方法，但在内核代码中看到xilinx特地限制了zynq不使用四字节模式，如下：

/*spi-nor.c (drivers\mtd\spi-nor)   937
* int spi_nor_scan(struct spi_nor *nor, const struct spi_device_id *id,
*           enum read_mode mode)
*/
#ifdef CONFIG_OF
        np_spi = of_get_next_parent(np);
        if (of_property_match_string(np_spi, "compatible",
                         "xlnx,zynq-qspi-1.0") >= 0) {
            int status;

            nor->addr_width = 3;
            set_4byte(nor, info->jedec_id, 0);
            status = read_ear(nor, info->jedec_id);
            if (status < 0)
                dev_warn(dev, "failed to read ear reg\n");
            else
                nor->curbank = status & EAR_SEGMENT_MASK;
        } else {
#endif