On Tue, Oct 1, 2019 at 7:23 AM <[email protected]> wrote: > > > > On 09/27/2019 07:43 AM, Vignesh Raghavendra wrote: > > External E-Mail > > > > > > mt25qu512a flash has Flag status register that indicates various errors > > that may be encountered during erase/write operations. Therefore add > > USE_FSR flag to the entry > > > n25q256 & 512 support flag status register command. Would you add the flag > for > them too? mtq256 probably too.
Right, the datasheet of my mt25q says flag status register command is supported. Otherwise: Reviewed-by: Simon Goldschmidt <[email protected]> Regards, Simon > > > Signed-off-by: Vignesh Raghavendra <[email protected]> > > anyway, looks good: > Reviewed-by: Tudor Ambarus <[email protected]> > > > --- > > drivers/mtd/spi/spi-nor-ids.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/mtd/spi/spi-nor-ids.c b/drivers/mtd/spi/spi-nor-ids.c > > index c7b6cf002c54..2e0aebbc44aa 100644 > > --- a/drivers/mtd/spi/spi-nor-ids.c > > +++ b/drivers/mtd/spi/spi-nor-ids.c > > @@ -166,7 +166,8 @@ const struct flash_info spi_nor_ids[] = { > > { INFO6("mt25qu256a", 0x20bb19, 0x104400, 64 * 1024, 512, SECT_4K | > > SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) }, > > { INFO("n25q256ax1", 0x20bb19, 0, 64 * 1024, 512, SECT_4K | > > SPI_NOR_QUAD_READ) }, > > { INFO6("mt25qu512a", 0x20bb20, 0x104400, 64 * 1024, 1024, > > - SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) > > }, > > + SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES | > > + USE_FSR) }, > > { INFO("n25q512a", 0x20bb20, 0, 64 * 1024, 1024, SECT_4K | USE_FSR > > | SPI_NOR_QUAD_READ) }, > > { INFO6("mt25ql512a", 0x20ba20, 0x104400, 64 * 1024, 1024, SECT_4K | > > USE_FSR | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) }, > > { INFO("n25q512ax3", 0x20ba20, 0, 64 * 1024, 1024, SECT_4K | USE_FSR > > | SPI_NOR_QUAD_READ) }, > > _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot

