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