[PATCH v5 02/10] mtd: spi-nor-ids: Add Cypress s25hl-t/s25hs-t

[Takahiro Kuwano]

Hi Jagan,

On 2/26/2021 7:35 PM, Jagan Teki wrote:
> On Fri, Feb 19, 2021 at 7:26 AM <tkuw584924 at gmail.com> wrote:
>>
>> From: Takahiro Kuwano <Takahiro.Kuwano at 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.Kuwano at infineon.com>
>> Reviewed-by: Pratyush Yadav <p.yadav at 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