diff options
| author | Boris BREZILLON <boris.brezillon@free-electrons.com> | 2016-03-07 09:46:51 (GMT) |
|---|---|---|
| committer | Brian Norris <computersforpeace@gmail.com> | 2016-03-08 00:23:09 (GMT) |
| commit | f5b8aa78ef086248145363bf1ffe5ca4348b6a98 (patch) | |
| tree | e1cbe852ebab047c0acf6af3c7498b11d37d2786 /drivers/mtd/nand/brcmnand | |
| parent | 9ebfdf5b18493f338237ef9861a555c2f79b0c17 (diff) | |
| download | linux-f5b8aa78ef086248145363bf1ffe5ca4348b6a98.tar.xz | |
mtd: kill the ecclayout->oobavail field
ecclayout->oobavail is just redundant with the mtd->oobavail field.
Moreover, it prevents static const definition of ecc layouts since the
NAND framework is calculating this value based on the ecclayout->oobfree
field.
Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Diffstat (limited to 'drivers/mtd/nand/brcmnand')
| -rw-r--r-- | drivers/mtd/nand/brcmnand/brcmnand.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/drivers/mtd/nand/brcmnand/brcmnand.c b/drivers/mtd/nand/brcmnand/brcmnand.c index f7009c1..e052839 100644 --- a/drivers/mtd/nand/brcmnand/brcmnand.c +++ b/drivers/mtd/nand/brcmnand/brcmnand.c @@ -828,7 +828,8 @@ static struct nand_ecclayout *brcmnand_create_layout(int ecc_level, idx2 >= MTD_MAX_OOBFREE_ENTRIES_LARGE - 1) break; } - goto out; + + return layout; } /* @@ -879,10 +880,7 @@ static struct nand_ecclayout *brcmnand_create_layout(int ecc_level, idx2 >= MTD_MAX_OOBFREE_ENTRIES_LARGE - 1) break; } -out: - /* Sum available OOB */ - for (i = 0; i < MTD_MAX_OOBFREE_ENTRIES_LARGE; i++) - layout->oobavail += layout->oobfree[i].length; + return layout; } |