Linux-mtd Archive on lore.kernel.org
 help / Atom feed
* [PATCH 1/2] mtd: spi-nor: Add support for EN25Q80A
@ 2019-01-23  7:56 Schrempf Frieder
  2019-01-23  7:56 ` [PATCH 2/2] mtd: spi-nor: Add support for MX25V8035F Schrempf Frieder
  2019-02-03 13:33 ` [PATCH 1/2] mtd: spi-nor: Add support for EN25Q80A Tudor.Ambarus
  0 siblings, 2 replies; 7+ messages in thread
From: Schrempf Frieder @ 2019-01-23  7:56 UTC (permalink / raw)
  To: Marek Vasut
  Cc: Boris Brezillon, Richard Weinberger, Tudor.Ambarus, linux-kernel,
	Schrempf Frieder, linux-mtd, Brian Norris, David Woodhouse

From: Frieder Schrempf <frieder.schrempf@kontron.de>

This adds support for the EON EN25Q80A, a 8Mb SPI NOR chip.

Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 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 6e13bbd1aaa5..aa8a04293a25 100644
--- a/drivers/mtd/spi-nor/spi-nor.c
+++ b/drivers/mtd/spi-nor/spi-nor.c
@@ -1737,6 +1737,8 @@ static const struct flash_info spi_nor_ids[] = {
 	/* EON -- en25xxx */
 	{ "en25f32",    INFO(0x1c3116, 0, 64 * 1024,   64, SECT_4K) },
 	{ "en25p32",    INFO(0x1c2016, 0, 64 * 1024,   64, 0) },
+	{ "en25q80a",   INFO(0x1c3014, 0, 64 * 1024,   16,
+			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
 	{ "en25q32b",   INFO(0x1c3016, 0, 64 * 1024,   64, 0) },
 	{ "en25p64",    INFO(0x1c2017, 0, 64 * 1024,  128, 0) },
 	{ "en25q64",    INFO(0x1c3017, 0, 64 * 1024,  128, SECT_4K) },
-- 
2.17.1

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

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

* [PATCH 2/2] mtd: spi-nor: Add support for MX25V8035F
  2019-01-23  7:56 [PATCH 1/2] mtd: spi-nor: Add support for EN25Q80A Schrempf Frieder
@ 2019-01-23  7:56 ` Schrempf Frieder
  2019-02-03 13:54   ` Tudor.Ambarus
  2019-02-03 13:33 ` [PATCH 1/2] mtd: spi-nor: Add support for EN25Q80A Tudor.Ambarus
  1 sibling, 1 reply; 7+ messages in thread
From: Schrempf Frieder @ 2019-01-23  7:56 UTC (permalink / raw)
  To: Marek Vasut
  Cc: Boris Brezillon, Richard Weinberger, Tudor.Ambarus, linux-kernel,
	Schrempf Frieder, linux-mtd, Brian Norris, David Woodhouse

From: Frieder Schrempf <frieder.schrempf@kontron.de>

This adds support for the Macronix MX25V8035F, a 8Mb SPI NOR chip.

Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 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 aa8a04293a25..f08ea9cf86b6 100644
--- a/drivers/mtd/spi-nor/spi-nor.c
+++ b/drivers/mtd/spi-nor/spi-nor.c
@@ -1849,6 +1849,8 @@ static const struct flash_info spi_nor_ids[] = {
 			 SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ)
 			 .fixups = &mx25l25635_fixups },
 	{ "mx25u25635f", INFO(0xc22539, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_4B_OPCODES) },
+	{ "mx25v8035f",  INFO(0xc22314, 0, 64 * 1024,  16,
+			 SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
 	{ "mx25l25655e", INFO(0xc22619, 0, 64 * 1024, 512, 0) },
 	{ "mx66l51235l", INFO(0xc2201a, 0, 64 * 1024, 1024, SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
 	{ "mx66u51235f", INFO(0xc2253a, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
-- 
2.17.1

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

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

* Re: [PATCH 1/2] mtd: spi-nor: Add support for EN25Q80A
  2019-01-23  7:56 [PATCH 1/2] mtd: spi-nor: Add support for EN25Q80A Schrempf Frieder
  2019-01-23  7:56 ` [PATCH 2/2] mtd: spi-nor: Add support for MX25V8035F Schrempf Frieder
@ 2019-02-03 13:33 ` Tudor.Ambarus
  2019-02-07 10:06   ` Schrempf Frieder
  1 sibling, 1 reply; 7+ messages in thread
From: Tudor.Ambarus @ 2019-02-03 13:33 UTC (permalink / raw)
  To: frieder.schrempf, marek.vasut
  Cc: bbrezillon, richard, linux-kernel, linux-mtd, computersforpeace, dwmw2

Hi, Frieder,

On 01/23/2019 09:56 AM, Schrempf Frieder wrote:
> From: Frieder Schrempf <frieder.schrempf@kontron.de>
> 
> This adds support for the EON EN25Q80A, a 8Mb SPI NOR chip.

I would suggest to specify who is using this flash and how did you test it. This
way we will not end up with support for flashes that are not actually used.

> 
> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
> ---
>  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 6e13bbd1aaa5..aa8a04293a25 100644
> --- a/drivers/mtd/spi-nor/spi-nor.c
> +++ b/drivers/mtd/spi-nor/spi-nor.c
> @@ -1737,6 +1737,8 @@ static const struct flash_info spi_nor_ids[] = {
>  	/* EON -- en25xxx */
>  	{ "en25f32",    INFO(0x1c3116, 0, 64 * 1024,   64, SECT_4K) },
>  	{ "en25p32",    INFO(0x1c2016, 0, 64 * 1024,   64, 0) },
> +	{ "en25q80a",   INFO(0x1c3014, 0, 64 * 1024,   16,
> +			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },

I'm reading EN25Q80A Rev. H, Issue Date: 2012/10/23
 datasheet. I don't see the bfpt table described, so probably it doesn't support
it. The flash advertises SPINOR_OP_READ_1_4_4 (0xeb), but not
SPINOR_OP_READ_1_1_4 (0x6b). In spi_nor_init_params(), when SPI_NOR_QUAD_READ is
set, we assume that SNOR_HWCAPS_READ_1_1_4 is supported, so we will use 0x6b for
quad reads. I can't see how the flash works with 0x6b, unless there is a bfpt
table that indicates support for 0xebh.

If it does support bfpt, set just SECT_4K | SPI_NOR_DUAL_READ, the latter will
trigger the bfpt parsing.

If you will resubmit, please order the entry in alphabetical order, by name.

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

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

* Re: [PATCH 2/2] mtd: spi-nor: Add support for MX25V8035F
  2019-01-23  7:56 ` [PATCH 2/2] mtd: spi-nor: Add support for MX25V8035F Schrempf Frieder
@ 2019-02-03 13:54   ` Tudor.Ambarus
  0 siblings, 0 replies; 7+ messages in thread
From: Tudor.Ambarus @ 2019-02-03 13:54 UTC (permalink / raw)
  To: frieder.schrempf, marek.vasut
  Cc: bbrezillon, richard, linux-kernel, linux-mtd, computersforpeace, dwmw2



On 01/23/2019 09:56 AM, Schrempf Frieder wrote:
> From: Frieder Schrempf <frieder.schrempf@kontron.de>
> 
> This adds support for the Macronix MX25V8035F, a 8Mb SPI NOR chip.

Please say who uses it and how you tested it. Looks good otherwise.

> 
> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
> ---
>  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 aa8a04293a25..f08ea9cf86b6 100644
> --- a/drivers/mtd/spi-nor/spi-nor.c
> +++ b/drivers/mtd/spi-nor/spi-nor.c
> @@ -1849,6 +1849,8 @@ static const struct flash_info spi_nor_ids[] = {
>  			 SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ)
>  			 .fixups = &mx25l25635_fixups },
>  	{ "mx25u25635f", INFO(0xc22539, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_4B_OPCODES) },
> +	{ "mx25v8035f",  INFO(0xc22314, 0, 64 * 1024,  16,
> +			 SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>  	{ "mx25l25655e", INFO(0xc22619, 0, 64 * 1024, 512, 0) },
>  	{ "mx66l51235l", INFO(0xc2201a, 0, 64 * 1024, 1024, SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
>  	{ "mx66u51235f", INFO(0xc2253a, 0, 64 * 1024, 1024, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) },
> 
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

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

* Re: [PATCH 1/2] mtd: spi-nor: Add support for EN25Q80A
  2019-02-03 13:33 ` [PATCH 1/2] mtd: spi-nor: Add support for EN25Q80A Tudor.Ambarus
@ 2019-02-07 10:06   ` Schrempf Frieder
  2019-02-07 10:16     ` Tudor.Ambarus
  0 siblings, 1 reply; 7+ messages in thread
From: Schrempf Frieder @ 2019-02-07 10:06 UTC (permalink / raw)
  To: Tudor.Ambarus, marek.vasut
  Cc: bbrezillon, richard, linux-kernel, linux-mtd, computersforpeace, dwmw2

Hi Tudor,

On 03.02.19 14:33, Tudor.Ambarus@microchip.com wrote:
> Hi, Frieder,
> 
> On 01/23/2019 09:56 AM, Schrempf Frieder wrote:
>> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>>
>> This adds support for the EON EN25Q80A, a 8Mb SPI NOR chip.
> 
> I would suggest to specify who is using this flash and how did you test it. This
> way we will not end up with support for flashes that are not actually used.

Ok. The flash is used by a board that I plan to upstream. Maybe I should 
just resubmit this together with the actual board support patches?

Likewise for my other patch (MX25V8035F), this is for another board I 
plan to upstream soon.

> 
>>
>> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
>> ---
>>   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 6e13bbd1aaa5..aa8a04293a25 100644
>> --- a/drivers/mtd/spi-nor/spi-nor.c
>> +++ b/drivers/mtd/spi-nor/spi-nor.c
>> @@ -1737,6 +1737,8 @@ static const struct flash_info spi_nor_ids[] = {
>>   	/* EON -- en25xxx */
>>   	{ "en25f32",    INFO(0x1c3116, 0, 64 * 1024,   64, SECT_4K) },
>>   	{ "en25p32",    INFO(0x1c2016, 0, 64 * 1024,   64, 0) },
>> +	{ "en25q80a",   INFO(0x1c3014, 0, 64 * 1024,   16,
>> +			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
> 
> I'm reading EN25Q80A Rev. H, Issue Date: 2012/10/23
>   datasheet. I don't see the bfpt table described, so probably it doesn't support
> it. The flash advertises SPINOR_OP_READ_1_4_4 (0xeb), but not
> SPINOR_OP_READ_1_1_4 (0x6b). In spi_nor_init_params(), when SPI_NOR_QUAD_READ is
> set, we assume that SNOR_HWCAPS_READ_1_1_4 is supported, so we will use 0x6b for
> quad reads. I can't see how the flash works with 0x6b, unless there is a bfpt
> table that indicates support for 0xebh.
> 
> If it does support bfpt, set just SECT_4K | SPI_NOR_DUAL_READ, the latter will
> trigger the bfpt parsing.

Thanks for explaining this. I missed the point, that SPI_NOR_QUAD_READ 
actually requires support for SPINOR_OP_READ_1_1_4.

> 
> If you will resubmit, please order the entry in alphabetical order, by name.

Ok.

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

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

* Re: [PATCH 1/2] mtd: spi-nor: Add support for EN25Q80A
  2019-02-07 10:06   ` Schrempf Frieder
@ 2019-02-07 10:16     ` Tudor.Ambarus
  2019-02-07 10:48       ` Boris Brezillon
  0 siblings, 1 reply; 7+ messages in thread
From: Tudor.Ambarus @ 2019-02-07 10:16 UTC (permalink / raw)
  To: frieder.schrempf, marek.vasut
  Cc: bbrezillon, richard, linux-kernel, linux-mtd, computersforpeace, dwmw2

Hi, Frieder,

On 02/07/2019 12:06 PM, Schrempf Frieder wrote:
> Hi Tudor,
> 
> On 03.02.19 14:33, Tudor.Ambarus@microchip.com wrote:
>> Hi, Frieder,
>>
>> On 01/23/2019 09:56 AM, Schrempf Frieder wrote:
>>> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>>>
>>> This adds support for the EON EN25Q80A, a 8Mb SPI NOR chip.
>> I would suggest to specify who is using this flash and how did you test it. This
>> way we will not end up with support for flashes that are not actually used.
> Ok. The flash is used by a board that I plan to upstream. Maybe I should 
> just resubmit this together with the actual board support patches?
> 
> Likewise for my other patch (MX25V8035F), this is for another board I 
> plan to upstream soon.
> 

Sounds good.

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

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

* Re: [PATCH 1/2] mtd: spi-nor: Add support for EN25Q80A
  2019-02-07 10:16     ` Tudor.Ambarus
@ 2019-02-07 10:48       ` Boris Brezillon
  0 siblings, 0 replies; 7+ messages in thread
From: Boris Brezillon @ 2019-02-07 10:48 UTC (permalink / raw)
  To: Tudor.Ambarus
  Cc: richard, linux-kernel, frieder.schrempf, marek.vasut, linux-mtd,
	computersforpeace, dwmw2

On Thu, 7 Feb 2019 10:16:05 +0000
<Tudor.Ambarus@microchip.com> wrote:

> Hi, Frieder,
> 
> On 02/07/2019 12:06 PM, Schrempf Frieder wrote:
> > Hi Tudor,
> > 
> > On 03.02.19 14:33, Tudor.Ambarus@microchip.com wrote:  
> >> Hi, Frieder,
> >>
> >> On 01/23/2019 09:56 AM, Schrempf Frieder wrote:  
> >>> From: Frieder Schrempf <frieder.schrempf@kontron.de>
> >>>
> >>> This adds support for the EON EN25Q80A, a 8Mb SPI NOR chip.  
> >> I would suggest to specify who is using this flash and how did you test it. This
> >> way we will not end up with support for flashes that are not actually used.  
> > Ok. The flash is used by a board that I plan to upstream. Maybe I should 
> > just resubmit this together with the actual board support patches?

No need to wait for the board patches, they'll be applied independently
anyway. Just mention the board you tested it on in the commit message
(even if it's not upstream yet) and we should be good.

Thanks,

Boris

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

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

end of thread, back to index

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-23  7:56 [PATCH 1/2] mtd: spi-nor: Add support for EN25Q80A Schrempf Frieder
2019-01-23  7:56 ` [PATCH 2/2] mtd: spi-nor: Add support for MX25V8035F Schrempf Frieder
2019-02-03 13:54   ` Tudor.Ambarus
2019-02-03 13:33 ` [PATCH 1/2] mtd: spi-nor: Add support for EN25Q80A Tudor.Ambarus
2019-02-07 10:06   ` Schrempf Frieder
2019-02-07 10:16     ` Tudor.Ambarus
2019-02-07 10:48       ` Boris Brezillon

Linux-mtd Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-mtd/0 linux-mtd/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-mtd linux-mtd/ https://lore.kernel.org/linux-mtd \
		linux-mtd@lists.infradead.org linux-mtd@archiver.kernel.org
	public-inbox-index linux-mtd


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.infradead.lists.linux-mtd


AGPL code for this site: git clone https://public-inbox.org/ public-inbox