RE: [EXT] [PATCH] mtd: spi-nor: spansion: fix writes on S25FS512S

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: linux-mtd <linux-mtd-bounces@xxxxxxxxxxxxxxxxxxx> On Behalf Of
> Sergei Shtylyov
> Sent: Tuesday, April 21, 2020 12:44 AM
> To: Tudor Ambarus <tudor.ambarus@xxxxxxxxxxxxx>; linux-
> mtd@xxxxxxxxxxxxxxxxxxx
> Cc: Richard Weinberger <richard@xxxxxx>; Yicong Yang
> <yangyicong@xxxxxxxxxxxxx>; Alexander Sverdlin
> <alexander.sverdlin@xxxxxxxxx>; Vignesh Raghavendra <vigneshr@xxxxxx>;
> Miquel Raynal <miquel.raynal@xxxxxxxxxxx>
> Subject: [EXT] [PATCH] mtd: spi-nor: spansion: fix writes on S25FS512S
> 
> Caution: EXT Email
> 
> Spansion S25FS-S family has an issue in the Basic Flash Parameter Table
> (BFPT): Dword-11 bits 7:4 specify a page size of 512 bytes.  Actually this is
> configurable in the vendor unique register (CR3V) and even the factory
> default setting is to "wrap at 256 bytes", so blindly relying on BFPT breaks
> the page writes on these chips. Add the post-BFPT fixup which restores the
> default page size of 256 bytes -- to properly read CR3V this early is quite
> intrusive and should better be done as a new feature; Alexander Sverdlin
> had the patch doing that:
> 
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatch
> work.ozlabs.org%2Fproject%2Flinux-
> mtd%2Fpatch%2F20200227123657.26030-1-
> alexander.sverdlin%40nokia.com%2F&amp;data=02%7C01%7Ckuldeep.sing
> h%40nxp.com%7Cb99b2075b3cc44c5f06108d7e55f0342%7C686ea1d3bc2b4
> c6fa92cd99c5c301635%7C0%7C0%7C637230068559206386&amp;sdata=BN
> ebniG68OFN7uJr1%2Fpm7wiQq6EPASNPrwEusHfc6zc%3D&amp;reserved=0
> 
> Fixes: dfd2b74530e ("mtd: spi-nor: add Spansion S25FS512S ID")
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx>

Tested-by: Kuldeep Singh <kuldeep.singh@xxxxxxx>
This page size fixup is required for various NXP platforms to run quad mode successfully.

-Kuldeep

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/



[Index of Archives]     [LARTC]     [Bugtraq]     [Yosemite Forum]     [Photo]

  Powered by Linux