mtd: use min_t() to refactor mtd_ooblayout_{get, set}_bytes()
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Wed, 9 Nov 2016 02:08:09 +0000 (11:08 +0900)
committerBoris Brezillon <boris.brezillon@free-electrons.com>
Sat, 19 Nov 2016 18:31:32 +0000 (19:31 +0100)
I hope this will make the code a little more readable.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Reviewed-by: Marek Vasut <marek.vasut@gmail.com>
Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
drivers/mtd/mtdcore.c

index cf85f2b..ca6a89a 100644 (file)
@@ -1283,7 +1283,7 @@ static int mtd_ooblayout_get_bytes(struct mtd_info *mtd, u8 *buf,
        while (!ret) {
                int cnt;
 
-               cnt = oobregion.length > nbytes ? nbytes : oobregion.length;
+               cnt = min_t(int, nbytes, oobregion.length);
                memcpy(buf, oobbuf + oobregion.offset, cnt);
                buf += cnt;
                nbytes -= cnt;
@@ -1326,7 +1326,7 @@ static int mtd_ooblayout_set_bytes(struct mtd_info *mtd, const u8 *buf,
        while (!ret) {
                int cnt;
 
-               cnt = oobregion.length > nbytes ? nbytes : oobregion.length;
+               cnt = min_t(int, nbytes, oobregion.length);
                memcpy(oobbuf + oobregion.offset, buf, cnt);
                buf += cnt;
                nbytes -= cnt;