On 04.04.2023 10:45, Tudor Ambarus wrote:
>
>
> On 04.04.2023 10:17, tkuw584924@xxxxxxxxx wrote:
>> From: Takahiro Kuwano <Takahiro.Kuwano@xxxxxxxxxxxx>
>>
>> Infineon(Cypress) SEMPER NOR flash family has on-die ECC and its program
>> granularity is 16-byte ECC data unit size. JFFS2 supports write buffer
>> mode for ECC'd NOR flash. To activate it, MTD_BIT_WRITEABLE needs to be
>> unset in mtd->flags.
>>
>> A new SNOR_F_ECC flag is introduced to determine if the part has on-die
>> ECC and if it has, MTD_BIT_WRITEABLE is unset.
>>
>> In vendor specific driver, a common cypress_nor_ecc_init() helper is
>> added. This helper takes care for ECC related initialization for SEMPER
>> flash family by setting up params->writesize and SNOR_F_ECC.
>>
>> Fixes: 6afcc84080c4 ("mtd: spi-nor: spansion: Add support for Infineon S25FS256T")
>> Fixes: b6b23833fc42 ("mtd: spi-nor: spansion: Add s25hl-t/s25hs-t IDs and fixups")
>> Fixes: c3266af101f2 ("mtd: spi-nor: spansion: add support for Cypress Semper flash")
>
> Would you please split this in 3 patches, first fixing c3266af101f2,
> then b6b23833fc42 and then 6afcc84080c4? It will help stable team
> backport each for each flash affected.
>
> Looks good otherwise.
Here's how you can specify patch prerequisites:
https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html#option-1
'''
Additionally, some patches submitted via Option 1 may have additional
patch prerequisites which can be cherry-picked. This can be specified in
the following format in the sign-off area:
Cc: <stable@xxxxxxxxxxxxxxx> # 3.3.x: a1f84a3: sched: Check for idle
Cc: <stable@xxxxxxxxxxxxxxx> # 3.3.x: 1b9508f: sched: Rate-limit newidle
Cc: <stable@xxxxxxxxxxxxxxx> # 3.3.x: fd21073: sched: Fix affinity logic
Cc: <stable@xxxxxxxxxxxxxxx> # 3.3.x
Signed-off-by: Ingo Molnar <mingo@xxxxxxx>
'''
