Hi,

> Add the 256MB W25Q02RV (JEDEC ID 0xef4022) to the SPI NOR ID table.
> 

I searched and found the datasheet [1], please confirm it's correct.
If it is, please give a Link: tag followed by the URL.

> Signed-off-by: Jim Liu <[email protected]>
> Signed-off-by: Stanley Chu <[email protected]>
> ---
>  drivers/mtd/spi/spi-nor-ids.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/mtd/spi/spi-nor-ids.c b/drivers/mtd/spi/spi-nor-ids.c
> index 31a2ba49a87..3e7096bd419 100644
> --- a/drivers/mtd/spi/spi-nor-ids.c
> +++ b/drivers/mtd/spi/spi-nor-ids.c
> @@ -633,6 +633,11 @@ const struct flash_info spi_nor_ids[] = {
>                         SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ |
>                         SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)
>         },
> +       {
> +               INFO("w25q02rv", 0xef4022, 0, 64 * 1024, 4096,

Per datasheet, W25Q02RV has ID variants, 0xef4022 and 0xef7022 for different
part number suffixes, -M and -Q/N. The name needs to contain suffix.

> +                       SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ |
> +                       SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)

Did you test locking feature?
The chip has 4-bit BP but current u-boot doesn't support it.

Thanks,
Takahiro

> +       },
>         {
>                 INFO("w25q01jvfim", 0xef7021, 0, 64 * 1024, 2048,
>                         SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ |
> --
> 2.34.1

[1] 
https://www.winbond.com/hq/support/documentation/downloadV2022.jsp?__locale=en&xmlPath=/support/resources/.content/item/DA00-W25Q51RV.html&level=1

Reply via email to