Hi Jagan, On 2/26/2021 7:35 PM, Jagan Teki wrote: > On Fri, Feb 19, 2021 at 7:26 AM <tkuw584...@gmail.com> wrote: >> >> From: Takahiro Kuwano <takahiro.kuw...@infineon.com> >> >> The S25HL-T/S25HS-T family is the Cypress Semper Flash with Quad SPI. >> >> https://www.cypress.com/file/424146/download (256Mb/512Mb/1Gb, single die) >> https://www.cypress.com/file/499246/download (2Gb/4Gb, dual/quad die) >> >> The full version can be found in the following links (registration >> required). >> https://community.cypress.com/t5/Semper-Flash-Access-Program/Datasheet-Semper-Flash-with-Quad-SPI/ta-p/260789?attachment-id=19522 >> https://community.cypress.com/t5/Semper-Flash-Access-Program/Datasheet-2Gb-MCP-Semper-Flash-with-Quad-SPI/ta-p/260823?attachment-id=29503 >> >> Tested on Xilinx Zynq-7000 FPGA board. >> >> Signed-off-by: Takahiro Kuwano <takahiro.kuw...@infineon.com> >> Reviewed-by: Pratyush Yadav <p.ya...@ti.com> >> --- >> Changes in v5: >> - Remove 256Mb and 4Gb parts >> >> drivers/mtd/spi/spi-nor-ids.c | 27 +++++++++++++++++++++++++++ >> 1 file changed, 27 insertions(+) >> >> diff --git a/drivers/mtd/spi/spi-nor-ids.c b/drivers/mtd/spi/spi-nor-ids.c >> index 5bd5dd3003..052a0b62ee 100644 >> --- a/drivers/mtd/spi/spi-nor-ids.c >> +++ b/drivers/mtd/spi/spi-nor-ids.c >> @@ -217,6 +217,33 @@ const struct flash_info spi_nor_ids[] = { >> { INFO("s25fl208k", 0x014014, 0, 64 * 1024, 16, SECT_4K | >> SPI_NOR_DUAL_READ) }, >> { INFO("s25fl064l", 0x016017, 0, 64 * 1024, 128, SECT_4K | >> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) }, >> { INFO("s25fl128l", 0x016018, 0, 64 * 1024, 256, SECT_4K | >> SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES) }, >> + >> + /* >> + * S25HL/HS-T (Semper Flash with Quad SPI) Family has >> user-configurable >> + * sector architecture. By default, the 512Mb and 1Gb, single-die >> + * package parts are configured to non-uniform that 4KB sectors >> overlaid >> + * on bottom address. To support this, an erase hook makes overlaid >> + * sectors appear as uniform sectors. The 2Gb, dual-die package parts >> + * are configured to uniform by default. >> + */ > > Nice to have these comments in commit instead of id's sections. > I will move it to commit in v6.
Thanks, Takahiro
