Re: [U-Boot] [PATCH] sf: set the Uniform Sector to CR3NV instead of CR3V

30 Jun 2016

From: Michael Trimarchi [mailto:michael@amarulasolutions.com]
Sent: Thursday, June 30, 2016 2:50 PM
To: Qianyu Gong
Cc: Mingkai Hu; u-boot@lists.denx.de; Yunhui Cui; jteki@openedev.com
Subject: Re: [U-Boot] [PATCH] sf: set the Uniform Sector to CR3NV instead of CR3V
Hi
On Jun 30, 2016 08:47, "Gong Qianyu" <Qianyu.Gong@nxp.commailto:Qianyu.Gong@nxp.com> wrote:
...
From: Mingkai Hu <mingkai.hu@nxp.commailto:mingkai.hu@nxp.com>
Set the flash to Uniform Sector Architecture in the non-volatile
register. After the power cycle, it's also Uniform Sector Architecture.
Signed-off-by: Mingkai Hu <mingkai.hu@nxp.commailto:mingkai.hu@nxp.com>
Signed-off-by: Gong Qianyu <Qianyu.Gong@nxp.commailto:Qianyu.Gong@nxp.com>

diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c
index 64d4e0f..366c362 100644
--- a/drivers/mtd/spi/spi_flash.c
+++ b/drivers/mtd/spi/spi_flash.c
@@ -975,7 +975,7 @@ int spi_flash_decode_fdt(const void *blob, struct spi_flash *flash)
 static int spansion_s25fss_disable_4KB_erase(struct spi_slave *spi)
 {
        u8 cmd[4];

  u32 offset = 0x800004; /* CR3V register offset */




  u32 offset = 0x4; /* CR3NV register offset */
  u8 cr3v;
  int ret;




I have already tested it and I have in my tree.
What does this mean?
I don't think that should be mandatory in general
The general code doesn’t handle different sector operation, so need to set it as uniform sector architecture.
Michael
...
--
2.1.0.27.g96db324

U-Boot mailing list
U-Boot@lists.denx.demailto:U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

    