diff options
author | Ben Hutchings <ben@decadent.org.uk> | 2018-05-10 14:20:54 -0400 |
---|---|---|
committer | Boris Brezillon <boris.brezillon@bootlin.com> | 2018-05-14 08:46:20 -0400 |
commit | ea739a287f4f16d6250bea779a1026ead79695f2 (patch) | |
tree | f6d61738cb9ee66b8a2771a67e0389125b5f1447 | |
parent | 67b8d5c7081221efa252e111cd52532ec6d4266f (diff) |
mtd: Fix comparison in map_word_andequal()
Commit 9e343e87d2c4 ("mtd: cfi: convert inline functions to macros")
changed map_word_andequal() into a macro, but also changed the right
hand side of the comparison from val3 to val2. Change it back to use
val3 on the right hand side.
Thankfully this did not cause a regression because all callers
currently pass the same argument for val2 and val3.
Fixes: 9e343e87d2c4 ("mtd: cfi: convert inline functions to macros")
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Signed-off-by: Boris Brezillon <boris.brezillon@bootlin.com>
-rw-r--r-- | include/linux/mtd/map.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/mtd/map.h b/include/linux/mtd/map.h index b5b43f94f311..01b990e4b228 100644 --- a/include/linux/mtd/map.h +++ b/include/linux/mtd/map.h | |||
@@ -312,7 +312,7 @@ void map_destroy(struct mtd_info *mtd); | |||
312 | ({ \ | 312 | ({ \ |
313 | int i, ret = 1; \ | 313 | int i, ret = 1; \ |
314 | for (i = 0; i < map_words(map); i++) { \ | 314 | for (i = 0; i < map_words(map); i++) { \ |
315 | if (((val1).x[i] & (val2).x[i]) != (val2).x[i]) { \ | 315 | if (((val1).x[i] & (val2).x[i]) != (val3).x[i]) { \ |
316 | ret = 0; \ | 316 | ret = 0; \ |
317 | break; \ | 317 | break; \ |
318 | } \ | 318 | } \ |