[PATCH 12/13] mtd/docg3: add ECC correction code

Mike Dunn mikedunn at newsguy.com
Mon Oct 31 13:32:28 EDT 2011


On 10/31/2011 09:39 AM, Robert Jarzmik wrote:
> Mike Dunn <mikedunn at newsguy.com> writes:
>
>> Another explanatory comment here...
>>     /* undo last step in BCH alg (modulo mirroring not needed) */
> Is that the same as the function comment about bit reversing (the modulo
> mirroring part) ? If so, the function comment might not be clear enough. If not,
> could you explain a bit further please ?
>
>


No, the "modulo mirroring" is not the same as the bit reversal.  See Ivan's
explanation here:

http://lists.infradead.org/pipermail/linux-mtd/2011-October/038060.html

Obviously the comment doesn't give much technical detail, but it points anyone
puzzled by the step in the right direction.  I don't like cryptic steps that
give no explanation whatsoever.  Just MHO.

Thanks,
Mike




More information about the linux-mtd mailing list