diff options
| author | Miquel Raynal <miquel.raynal@bootlin.com> | 2020-04-28 11:43:01 +0200 |
|---|---|---|
| committer | Miquel Raynal <miquel.raynal@bootlin.com> | 2020-05-11 09:51:40 +0200 |
| commit | 2e8f56f2a9417692b758fee9ee5a01d857dd980f (patch) | |
| tree | a1f452dfea4ae18f25f1442d1ee919579f525b42 /drivers/mtd/nand/raw/nand_jedec.c | |
| parent | 6e9c65d87c6d92ed988ecdab3ae6ed4d7d94f67c (diff) | |
| download | linux-2e8f56f2a9417692b758fee9ee5a01d857dd980f.tar.bz2 | |
mtd: rawnand: jedec: Define the number of parameter pages
Use a macro to define the number of parameter page instead of
hardcoding it everywhere.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com>
Link: https://lore.kernel.org/linux-mtd/20200428094302.14624-9-miquel.raynal@bootlin.com
Diffstat (limited to 'drivers/mtd/nand/raw/nand_jedec.c')
| -rw-r--r-- | drivers/mtd/nand/raw/nand_jedec.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/mtd/nand/raw/nand_jedec.c b/drivers/mtd/nand/raw/nand_jedec.c index 9b540e76f84f..0cd322a8be24 100644 --- a/drivers/mtd/nand/raw/nand_jedec.c +++ b/drivers/mtd/nand/raw/nand_jedec.c @@ -16,6 +16,8 @@ #include "internals.h" +#define JEDEC_PARAM_PAGES 3 + /* * Check if the NAND chip is JEDEC compliant, returns 1 if it is, 0 otherwise. */ @@ -47,7 +49,7 @@ int nand_jedec_detect(struct nand_chip *chip) goto free_jedec_param_page; } - for (i = 0; i < 3; i++) { + for (i = 0; i < JEDEC_PARAM_PAGES; i++) { ret = nand_read_data_op(chip, p, sizeof(*p), true); if (ret) { ret = 0; @@ -59,7 +61,7 @@ int nand_jedec_detect(struct nand_chip *chip) break; } - if (i == 3) { + if (i == JEDEC_PARAM_PAGES) { pr_err("Could not find valid JEDEC parameter page; aborting\n"); goto free_jedec_param_page; } |