[PATCH v4 1/5] mtd: nand: Detect Micron flash with on-die ECC (aka "internal ECC") enabled.

[David Mosberger]

[Resend in plain text, sorry...]

On Thu, Apr 3, 2014 at 9:24 AM, David Mosberger <davidm at egauge.net> wrote:
>
> Gerhard,
>
> On Thu, Apr 3, 2014 at 1:10 AM, Gerhard Sittig <gsi at denx.de> wrote:
>>
>> I really don't see that enabling on-die-ECC in one stage requires
>> all other stages to follow.  It's one apparent approach, but need
>> not be the only one.  That's what previous messages tried to say.
>
>
>>
>> Does this explanation help answer your question?
>
>
> Yes, that helps, thanks for taking the time to answer!
>
> Leaving on-die ECC on is certainly not the only way to do it,
> but I still think it's a reasonable way.
>
> I doubt it's reasonable to modify each and every platform-driver
> to be able to decide whether to use on-die ECC or the best
> ECC mode it can supply.
>
> Maybe it's possible to use the new(ish) ecc_strength_ds/ecc_step_ds
> to chose whether or not to keep on-die ECC on?  Say, if the driver's
> ECC is stronger, turn on-die ECC off, otherwise leave it on?
>
> Again, I think the present patch is safe and reasonable, but I'm certainly
> open to better suggestions.
>
>   --david
> --
> eGauge Systems LLC, http://egauge.net/, 1.877-EGAUGE1, fax 720.545.9768




-- 
eGauge Systems LLC, http://egauge.net/, 1.877-EGAUGE1, fax 720.545.9768