From 2c8cfdcbeb1ab0ec7bbd5af1be311b55281154c4 Mon Sep 17 00:00:00 2001
From: David Woodhouse <dwmw2@infradead.org>
Date: Fri, 27 Oct 2006 09:53:08 +0300
Subject: [PATCH] =?UTF-8?q?[MTD]=20NAND:=20Caf=C3=A9=20ECC=20--=20remove?=
=?UTF-8?q?=20spurious=20BUG=5FON()=20in=20err=5Fpos()?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Being a value which isn't in the table is a case we explicitly check for
in the caller. Don't BUG_ON() because it does actually happen in
practice.
Signed-off-by: David Woodhouse <dwmw2@infradead.org>
---
drivers/mtd/nand/cafe_ecc.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/mtd/nand/cafe_ecc.c b/drivers/mtd/nand/cafe_ecc.c
index c4bec37e73eb..46214602d205 100644
--- a/drivers/mtd/nand/cafe_ecc.c
+++ b/drivers/mtd/nand/cafe_ecc.c
@@ -1045,7 +1045,6 @@ static unsigned short err_pos_lut[4096] = {
static unsigned short err_pos(unsigned short din)
{
BUG_ON(din > 4096);
- BUG_ON(err_pos_lut[din] == 0xfff);
return err_pos_lut[din];
}
static int chk_no_err_only(unsigned short *chk_syndrome_list, unsigned short *err_info)
--
GitLab