[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <75ef7936bd1da87f54cac867103a83d6@codeaurora.org>
Date: Tue, 22 May 2018 19:45:57 +0530
From: Abhishek Sahu <absahu@...eaurora.org>
To: Miquel Raynal <miquel.raynal@...tlin.com>
Cc: Boris Brezillon <boris.brezillon@...tlin.com>,
David Woodhouse <dwmw2@...radead.org>,
Brian Norris <computersforpeace@...il.com>,
Marek Vasut <marek.vasut@...il.com>,
Richard Weinberger <richard@....at>,
linux-arm-msm@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-mtd@...ts.infradead.org, Andy Gross <andy.gross@...aro.org>,
Archit Taneja <architt@...eaurora.org>
Subject: Re: [PATCH v2 10/14] mtd: rawnand: qcom: fix return value for raw
page read
On 2018-05-22 17:34, Miquel Raynal wrote:
> Hi Abhishek,
>
> On Thu, 3 May 2018 17:50:37 +0530, Abhishek Sahu
> <absahu@...eaurora.org> wrote:
>
>> Currently zero is being returned for all raw page read so
>> fix the same.
>
> What about "Fix value returned by ->read_page_raw() to be the
> actual operation status, instead of always 0."?
>
Sure Miquel. It looks better.
I will change this.
Thanks,
Abhishek
>>
>> Signed-off-by: Abhishek Sahu <absahu@...eaurora.org>
>> ---
>> * Changes from v1:
>>
>> NEW CHANGE
>>
>> drivers/mtd/nand/raw/qcom_nandc.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/mtd/nand/raw/qcom_nandc.c
>> b/drivers/mtd/nand/raw/qcom_nandc.c
>> index f85d8ab..17b7f3af 100644
>> --- a/drivers/mtd/nand/raw/qcom_nandc.c
>> +++ b/drivers/mtd/nand/raw/qcom_nandc.c
>> @@ -1900,7 +1900,7 @@ static int qcom_nandc_read_page_raw(struct
>> mtd_info *mtd,
>>
>> free_descs(nandc);
>>
>> - return 0;
>> + return ret;
>> }
>>
>> /* implements ecc->read_oob() */
>
> Thanks,
> Miquèl
Powered by blists - more mailing lists