[PATCH 7/7] mtd: driver _read() returns max_bitflips; mtd_read() returns -EUCLEAN
Mike Dunn
mikedunn at newsguy.com
Mon Apr 30 15:55:18 EDT 2012
On 04/29/2012 12:24 PM, Artem Bityutskiy wrote:
> On Wed, 2012-04-25 at 12:13 -0700, Mike Dunn wrote:
>> Thanks for the review Robert.
>>
>> On 04/25/2012 11:27 AM, Robert Jarzmik wrote:
>>>
>>> I think you should change :
>>>> - while (!ret && (len > 0 || ooblen > 0)) {" will exit because ret is not
>>> into
>>>> - while (ret >= 0 && (len > 0 || ooblen > 0)) {".
>>>
>>> With that change, please add my:
>>> Acked-by: Robert Jarzmik <robert.jarzmik at free.fr>
>>
>>
>> In my exuberance, I prematurely sent the next version of the whole set. Artem,
>> Robert's requested change is in patch 7/7. If no other problems come to light,
>> maybe you could consider merging the first 6? Then I only need to prepare a
>> corrected version of patch 7.
>
> Pushed the series to l2-mtd.git, thanks! You can send the newer version
> and I'll put it instead of the older one. Do not forget to add
> Acked-by's when you re-send, please.
>
Thanks Artem. Since you merged all 7, the only thing remaining is the fix to
docg3 that Robert describes above. Robert, if you would rather prepare that
patch, I defer to you. Otherwise, I'll submit it no later than tomorrow.
Yeah, sorry Artem, I neglected to add the Acked-by's in the patches you just merged.
Mike
More information about the linux-mtd
mailing list