2021-08-30 08:48:54

by GONG, Ruiqi

[permalink] [raw]
Subject: [PATCH] mtd: Remove obsolete macros only used by the old nand_ecclayout struct

All uses of MTD_MAX_{OOBFREE,ECCPOS}_ENTRIES_LARGE have been removed as
commit ef5eeea6e911 ("mtd: nand: brcm: switch to mtd_ooblayout_ops") and
commit aab616e31d1c ("mtd: kill the nand_ecclayout struct") replaced
struct nand_ecclayout by the new mtd_ooblayout_ops interface. Remove
these two macros therefore.

Reported-by: Yi Yang <[email protected]>
Signed-off-by: GONG, Ruiqi <[email protected]>
---
include/linux/mtd/mtd.h | 2 --
1 file changed, 2 deletions(-)

diff --git a/include/linux/mtd/mtd.h b/include/linux/mtd/mtd.h
index 88227044fc86..f5e7dfc2e4e9 100644
--- a/include/linux/mtd/mtd.h
+++ b/include/linux/mtd/mtd.h
@@ -72,8 +72,6 @@ struct mtd_oob_ops {
uint8_t *oobbuf;
};

-#define MTD_MAX_OOBFREE_ENTRIES_LARGE 32
-#define MTD_MAX_ECCPOS_ENTRIES_LARGE 640
/**
* struct mtd_oob_region - oob region definition
* @offset: region offset
--
2.17.1


2021-09-14 17:44:24

by Miquel Raynal

[permalink] [raw]
Subject: Re: [PATCH] mtd: Remove obsolete macros only used by the old nand_ecclayout struct

On Mon, 2021-08-30 at 08:33:56 UTC, "GONG, Ruiqi" wrote:
> All uses of MTD_MAX_{OOBFREE,ECCPOS}_ENTRIES_LARGE have been removed as
> commit ef5eeea6e911 ("mtd: nand: brcm: switch to mtd_ooblayout_ops") and
> commit aab616e31d1c ("mtd: kill the nand_ecclayout struct") replaced
> struct nand_ecclayout by the new mtd_ooblayout_ops interface. Remove
> these two macros therefore.
>
> Reported-by: Yi Yang <[email protected]>
> Signed-off-by: GONG, Ruiqi <[email protected]>

Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks.

Miquel