u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
* [PATCH] drivers: spi-nor: Add JEDEC id for W25Q16JV
@ 2021-11-28 15:09 Angus Ainslie
  2021-12-02  5:59 ` Jagan Teki
  2021-12-17 14:41 ` Angus Ainslie
  0 siblings, 2 replies; 4+ messages in thread
From: Angus Ainslie @ 2021-11-28 15:09 UTC (permalink / raw)
  To: Jagan Teki, Vignesh R; +Cc: u-boot, kernel, Angus Ainslie

Add a JEDEC id for the Winbond W25Q16JV 16M-BIT serial flash memory with
DUAL/QUAD SPI

Signed-off-by: Angus Ainslie <angus@akkea.ca>
---
 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 3ae7bb1ed7e..f20b21e0d06 100644
--- a/drivers/mtd/spi/spi-nor-ids.c
+++ b/drivers/mtd/spi/spi-nor-ids.c
@@ -314,6 +314,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("w25q16jv", 0xef7015, 0, 64 * 1024,  32,
+			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ |
+			SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)
+	},
 	{
 		INFO("w25q32jv", 0xef7016, 0, 64 * 1024,  64,
 			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ |
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] drivers: spi-nor: Add JEDEC id for W25Q16JV
  2021-11-28 15:09 [PATCH] drivers: spi-nor: Add JEDEC id for W25Q16JV Angus Ainslie
@ 2021-12-02  5:59 ` Jagan Teki
  2022-01-13 17:03   ` Angus Ainslie
  2021-12-17 14:41 ` Angus Ainslie
  1 sibling, 1 reply; 4+ messages in thread
From: Jagan Teki @ 2021-12-02  5:59 UTC (permalink / raw)
  To: Angus Ainslie; +Cc: Vignesh R, u-boot, kernel

On Sun, Nov 28, 2021 at 8:39 PM Angus Ainslie <angus@akkea.ca> wrote:
>
> Add a JEDEC id for the Winbond W25Q16JV 16M-BIT serial flash memory with
> DUAL/QUAD SPI
>
> Signed-off-by: Angus Ainslie <angus@akkea.ca>
> ---
>  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 3ae7bb1ed7e..f20b21e0d06 100644
> --- a/drivers/mtd/spi/spi-nor-ids.c
> +++ b/drivers/mtd/spi/spi-nor-ids.c
> @@ -314,6 +314,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("w25q16jv", 0xef7015, 0, 64 * 1024,  32,
> +                       SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ |
> +                       SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)

Look like this part is same for im/jm suffix parts. would you update
the name to use that - "w25q16jv-im/jm"

Jagan.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] drivers: spi-nor: Add JEDEC id for W25Q16JV
  2021-11-28 15:09 [PATCH] drivers: spi-nor: Add JEDEC id for W25Q16JV Angus Ainslie
  2021-12-02  5:59 ` Jagan Teki
@ 2021-12-17 14:41 ` Angus Ainslie
  1 sibling, 0 replies; 4+ messages in thread
From: Angus Ainslie @ 2021-12-17 14:41 UTC (permalink / raw)
  To: Jagan Teki, Vignesh R; +Cc: u-boot, kernel

Hi,

Are there any problems with this patch ?

Thanks
Angus

On 2021-11-28 07:09, Angus Ainslie wrote:
> Add a JEDEC id for the Winbond W25Q16JV 16M-BIT serial flash memory 
> with
> DUAL/QUAD SPI
> 
> Signed-off-by: Angus Ainslie <angus@akkea.ca>
> ---
>  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 3ae7bb1ed7e..f20b21e0d06 100644
> --- a/drivers/mtd/spi/spi-nor-ids.c
> +++ b/drivers/mtd/spi/spi-nor-ids.c
> @@ -314,6 +314,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("w25q16jv", 0xef7015, 0, 64 * 1024,  32,
> +			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ |
> +			SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)
> +	},
>  	{
>  		INFO("w25q32jv", 0xef7016, 0, 64 * 1024,  64,
>  			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ |

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] drivers: spi-nor: Add JEDEC id for W25Q16JV
  2021-12-02  5:59 ` Jagan Teki
@ 2022-01-13 17:03   ` Angus Ainslie
  0 siblings, 0 replies; 4+ messages in thread
From: Angus Ainslie @ 2022-01-13 17:03 UTC (permalink / raw)
  To: Jagan Teki; +Cc: Vignesh R, u-boot, kernel

Hi Jagan,

On 2021-12-01 21:59, Jagan Teki wrote:
> On Sun, Nov 28, 2021 at 8:39 PM Angus Ainslie <angus@akkea.ca> wrote:
>> 
>> Add a JEDEC id for the Winbond W25Q16JV 16M-BIT serial flash memory 
>> with
>> DUAL/QUAD SPI
>> 
>> Signed-off-by: Angus Ainslie <angus@akkea.ca>
>> ---
>>  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 3ae7bb1ed7e..f20b21e0d06 100644
>> --- a/drivers/mtd/spi/spi-nor-ids.c
>> +++ b/drivers/mtd/spi/spi-nor-ids.c
>> @@ -314,6 +314,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("w25q16jv", 0xef7015, 0, 64 * 1024,  32,
>> +                       SECT_4K | SPI_NOR_DUAL_READ | 
>> SPI_NOR_QUAD_READ |
>> +                       SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)
> 
> Look like this part is same for im/jm suffix parts. would you update
> the name to use that - "w25q16jv-im/jm"
> 

Sorry I missed this email I'll send an update.

Angus

> Jagan.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-01-13 17:03 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-28 15:09 [PATCH] drivers: spi-nor: Add JEDEC id for W25Q16JV Angus Ainslie
2021-12-02  5:59 ` Jagan Teki
2022-01-13 17:03   ` Angus Ainslie
2021-12-17 14:41 ` Angus Ainslie

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).