2022-01-21 22:18:36

by Michael Walle

Subject: [PATCH] mtd: spi-nor: winbond: fix the name of W25Q16JV-IM/-JM

We don't include the package encoding in the name. Thus, just use
w25q15jvm as the name for this flash.

Signed-off-by: Michael Walle <[email protected]>
---
drivers/mtd/spi-nor/winbond.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/spi-nor/winbond.c b/drivers/mtd/spi-nor/winbond.c
index 93b537f4f0cb..3d91888882e4 100644
--- a/drivers/mtd/spi-nor/winbond.c
+++ b/drivers/mtd/spi-nor/winbond.c
@@ -52,7 +52,7 @@ static const struct flash_info winbond_parts[] = {
SPI_NOR_QUAD_READ) },
{ "w25x32", INFO(0xef3016, 0, 64 * 1024, 64)
NO_SFDP_FLAGS(SECT_4K) },
- { "w25q16jv-im/jm", INFO(0xef7015, 0, 64 * 1024, 32)
+ { "w25q16jvm", INFO(0xef7015, 0, 64 * 1024, 32)
FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)
NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ |
SPI_NOR_QUAD_READ) },
--
2.30.2

2022-02-28 10:51:23

by Tudor Ambarus

Subject: Re: [PATCH] mtd: spi-nor: winbond: fix the name of W25Q16JV-IM/-JM

On 1/20/22 18:14, Tudor Ambarus wrote:
> On 1/20/22 4:57 PM, Michael Walle wrote:
>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>
>> We don't include the package encoding in the name. Thus, just use
>
> s/package encoding/ temp

>> w25q15jvm as the name for this flash.

If I remember correctly I think that we agreed on u-boot ml that we should rename
all -jvm flashes to -dtr. Let me know if you want to reiterate this or not, I
can add it to a todo list if you don't feel like doing it right now.

Cheers,
ta

>>
>> Signed-off-by: Michael Walle <[email protected]>
>> ---
>> drivers/mtd/spi-nor/winbond.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/mtd/spi-nor/winbond.c b/drivers/mtd/spi-nor/winbond.c
>> index 93b537f4f0cb..3d91888882e4 100644
>> --- a/drivers/mtd/spi-nor/winbond.c
>> +++ b/drivers/mtd/spi-nor/winbond.c
>> @@ -52,7 +52,7 @@ static const struct flash_info winbond_parts[] = {
>> SPI_NOR_QUAD_READ) },
>> { "w25x32", INFO(0xef3016, 0, 64 * 1024, 64)
>> NO_SFDP_FLAGS(SECT_4K) },
>> - { "w25q16jv-im/jm", INFO(0xef7015, 0, 64 * 1024, 32)
>> + { "w25q16jvm", INFO(0xef7015, 0, 64 * 1024, 32)
>> FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)
>> NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ |
>> SPI_NOR_QUAD_READ) },
>> --
>> 2.30.2
>>
>