[PATCH] mtd: spi-nor: infineon: add Cypress (Ramtron) CY15B102QN F-RAM support

[PATCH] mtd: spi-nor: infineon: add Cypress (Ramtron) CY15B102QN F-RAM support

[A. Sverdlin]

From: Alexander Sverdlin <alexander.sverdlin@siemens.com>

Similar sad story as with "cy15x104q" where JEDEC ID comes last-byte-first.

Link: https://www.infineon.com/dgdl/Infineon-CY15B102QN_CY15V102QN_EXCELON_-AUTO_2-MBIT_(256K_X_8)_AUTOMOTIVE-E_SERIAL_(SPI)_F-RAM-DataSheet-v15_00-EN.pdf?fileId=8ac78c8c7d0d8da4017d0ee5bc3f6d8a
Signed-off-by: Alexander Sverdlin <alexander.sverdlin@siemens.com>
---
 drivers/mtd/spi-nor/spansion.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/mtd/spi-nor/spansion.c b/drivers/mtd/spi-nor/spansion.c
index d6c92595f6bc9..3515ede0c3b80 100644
--- a/drivers/mtd/spi-nor/spansion.c
+++ b/drivers/mtd/spi-nor/spansion.c
@@ -915,6 +915,12 @@ static const struct flash_info spansion_nor_parts[] = {
 		.size = SZ_32M,
 		.no_sfdp_flags = SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ,
 		.fixup_flags = SPI_NOR_4B_OPCODES,
+	}, {
+		.id = SNOR_ID(0x60, 0x2a, 0xc2, 0x7f, 0x7f, 0x7f),
+		.name = "cy15b102qn",
+		.size = SZ_256K,
+		.sector_size = SZ_256K,
+		.flags = SPI_NOR_NO_ERASE,
 	}, {
 		.id = SNOR_ID(0x04, 0x2c, 0xc2, 0x7f, 0x7f, 0x7f),
 		.name = "cy15x104q",
-- 
2.46.1


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

Re: [PATCH] mtd: spi-nor: infineon: add Cypress (Ramtron) CY15B102QN F-RAM support

[Michael Walle]

[-- Attachment #1.1: Type: text/plain, Size: 248 bytes --]

Hi,

> Similar sad story as with "cy15x104q" where JEDEC ID comes last-byte-first.

Please have a look at
https://lore.kernel.org/linux-mtd/20240729102016.8527-2-alessandro.zini@siemens.com/

TLDR, please have a look at at25.c.

-michael

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 297 bytes --]

[-- Attachment #2: Type: text/plain, Size: 144 bytes --]

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

Re: [PATCH] mtd: spi-nor: infineon: add Cypress (Ramtron) CY15B102QN F-RAM support

[Sverdlin, Alexander]

Hello Michael!

On Tue, 2024-09-24 at 10:23 +0200, Michael Walle wrote:
> > Similar sad story as with "cy15x104q" where JEDEC ID comes last-byte-first.
> 
> Please have a look at
> https://lore.kernel.org/linux-mtd/20240729102016.8527-2-alessandro.zini@siemens.com/
> 
> TLDR, please have a look at at25.c.

It unfortunately doesn't work with SPI host drivers which only implement spi-mem API.
Or did I miss something? Otherwise "cy15x104q" could also be handled by at25.c.

-- 
Alexander Sverdlin
Siemens AG
www.siemens.com
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

Re: [PATCH] mtd: spi-nor: infineon: add Cypress (Ramtron) CY15B102QN F-RAM support

[Michael Walle]

Hi,

>> > Similar sad story as with "cy15x104q" where JEDEC ID comes last-byte-first.
>> 
>> Please have a look at
>> https://lore.kernel.org/linux-mtd/20240729102016.8527-2-alessandro.zini@siemens.com/
>> 
>> TLDR, please have a look at at25.c.
> 
> It unfortunately doesn't work with SPI host drivers which only 
> implement spi-mem API.
> Or did I miss something? Otherwise "cy15x104q" could also be handled by 
> at25.c.

The solution here is to add spi-mem support to the at25 driver :) Maybe 
you
can work together wit Hui Wang [1].

-michael

[1] 
https://lore.kernel.org/linux-mtd/20240408124328.777375-1-hui.wang@canonical.com/

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

Re: [PATCH] mtd: spi-nor: infineon: add Cypress (Ramtron) CY15B102QN F-RAM support

[Sverdlin, Alexander]

Hi Michael,

On Wed, 2024-09-25 at 21:19 +0200, Michael Walle wrote:
> > > > Similar sad story as with "cy15x104q" where JEDEC ID comes last-byte-first.
> > > 
> > > Please have a look at
> > > https://lore.kernel.org/linux-mtd/20240729102016.8527-2-alessandro.zini@siemens.com/
> > > 
> > > TLDR, please have a look at at25.c.
> > 
> > It unfortunately doesn't work with SPI host drivers which only 
> > implement spi-mem API.
> > Or did I miss something? Otherwise "cy15x104q" could also be handled by 
> > at25.c.
> 
> The solution here is to add spi-mem support to the at25 driver :) Maybe 
> you
> can work together wit Hui Wang [1].
> 
> -michael
> 
> [1] 
> https://lore.kernel.org/linux-mtd/20240408124328.777375-1-hui.wang@canonical.com/

thanks for the reference!

FWIW, F-RAMs can work without SPI_NOR_NO_ERASE flag, so neither is it blocking point
to remove it, nor will it be broken by such removal. Even though they do not support
the command, they seem to simply ignore unknown commands and do not alter BUSY/READY
status bit.

But I've got your point and can look into it with lower prio...

-- 
Alexander Sverdlin
Siemens AG
www.siemens.com
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/