Message ID | 1530047647-5744-1-git-send-email-oe5hpm@oevsv.at |
---|---|
State | Accepted |
Commit | 760b75564fdf2fe53d8c4069a6fb3320586eb662 |
Delegated to: | Jagannadha Sutradharudu Teki |
Headers | show |
Series | [U-Boot] spi_flash: add a bunch of winbond flashes to id-table | expand |
On Wed, Jun 27, 2018 at 2:44 AM, Hannes Schmelzer <oe5hpm@oevsv.at> wrote: > This commit adds the following flashes to the id-table > > - W25Q16JV > - W25Q32JV > - W25Q64JV > - W25Q128JV > - W25Q256JV Just to curious, have you tested all these?
On 06/27/2018 01:57 PM, Jagan Teki wrote: > On Wed, Jun 27, 2018 at 2:44 AM, Hannes Schmelzer <oe5hpm@oevsv.at> wrote: >> This commit adds the following flashes to the id-table >> >> - W25Q16JV >> - W25Q32JV >> - W25Q64JV >> - W25Q128JV >> - W25Q256JV > Just to curious, have you tested all these? > > hi jagan, yes they are tested ... we were forced due to discontinuation of other flashes to replace them. So we had samples here for testing all of those, some of them are already in series production. cheers, Hannes
On Wed, Jun 27, 2018 at 6:01 PM, Hannes Schmelzer <hannes@schmelzer.or.at> wrote: > > On 06/27/2018 01:57 PM, Jagan Teki wrote: >> >> On Wed, Jun 27, 2018 at 2:44 AM, Hannes Schmelzer <oe5hpm@oevsv.at> wrote: >>> >>> This commit adds the following flashes to the id-table >>> >>> - W25Q16JV >>> - W25Q32JV >>> - W25Q64JV >>> - W25Q128JV >>> - W25Q256JV >> >> Just to curious, have you tested all these? >> >> > hi jagan, > > yes they are tested ... we were forced due to discontinuation of other > flashes to replace them. > So we had samples here for testing all of those, some of them are already in > series production. OK. Reviewed-by: Jagan Teki <jagan@openedev.com>
On Wed, Jun 27, 2018 at 9:19 PM, Jagan Teki <jagannadh.teki@gmail.com> wrote: > On Wed, Jun 27, 2018 at 6:01 PM, Hannes Schmelzer > <hannes@schmelzer.or.at> wrote: >> >> On 06/27/2018 01:57 PM, Jagan Teki wrote: >>> >>> On Wed, Jun 27, 2018 at 2:44 AM, Hannes Schmelzer <oe5hpm@oevsv.at> wrote: >>>> >>>> This commit adds the following flashes to the id-table >>>> >>>> - W25Q16JV >>>> - W25Q32JV >>>> - W25Q64JV >>>> - W25Q128JV >>>> - W25Q256JV >>> >>> Just to curious, have you tested all these? >>> >>> >> hi jagan, >> >> yes they are tested ... we were forced due to discontinuation of other >> flashes to replace them. >> So we had samples here for testing all of those, some of them are already in >> series production. > > OK. > > Reviewed-by: Jagan Teki <jagan@openedev.com> Applied to u-boot-spi/master
diff --git a/drivers/mtd/spi/spi_flash_ids.c b/drivers/mtd/spi/spi_flash_ids.c index c45d2e8..5e661a6 100644 --- a/drivers/mtd/spi/spi_flash_ids.c +++ b/drivers/mtd/spi/spi_flash_ids.c @@ -176,10 +176,15 @@ const struct spi_flash_info spi_flash_ids[] = { {"w25q256", INFO(0xef4019, 0x0, 64 * 1024, 512, RD_FULL | WR_QPP | SECT_4K) }, {"w25q80bw", INFO(0xef5014, 0x0, 64 * 1024, 16, RD_FULL | WR_QPP | SECT_4K) }, {"w25q16dw", INFO(0xef6015, 0x0, 64 * 1024, 32, RD_FULL | WR_QPP | SECT_4K) }, + {"w25q16jv", INFO(0xef7015, 0x0, 64 * 1024, 32, RD_FULL | WR_QPP | SECT_4K) }, {"w25q32dw", INFO(0xef6016, 0x0, 64 * 1024, 64, RD_FULL | WR_QPP | SECT_4K) }, + {"w25q32jv", INFO(0xef7016, 0x0, 64 * 1024, 64, RD_FULL | WR_QPP | SECT_4K) }, {"w25q64dw", INFO(0xef6017, 0x0, 64 * 1024, 128, RD_FULL | WR_QPP | SECT_4K) }, + {"w25q64jv", INFO(0xef7017, 0x0, 64 * 1024, 128, RD_FULL | WR_QPP | SECT_4K) }, {"w25q128fw", INFO(0xef6018, 0x0, 64 * 1024, 256, RD_FULL | WR_QPP | SECT_4K) }, + {"w25q128jv", INFO(0xef7018, 0x0, 64 * 1024, 256, RD_FULL | WR_QPP | SECT_4K) }, {"w25q256fw", INFO(0xef6019, 0x0, 64 * 1024, 512, RD_FULL | WR_QPP | SECT_4K) }, + {"w25q256jw", INFO(0xef7019, 0x0, 64 * 1024, 512, RD_FULL | WR_QPP | SECT_4K) }, #endif {}, /* Empty entry to terminate the list */ /*
This commit adds the following flashes to the id-table - W25Q16JV - W25Q32JV - W25Q64JV - W25Q128JV - W25Q256JV Signed-off-by: Hannes Schmelzer <oe5hpm@oevsv.at> --- drivers/mtd/spi/spi_flash_ids.c | 5 +++++ 1 file changed, 5 insertions(+)