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

30 Jun 2016

...
-----Original Message-----
From: Michael Trimarchi [mailto:michael@amarulasolutions.com]
Sent: Thursday, June 30, 2016 3:33 PM
To: Mingkai Hu
Cc: Qianyu Gong; 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 Thu, Jun 30, 2016 at 9:29 AM, Mingkai Hu mingkai.hu@nxp.com wrote:
...
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.com wrote:
...
From: Mingkai Hu 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.com
Signed-off-by: Gong Qianyu 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.
And what about board that does not use the flash in uboot but manage in some
other way. I'm not saying that is wrong but I don't know if this setting must be
no-volatile. Is it something connected from booting on QSPI?
If it was not used as uniform sector architecture, the spansion_s25fss_disable_4KB_erase
should not be called, correct?
Thanks,
Mingkai
...
...
...
--
2.1.0.27.g96db324

U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
--
| Michael Nazzareno Trimarchi                     Amarula Solutions BV |
| COO  -  Founder                                      Cruquiuskade 47 |
| +31(0)851119172                                 Amsterdam 1018 AM NL |
|                  [`as] http://www.amarulasolutions.com               |

    