From: Brian Norris <computersforpeace@gmail.com>
Date: Wed, 7 Sep 2011 20:13:41 +0000 (-0700)
Subject: mtd: nand: switch `check_pattern()' to standard `memcmp()'
X-Git-Tag: firefly_0821_release~3680^2~4158^2~15
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=75b66d8ccd5772b00a7328c2cf75bc506ec532a1;p=firefly-linux-kernel-4.4.55.git

mtd: nand: switch `check_pattern()' to standard `memcmp()'

A portion of the `check_pattern()' function is basically a `memcmp()'.
Since it's possible for `memcmp()' to be optimized for a particular
architecture, we should use it instead.

Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@intel.com>
---

diff --git a/drivers/mtd/nand/nand_bbt.c b/drivers/mtd/nand/nand_bbt.c
index 783093d1a2e5..5c9c0b2f09d4 100644
--- a/drivers/mtd/nand/nand_bbt.c
+++ b/drivers/mtd/nand/nand_bbt.c
@@ -107,10 +107,8 @@ static int check_pattern(uint8_t *buf, int len, int paglen, struct nand_bbt_desc
 	p += end;
 
 	/* Compare the pattern */
-	for (i = 0; i < td->len; i++) {
-		if (p[i] != td->pattern[i])
-			return -1;
-	}
+	if (memcmp(p, td->pattern, td->len))
+		return -1;
 
 	if (td->options & NAND_BBT_SCANEMPTY) {
 		p += td->len;