linux-mtd.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] mtd: spi-nor: add Winbond 1.8v SPI NOR Flash ID
@ 2019-09-26  9:28 Shih-Ting Lin
  2019-10-16  8:07 ` Vignesh Raghavendra
  0 siblings, 1 reply; 2+ messages in thread
From: Shih-Ting Lin @ 2019-09-26  9:28 UTC (permalink / raw)
  To: linux-mtd

Winbond SPI NOR Flash have new product for 1.8V production line.
Following is the patch add new device ID and part number information in spi-nor.c file.

Signed-off-by: Shih-Ting Lin <stlin2@winbond.com>

--- a/drivers/mtd/spi-nor/spi-nor.c
+++ b/drivers/mtd/spi-nor/spi-nor.c
@@ -2153,6 +2153,22 @@
>  { "w25q256", INFO(0xef4019, 0, 64 * 1024, 512, SECT_4K |
> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },  { "w25m512jv",
> INFO(0xef7119, 0, 64 * 1024, 1024,  SECT_4K | SPI_NOR_QUAD_READ |
> SPI_NOR_DUAL_READ) },
> +{ "w25q32jwxxIM", INFO(0xef8016, 0, 64 * 1024,  64, SECT_4K |
> +SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_HAS_LOCK |
> +SPI_NOR_HAS_TB) }, { "w25q64jwxxIM", INFO(0xef8017, 0, 64 * 1024,
> +128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ |
> +SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB) }, { "w25q128jwxxIM",
> +INFO(0xef8018, 0, 64 * 1024, 256, SECT_4K | SPI_NOR_DUAL_READ |
> +SPI_NOR_QUAD_READ | SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB) }, {
> +"w25q256jwxxIM", INFO(0xef8019, 0, 64 * 1024, 512, SECT_4K |
> +SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_HAS_LOCK |
> +SPI_NOR_HAS_TB) },

TEST=rebuild kernel,update to DUT
    check DUT:
    cat /var/log/messages | grep mtk-nor
    ---
    localhost ~ # cat /var/log/messages | grep mtk-nor
    2019-07-08T04:42:51.636356-07:00 INFO kernel: [0.301351] mtk-nor \
    1100d000.spi:w25q32jw (4096 Kbytes)



________________________________

The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Winbond is strictly prohibited; and any information in this email irrelevant to the official business of Winbond shall be deemed as neither given nor endorsed by Winbond.

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

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

* Re: [PATCH] mtd: spi-nor: add Winbond 1.8v SPI NOR Flash ID
  2019-09-26  9:28 [PATCH] mtd: spi-nor: add Winbond 1.8v SPI NOR Flash ID Shih-Ting Lin
@ 2019-10-16  8:07 ` Vignesh Raghavendra
  0 siblings, 0 replies; 2+ messages in thread
From: Vignesh Raghavendra @ 2019-10-16  8:07 UTC (permalink / raw)
  To: Shih-Ting Lin, linux-mtd

Hi,

On 26/09/19 2:58 PM, Shih-Ting Lin wrote:
> Winbond SPI NOR Flash have new product for 1.8V production line.
> Following is the patch add new device ID and part number information in spi-nor.c file.
> 

This patch does not meet kernel coding guidelines. Use git format-patch
to generate patch and run ./scripts/checkpatch.pl --strict on the patch
and fix reported issues.

https://www.kernel.org/doc/html/latest/process/submitting-patches.html

Also please what modes were tested?

Regards
Vignesh

> Signed-off-by: Shih-Ting Lin <stlin2@winbond.com>
> 
> --- a/drivers/mtd/spi-nor/spi-nor.c
> +++ b/drivers/mtd/spi-nor/spi-nor.c
> @@ -2153,6 +2153,22 @@
>>  { "w25q256", INFO(0xef4019, 0, 64 * 1024, 512, SECT_4K |
>> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },  { "w25m512jv",
>> INFO(0xef7119, 0, 64 * 1024, 1024,  SECT_4K | SPI_NOR_QUAD_READ |
>> SPI_NOR_DUAL_READ) },
>> +{ "w25q32jwxxIM", INFO(0xef8016, 0, 64 * 1024,  64, SECT_4K |
>> +SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_HAS_LOCK |
>> +SPI_NOR_HAS_TB) }, { "w25q64jwxxIM", INFO(0xef8017, 0, 64 * 1024,
>> +128, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ |
>> +SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB) }, { "w25q128jwxxIM",
>> +INFO(0xef8018, 0, 64 * 1024, 256, SECT_4K | SPI_NOR_DUAL_READ |
>> +SPI_NOR_QUAD_READ | SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB) }, {
>> +"w25q256jwxxIM", INFO(0xef8019, 0, 64 * 1024, 512, SECT_4K |
>> +SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_HAS_LOCK |
>> +SPI_NOR_HAS_TB) },
> 
> TEST=rebuild kernel,update to DUT
>     check DUT:
>     cat /var/log/messages | grep mtk-nor
>     ---
>     localhost ~ # cat /var/log/messages | grep mtk-nor
>     2019-07-08T04:42:51.636356-07:00 INFO kernel: [0.301351] mtk-nor \
>     1100d000.spi:w25q32jw (4096 Kbytes)
> 
> 
> 
> ________________________________
> 
> The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Winbond is strictly prohibited; and any information in this email irrelevant to the official business of Winbond shall be deemed as neither given nor endorsed by Winbond.
> 
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/
> 

-- 
Regards
Vignesh

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

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

end of thread, other threads:[~2019-10-16  8:07 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-26  9:28 [PATCH] mtd: spi-nor: add Winbond 1.8v SPI NOR Flash ID Shih-Ting Lin
2019-10-16  8:07 ` Vignesh Raghavendra

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).