On Wed, Oct 26, 2016 at 9:18 PM, Bacrau, Dumitru <dumitru.bac...@intel.com> wrote: > Hello Teki, > > The device that I have used is MT25QU02GCBB8E12-0SIT, with the datasheet > available here: > https://www.micron.com/~/media/documents/products/data-sheet/nor-flash/serial-nor/mt25q/die-rev-b/mt25q_qlkt_u_02g_cbb_0.pdf.
Thanks. > > -----Original Message----- > From: Jagan Teki [mailto:ja...@openedev.com] > Sent: Wednesday, October 26, 2016 2:25 AM > To: Bacrau, Dumitru <dumitru.bac...@intel.com> > Cc: u-boot@lists.denx.de; Chin Liang See <cl...@altera.com>; Dinh Nguyen > <dingu...@opensource.altera.com>; Radu Bacrau <radu.bac...@gmail.com> > Subject: Re: [PATCH] spi: Add support for MX66U51235F, MX66L1G45G and > MT25QU02G > > On Wed, Oct 12, 2016 at 7:45 PM, <dumitru.bac...@intel.com> wrote: >> From: Radu Bacrau <dumitru.bac...@intel.com> >> >> This commit adds support for the Macronix MX66U51235F, MX66L1G45G and Micron >> MT25QU02G flash parts. >> >> Signed-off-by: Radu Bacrau <dumitru.bac...@intel.com> >> Cc: Chin Liang See <cl...@altera.com> >> Cc: Dinh Nguyen <dingu...@opensource.altera.com> >> Cc: Jagan Teki <jt...@openedev.com> >> Cc: Radu Bacrau <radu.bac...@gmail.com> >> --- >> drivers/mtd/spi/sf_params.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c >> index 5b50114..8b50465 100644 >> --- a/drivers/mtd/spi/sf_params.c >> +++ b/drivers/mtd/spi/sf_params.c >> @@ -52,6 +52,8 @@ const struct spi_flash_params spi_flash_params_table[] = { >> {"MX25L25635F", 0xc22019, 0x0, 64 * 1024, 512, RD_FULL | >> WR_QPP}, >> {"MX25L51235F", 0xc2201a, 0x0, 64 * 1024, 1024, RD_FULL | >> WR_QPP}, >> {"MX25L12855E", 0xc22618, 0x0, 64 * 1024, 256, RD_FULL | >> WR_QPP}, >> + {"MX66U51235F", 0xc2253a, 0x0, 64 * 1024, 1024, RD_FULL | >> WR_QPP}, >> + {"MX66L1G45G", 0xc2201b, 0x0, 64 * 1024, 2048, RD_FULL | >> WR_QPP}, > > OK. > >> #endif >> #ifdef CONFIG_SPI_FLASH_SPANSION /* SPANSION */ >> {"S25FL008A", 0x010213, 0x0, 64 * 1024, 16, 0}, >> @@ -98,6 +100,7 @@ const struct spi_flash_params spi_flash_params_table[] = { >> {"N25Q512A", 0x20bb20, 0x0, 64 * 1024, 1024, RD_FULL | >> WR_QPP | E_FSR | SECT_4K}, >> {"N25Q1024", 0x20ba21, 0x0, 64 * 1024, 2048, RD_FULL | >> WR_QPP | E_FSR | SECT_4K}, >> {"N25Q1024A", 0x20bb21, 0x0, 64 * 1024, 2048, RD_FULL | >> WR_QPP | E_FSR | SECT_4K}, >> + {"MT25QU02G", 0x20bb22, 0x0, 64 * 1024, 4096, RD_FULL | >> WR_QPP | E_FSR | SECT_4K}, Since it's bb 1.8v we usually differentiate to use 'A' at the end of the name like N25Q1024 - 0x20ba N25Q1024A - 0x20bb So, we need append A at end of the name - MT25QU02GA. If you agree please try to send the next version with this update. thanks! -- Jagan Teki Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream Maintainer Hyderabad, India. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot