diff options
| author |   Brian Norris <computersforpeace@gmail.com> | 2017-05-01 17:04:55 -0700 |
|---|---|---|
| committer |   Boris Brezillon <boris.brezillon@free-electrons.com> | 2017-05-22 09:42:29 +0200 |
| commit | d24197907454b902908e025bce4b8bc677d3ba6b (patch) | |
| tree | 3f5ca1529e25a793ef58553730707bef3f0915e6 /drivers/mtd/nand | |
| parent | mtd: nand: free vendor-specific resources in init failure paths (diff) | |
| download | linux-dev-d24197907454b902908e025bce4b8bc677d3ba6b.tar.xz linux-dev-d24197907454b902908e025bce4b8bc677d3ba6b.zip | |
mtd: nand: samsung: warn about un-parseable ECC info
We don't handle cases larger than 7. We probably shouldn't pretend we
know the ECC step size in this case, and it's probably also good to
WARN() like we do in many other similar cases.
Fixes: 8fc82d456e40 ("mtd: nand: samsung: Retrieve ECC requirements from extended ID")
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
Diffstat (limited to 'drivers/mtd/nand')
| -rw-r--r-- | drivers/mtd/nand/nand_samsung.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/mtd/nand/nand_samsung.c b/drivers/mtd/nand/nand_samsung.c index 9cfc4035a420..1e0755997762 100644 --- a/drivers/mtd/nand/nand_samsung.c +++ b/drivers/mtd/nand/nand_samsung.c @@ -84,6 +84,9 @@ static void samsung_nand_decode_id(struct nand_chip *chip) case 7: chip->ecc_strength_ds = 60; break; + default: + WARN(1, "Could not decode ECC info"); + chip->ecc_step_ds = 0; } } } else { |