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

[Mingkai Hu]

From: Michael Trimarchi [mailto:michael at amarulasolutions.com]
Sent: Thursday, June 30, 2016 2:50 PM
To: Qianyu Gong
Cc: Mingkai Hu; u-boot at lists.denx.de; Yunhui Cui; jteki at 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 at nxp.com<mailto:Qianyu.Gong at nxp.com>> wrote:
>
> From: Mingkai Hu <mingkai.hu at nxp.com<mailto:mingkai.hu at 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 at nxp.com<mailto:mingkai.hu at nxp.com>>
> Signed-off-by: Gong Qianyu <Qianyu.Gong at nxp.com<mailto:Qianyu.Gong at 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 at lists.denx.de<mailto:U-Boot at lists.denx.de>
> http://lists.denx.de/mailman/listinfo/u-boot