Hi Jaime,
+ linux-mtd which was missing to your contribution.
jaimeliao.tw@xxxxxxxxx wrote on Thu, 25 Jan 2024 10:48:16 +0800:
> From: JaimeLiao <jaimeliao@xxxxxxxxxxx>
>
> Support for MX35LF{2,4}GE4AD chips was added in mainline through
> upstream commit 5ece78de88739b4c68263e9f2582380c1fd8314f.
>
> The patch was later adapted to 5.4.y and backported through
> stable commit 85258ae3070848d9d0f6fbee385be2db80e8cf26.
>
> Fix the backport mentioned right above as it is wrong: the bigger chip
> features 4kiB pages and not 2kiB pages.
>
> Fixes: 85258ae30708 ("mtd: spinand: macronix: Add support for MX35LFxGE4AD")
> Cc: stable@xxxxxxxxxxxxxxx # v5.4.y
> Cc: Miquel Raynal <miquel.raynal@xxxxxxxxxxx>
> Signed-off-by: JaimeLiao <jaimeliao@xxxxxxxxxxx>
Looks legitimate.
Acked-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx>
> ---
> Hello,
>
> This is my third attempt to fix a stable kernel. This patch is not a
> backport from Linus' tree per-se, but a fix of a backport. The original
> mainline commit is fine but the backported one is not, we need to fix
> the backported commit in the 5.4.y stable kernel, and this is what I am
> attempting to do. Let me know if further explanations are needed.
>
> Regards,
> Jaime
> ---
> drivers/mtd/nand/spi/macronix.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mtd/nand/spi/macronix.c b/drivers/mtd/nand/spi/macronix.c
> index bbb1d68bce4a..f18c6cfe8ff5 100644
> --- a/drivers/mtd/nand/spi/macronix.c
> +++ b/drivers/mtd/nand/spi/macronix.c
> @@ -125,7 +125,7 @@ static const struct spinand_info macronix_spinand_table[] = {
> SPINAND_HAS_QE_BIT,
> SPINAND_ECCINFO(&mx35lfxge4ab_ooblayout, NULL)),
> SPINAND_INFO("MX35LF4GE4AD", 0x37,
> - NAND_MEMORG(1, 2048, 128, 64, 2048, 40, 1, 1, 1),
> + NAND_MEMORG(1, 4096, 128, 64, 2048, 40, 1, 1, 1),
> NAND_ECCREQ(8, 512),
> SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
> &write_cache_variants,
Thanks,
Miquèl
