[PATCH] mtd: spi-nor: spansion: Remove s70fl01gs from flash_info

[PATCH] mtd: spi-nor: spansion: Remove s70fl01gs from flash_info

[kuwano.takahiro]

From: Takahiro Kuwano <Takahiro.Kuwano@cypress.com>

The s70fl01gs is a dual die stack of two s25fl512s die with dedicated chip
select pins to each. Tested with the device and confirmed that is working
as two s25fl512s devices. The current device ID in the flash_info table
matches with s70fs01gs which does not work with current MTD.

Signed-off-by: Takahiro Kuwano <Takahiro.Kuwano@cypress.com>
---
 drivers/mtd/spi-nor/spansion.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/mtd/spi-nor/spansion.c b/drivers/mtd/spi-nor/spansion.c
index e550cd5c9d3a..953df1c488ec 100644
--- a/drivers/mtd/spi-nor/spansion.c
+++ b/drivers/mtd/spi-nor/spansion.c
@@ -64,7 +64,6 @@ static const struct flash_info spansion_parts[] = {
 	{ "s25fs512s",  INFO6(0x010220, 0x4d0081, 256 * 1024, 256,
 			      SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | USE_CLSR)
 	  .fixups = &s25fs_s_fixups, },
-	{ "s70fl01gs",  INFO(0x010221, 0x4d00, 256 * 1024, 256, 0) },
 	{ "s25sl12800", INFO(0x012018, 0x0300, 256 * 1024,  64, 0) },
 	{ "s25sl12801", INFO(0x012018, 0x0301,  64 * 1024, 256, 0) },
 	{ "s25fl129p0", INFO(0x012018, 0x4d00, 256 * 1024,  64,
-- 
2.17.1


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

Re: [PATCH] mtd: spi-nor: spansion: Remove s70fl01gs from flash_info

[Tudor.Ambarus]

On 6/26/20 8:16 AM, kuwano.takahiro@gmail.com wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> From: Takahiro Kuwano <Takahiro.Kuwano@cypress.com>
> 
> The s70fl01gs is a dual die stack of two s25fl512s die with dedicated chip
> select pins to each. Tested with the device and confirmed that is working
> as two s25fl512s devices. The current device ID in the flash_info table

Right, the datasheet specifies too that the S70FS01GS device is a dual die
stack of two FS512S die.

> matches with s70fs01gs which does not work with current MTD.

s70fs01gs is not correctly supported indeed, but the lower half of it
(first s25fl512s die) should work. Can't you correctly read/erase/write
on the lower half of the s70fs01gs flash with the current code?

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

Re: [PATCH] mtd: spi-nor: spansion: Remove s70fl01gs from flash_info

[Takahiro Kuwano]

On 7/3/20 9:20 PM, Tudor.Ambarus@microchip.com wrote:
> On 6/26/20 8:16 AM, kuwano.takahiro@gmail.com wrote:
>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>
>> From: Takahiro Kuwano <Takahiro.Kuwano@cypress.com>
>>
>> The s70fl01gs is a dual die stack of two s25fl512s die with dedicated chip
>> select pins to each. Tested with the device and confirmed that is working
>> as two s25fl512s devices. The current device ID in the flash_info table
> 
> Right, the datasheet specifies too that the S70FS01GS device is a dual die
> stack of two FS512S die.
> 
>> matches with s70fs01gs which does not work with current MTD.
> 
> s70fs01gs is not correctly supported indeed, but the lower half of it
> (first s25fl512s die) should work. Can't you correctly read/erase/write
> on the lower half of the s70fs01gs flash with the current code?

No, because the S70FS01GS does not support RDSR(05h) which is critical
for erase/write. The S70FS01GS is a dual die stack with one chip select.
Accessing to the registers requires RDAR(65h)/WRAR(71h) designating die
and register address.  

Best Regards,
Takahiro


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

Re: [PATCH] mtd: spi-nor: spansion: Remove s70fl01gs from flash_info

[Tudor Ambarus]

On Fri, 26 Jun 2020 14:16:50 +0900, kuwano.takahiro@gmail.com wrote:
> The s70fl01gs is a dual die stack of two s25fl512s die with dedicated chip
> select pins to each. Tested with the device and confirmed that is working
> as two s25fl512s devices. The current device ID in the flash_info table
> matches with s70fs01gs which does not work with current MTD.

Applied to spi-nor/next, thanks!

[1/1] mtd: spi-nor: spansion: Remove s70fl01gs from flash_info
      https://git.kernel.org/mtd/c/8c8f97573eb3

Best regards,
-- 
Tudor Ambarus <tudor.ambarus@microchip.com>

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