Am 16.06.2015 um 15:33 schrieb KOBAYASHI Yoshitake:
>>> + /* correctable */
>>> + else if (status & NAND_STATUS_RECOM_REWRT) {
>>> + pr_info("BENAND : Recommended to rewrite!\n");
>>> + bitflips = chip->ecc.strength;
>>
>> In your case this might be okay, as you set strength to 1.
>> Otherweise you'd have to report the real number of bitflips.
>
> I also thought it is okay in this case.
> BENAND return corrected data to Host NAND Controller till uncorrectable
> status.
> The current patch uses this Read Status command 70h to abstract BENAND Multi
> bit ECC and Need to Rewrite judgement so BENAND would look like 1bit ECC
> device.
The layers above MTD need to know how many bits got repaired.
It seems like BENAND suffers from the same shortcomings than On-Die-ECC. ;-\
Please see my patches how to deal with that. Maybe you can find a better
solution.
Thanks,
//richard
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
