From: David Woodhouse <dwmw2@infradead.org>
Date: Fri, 27 Oct 2006 06:53:08 +0000 (+0300)
Subject: [MTD] NAND: Café ECC -- remove spurious BUG_ON() in err_pos()
X-Git-Tag: v2.6.20-rc6~47^2~37^2~8
X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2c8cfdcbeb1ab0ec7bbd5af1be311b55281154c4;p=linux-2.6

[MTD] NAND: Café ECC -- remove spurious BUG_ON() in err_pos()

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>
---

diff --git a/drivers/mtd/nand/cafe_ecc.c b/drivers/mtd/nand/cafe_ecc.c
index c4bec37e73..46214602d2 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)