From: Chuanhong Guo Date: Sat, 21 Jan 2023 02:47:59 +0000 (+0800) Subject: kernel: mtk-bmt: fix usage of _oob_read X-Git-Tag: v23.05.0-rc1~1196 X-Git-Url: http://git.openwrt.org/?a=commitdiff_plain;h=f183ce35b8ea2fd991ac489fb223b09a1ecb4db0;p=openwrt%2Fopenwrt.git kernel: mtk-bmt: fix usage of _oob_read _oob_read returns number of bitflips on success while bbt_nand_read should return 0. Fixes: 2d49e49b18 ("mediatek: bmt: use generic mtd api") Signed-off-by: Chuanhong Guo --- diff --git a/target/linux/generic/files/drivers/mtd/nand/mtk_bmt.h b/target/linux/generic/files/drivers/mtd/nand/mtk_bmt.h index dff1f28c81..517ff7414f 100644 --- a/target/linux/generic/files/drivers/mtd/nand/mtk_bmt.h +++ b/target/linux/generic/files/drivers/mtd/nand/mtk_bmt.h @@ -95,8 +95,14 @@ bbt_nand_read(u32 page, unsigned char *dat, int dat_len, .datbuf = dat, .len = dat_len, }; - - return bmtd._read_oob(bmtd.mtd, page << bmtd.pg_shift, &ops); + int ret; + + ret = bmtd._read_oob(bmtd.mtd, page << bmtd.pg_shift, &ops); + if (ret < 0) + return ret; + if (ret) + pr_info("%s: %d bitflips\n", __func__, ret); + return 0; } static inline int bbt_nand_erase(u16 block)