mbox series

[RFC,0/3] mtd: spi-nor: Add support of Sector Map Parameter Table

Message ID 1514548432-13302-1-git-send-email-prabhakar.kushwaha@nxp.com
Headers show
Series mtd: spi-nor: Add support of Sector Map Parameter Table | expand

Message

Prabhakar Kushwaha Dec. 29, 2017, 11:53 a.m. UTC 
JESD216B defines Sector Map Parameter Table. It identifies the
location and size of sectors within the main data array of the
flash memory device and identifies which Erase Types are supported
by each sector. This table is used when a memory device has sectors
of more than one size Or, does not allow all Erase Type commands to
be applied to all sectors.

This patch-set add support of Sector Map Parameter Table parsing.
It also update framework to support read/write anyreg commands
required by S25FS512S flash.

Tested with "S25FS512S" flash and by over-riding fsl_qspi_read_anyreg()
return with desired configuration id i.e. 0x05. Flash was
pre-programmed for Uniform 256-kB sectors via non-volatile registers.

Prabhakar Kushwaha (3):
  mtd: spi-nor: Add support of read/write any reg commands
  mtd: spi-nor: Implement anyreg functions for fsl-quadspi
  mtd: spi-nor: Add support of Sector Map Parameter Table

 drivers/mtd/spi-nor/fsl-quadspi.c |  18 +++
 drivers/mtd/spi-nor/spi-nor.c     | 266 +++++++++++++++++++++++++++++++++++++-
 include/linux/mtd/spi-nor.h       |  21 +++
 3 files changed, 302 insertions(+), 3 deletions(-)

Comments

Prabhakar Kushwaha Jan. 4, 2018, 1:33 a.m. UTC | #1 
Hi Cyrille,


> -----Original Message-----
> From: Prabhakar Kushwaha [mailto:prabhakar.kushwaha@nxp.com]
> Sent: Friday, December 29, 2017 5:24 PM
> To: linux-mtd@lists.infradead.org
> Cc: computersforpeace@gmail.com; boris.brezillon@free-electrons.com;
> marek.vasut@gmail.com; cyrille.pitchen@wedev4u.fr; Poonam Aggrwal
> <poonam.aggrwal@nxp.com>; Suresh Gupta <suresh.gupta@nxp.com>; Ashish
> Kumar <ashish.kumar@nxp.com>; Yogesh Narayan Gaur
> <yogeshnarayan.gaur@nxp.com>; Rajat Srivastava <rajat.srivastava@nxp.com>;
> Fabio Estevam <fabio.estevam@nxp.com>; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>
> Subject: [RFC 0/3] mtd: spi-nor: Add support of Sector Map Parameter Table
> 
> JESD216B defines Sector Map Parameter Table. It identifies the
> location and size of sectors within the main data array of the
> flash memory device and identifies which Erase Types are supported
> by each sector. This table is used when a memory device has sectors
> of more than one size Or, does not allow all Erase Type commands to
> be applied to all sectors.
> 
> This patch-set add support of Sector Map Parameter Table parsing.
> It also update framework to support read/write anyreg commands
> required by S25FS512S flash.
> 
> Tested with "S25FS512S" flash and by over-riding fsl_qspi_read_anyreg()
> return with desired configuration id i.e. 0x05. Flash was
> pre-programmed for Uniform 256-kB sectors via non-volatile registers.
> 
> Prabhakar Kushwaha (3):
>   mtd: spi-nor: Add support of read/write any reg commands
>   mtd: spi-nor: Implement anyreg functions for fsl-quadspi
>   mtd: spi-nor: Add support of Sector Map Parameter Table
> 
>  drivers/mtd/spi-nor/fsl-quadspi.c |  18 +++
>  drivers/mtd/spi-nor/spi-nor.c     | 266
> +++++++++++++++++++++++++++++++++++++-
>  include/linux/mtd/spi-nor.h       |  21 +++
>  3 files changed, 302 insertions(+), 3 deletions(-)
> 

Can you please review this patch-set. 
I want to incorporate review comments for the v1 patch.

Thanks
-pk