Hi Sergei, On 2020/4/20 19:05, Sergei Shtylyov wrote: > On 20.04.2020 13:18, Tudor.Ambarus@xxxxxxxxxxxxx wrote: > >> The flash should be named "s25fs128s1" - this would be the flavor with uniform >> 64kB sectors. One should add a "s25fs128s0" entry for the flavor with uniform >> 256kB sectors. >> >> We usually don't add new flashes if they are not tested. Please specify in the >> commit message with which controller you tested the flash. The minimal test is >> a read, erase, read back, write and read back. This sequence should verify if >> read, erase and pp work fine. > > This chip (as part of the S25FS-S family) most probably requires a post-BFPT fixup to make the writes work. I was going to post the patch adding this fixup for S25FS512S chip. Thanks for the information, I'll test whether s25fs128s1 needs this fixup with your patch. Regards, Yicong > >> Cheers, >> ta > > MBR, Sergei > . > ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/