On 2018-04-10 15:33, Miquel Raynal wrote:
Hi Abhishek,
On Wed, 4 Apr 2018 18:12:21 +0530, Abhishek Sahu
<absahu@xxxxxxxxxxxxxx> wrote:
read_page and read_oob both calls the read_page_ecc function.
The QCOM NAND controller protect the OOB available bytes with
ECC so read errors should be checked for read_oob also. Now
this patch moves the error checking code inside read_page_ecc
so caller does not have to check explicitly for read errors.
Signed-off-by: Abhishek Sahu <absahu@xxxxxxxxxxxxxx>
Nitpick: the prefix should be "mtd: rawnand: qcom: " now as this driver
has been moved to drivers/mtd/nand/raw/.
Otherwise:
Reviewed-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx>
Thanks Miquel for your review.
I will update the same in this patch and other patches.
and rebase my patches over 4.17-rc1 once its available.
Thanks,
Abhishek
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html