This patch adds support for a new ISSI 1MB SPI NOR chip that was tested in our
lab. Datasheet is available at: http://www.issi.com/WW/pdf/25LP-WP080D.pdf
Tested-by: Pascal Fabreges <[email protected]>
Reviewed-by: Alexander Sverdlin <[email protected]>
---
drivers/mtd/spi-nor/spi-nor.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c
index bc266f70a15b..92cd62f02176 100644
--- a/drivers/mtd/spi-nor/spi-nor.c
+++ b/drivers/mtd/spi-nor/spi-nor.c
@@ -1021,6 +1021,7 @@ static const struct flash_info spi_nor_ids[] = {
/* ISSI */
{ "is25cd512", INFO(0x7f9d20, 0, 32 * 1024, 2, SECT_4K) },
+ { "is25lp080d", INFO(0x9d6014, 0, 64 * 1024, 16, SECT_4K) },
/* Macronix */
{ "mx25l512e", INFO(0xc22010, 0, 64 * 1024, 1, SECT_4K) },
--
2.11.0
Hi Romain,
Le 19/12/2017 à 14:43, Romain Porte a écrit :
> This patch adds support for a new ISSI 1MB SPI NOR chip that was tested in our
> lab. Datasheet is available at: http://www.issi.com/WW/pdf/25LP-WP080D.pdf
>
> Tested-by: Pascal Fabreges <[email protected]>
> Reviewed-by: Alexander Sverdlin <[email protected]>
> ---
> drivers/mtd/spi-nor/spi-nor.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c
> index bc266f70a15b..92cd62f02176 100644
> --- a/drivers/mtd/spi-nor/spi-nor.c
> +++ b/drivers/mtd/spi-nor/spi-nor.c
> @@ -1021,6 +1021,7 @@ static const struct flash_info spi_nor_ids[] = {
>
> /* ISSI */
> { "is25cd512", INFO(0x7f9d20, 0, 32 * 1024, 2, SECT_4K) },
> + { "is25lp080d", INFO(0x9d6014, 0, 64 * 1024, 16, SECT_4K) },
>
Based on the provided datasheet, both SPI_NOR_DUAL_READ (3Bh) and
SPI_NOR_QUAD_READ (6Bh) are missing. Besides adding at least one them will
enable the parsing of SFDP tables, which are also supported by this memory
part.
Then please add both SPI_NOR_DUAL_READ and SPI_NOR_QUAD_READ with SECT_4K.
Also, please rebase your patch on the spi-nor/next branch of l2-mtd (moving
to linux-mtd ?) as another patch has already added an entry for is25lp128
Best regards,
Cyrille
> /* Macronix */
> { "mx25l512e", INFO(0xc22010, 0, 64 * 1024, 1, SECT_4K) },
>
This patch adds support for a new ISSI 1MB SPI NOR chip that was tested in our
lab. Datasheet is available at: http://www.issi.com/WW/pdf/25LP-WP080D.pdf
Tested-by: Pascal Fabreges <[email protected]>
Reviewed-by: Alexander Sverdlin <[email protected]>
---
drivers/mtd/spi-nor/spi-nor.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c
index 8bafd462f..18e826d36 100644
--- a/drivers/mtd/spi-nor/spi-nor.c
+++ b/drivers/mtd/spi-nor/spi-nor.c
@@ -1050,6 +1050,8 @@ static const struct flash_info spi_nor_ids[] = {
SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
{ "is25lp128", INFO(0x9d6018, 0, 64 * 1024, 256,
SECT_4K | SPI_NOR_DUAL_READ) },
+ { "is25lp080d", INFO(0x9d6014, 0, 64 * 1024, 16,
+ SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
/* Macronix */
{ "mx25l512e", INFO(0xc22010, 0, 64 * 1024, 1, SECT_4K) },
--
2.11.0
On 27/12/2017 14:58, Romain Porte wrote:
> diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c
> index 8bafd462f..18e826d36 100644
> --- a/drivers/mtd/spi-nor/spi-nor.c
> +++ b/drivers/mtd/spi-nor/spi-nor.c
> @@ -1050,6 +1050,8 @@ static const struct flash_info spi_nor_ids[] = {
> SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
> { "is25lp128", INFO(0x9d6018, 0, 64 * 1024, 256,
> SECT_4K | SPI_NOR_DUAL_READ) },
> + { "is25lp080d", INFO(0x9d6014, 0, 64 * 1024, 16,
> + SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>
Please note that the new SPI_NOR_DUAL_READ and SPI_NOR_QUAD_READ options
have not been tested on our hardware for the moment. The Tested-by flag
applies only for the previous configuration without these two new flags.
We may be able to test these additional flags in the future, but they
were added only from your (Cyrille's) review from previous patch.
Please let me know if you want to wait for our hardware test results
before merging into spi-nor/next, or if you are confident enough to
merge it as this.
Romain.
Hi Romain,
Le 27/12/2017 à 15:46, Romain Porte a écrit :
> On 27/12/2017 14:58, Romain Porte wrote:
>> diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c
>> index 8bafd462f..18e826d36 100644
>> --- a/drivers/mtd/spi-nor/spi-nor.c
>> +++ b/drivers/mtd/spi-nor/spi-nor.c
>> @@ -1050,6 +1050,8 @@ static const struct flash_info spi_nor_ids[] = {
>> SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>> { "is25lp128", INFO(0x9d6018, 0, 64 * 1024, 256,
>> SECT_4K | SPI_NOR_DUAL_READ) },
>> + { "is25lp080d", INFO(0x9d6014, 0, 64 * 1024, 16,
>> + SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>>
> Please note that the new SPI_NOR_DUAL_READ and SPI_NOR_QUAD_READ options have not been tested on our hardware for the moment. The Tested-by flag applies only for the previous configuration without these two new flags. We may be able to test these additional flags in the future, but they were added only from your (Cyrille's) review from previous patch.
>
> Please let me know if you want to wait for our hardware test results before merging into spi-nor/next, or if you are confident enough to merge it as this.
Based on the manufacturer's datasheet, I'd rather add the
SPI_NOR_{DUAL,QUAD}_READ flags now than later, even if not tested yet.
Otherwise, it would never be tested.
Of course, if you can test on your side, it would be even better!
However, who is the author of this patch? There is no Signed-off-by tag. So
I shouldn't take this patch without this tag.
Besides, the subject line should be something like:
"mtd: spi-nor: Add ISSI is25lp080d support"
>From Documentation/process/submitting-patches.rst:
"""
Describe your changes in imperative mood, e.g. "make xyzzy do frotz"
instead of "[This patch] makes xyzzy do frotz" or "[I] changed xyzzy
to do frotz", as if you are giving orders to the codebase to change
its behaviour.
"""
Finally, just to let you know if you're not very familiar with posting to
linux mailing lists: some may complain that you didn't configure your mail
client to truncate lines at the 75th or 80th column ;)
It's not a big deal but it's still better to follow the guide lines.
Best regards,
Cyrille
>
> Romain.
>
Add support for a new ISSI 1MB SPI NOR chip that was tested in our lab.
Datasheet is available at: http://www.issi.com/WW/pdf/25LP-WP080D.pdf
Testing was done only without the SPI_NOR_{DUAL,QUAD}_READ flags that
were added later, according to the datasheet.
Tested-by: Pascal Fabreges <[email protected]>
Reviewed-by: Alexander Sverdlin <[email protected]>
Signed-off-by: Romain Porte <[email protected]>
---
drivers/mtd/spi-nor/spi-nor.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c
index 8bafd462f..18e826d36 100644
--- a/drivers/mtd/spi-nor/spi-nor.c
+++ b/drivers/mtd/spi-nor/spi-nor.c
@@ -1050,6 +1050,8 @@ static const struct flash_info spi_nor_ids[] = {
SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
{ "is25lp128", INFO(0x9d6018, 0, 64 * 1024, 256,
SECT_4K | SPI_NOR_DUAL_READ) },
+ { "is25lp080d", INFO(0x9d6014, 0, 64 * 1024, 16,
+ SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
/* Macronix */
{ "mx25l512e", INFO(0xc22010, 0, 64 * 1024, 1, SECT_4K) },
--
2.11.0
Le 28/12/2017 à 11:03, Romain Porte a écrit :
> Add support for a new ISSI 1MB SPI NOR chip that was tested in our lab.
> Datasheet is available at: http://www.issi.com/WW/pdf/25LP-WP080D.pdf
>
> Testing was done only without the SPI_NOR_{DUAL,QUAD}_READ flags that
> were added later, according to the datasheet.
>
> Tested-by: Pascal Fabreges <[email protected]>
> Reviewed-by: Alexander Sverdlin <[email protected]>
> Signed-off-by: Romain Porte <[email protected]>
Applied to the spi-nor/next branch of linux-mtd
Thanks!
> ---
> drivers/mtd/spi-nor/spi-nor.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c
> index 8bafd462f..18e826d36 100644
> --- a/drivers/mtd/spi-nor/spi-nor.c
> +++ b/drivers/mtd/spi-nor/spi-nor.c
> @@ -1050,6 +1050,8 @@ static const struct flash_info spi_nor_ids[] = {
> SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
> { "is25lp128", INFO(0x9d6018, 0, 64 * 1024, 256,
> SECT_4K | SPI_NOR_DUAL_READ) },
> + { "is25lp080d", INFO(0x9d6014, 0, 64 * 1024, 16,
> + SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>
> /* Macronix */
> { "mx25l512e", INFO(0xc22010, 0, 64 * 1024, 1, SECT_4K) },
>