Hi Stefan:

Agreed the ordering is working now. Previously the ordering is 213 and with
CONFIG_MTD_NAND_ECC_SMC defined, wrong ECC error bit position was calculated.
What about the boards that are now stuck on the 213 ordering. Also in linux,
the ordering is not fixed down as in u-boot. Hmm, perhaps that is another
approach to the problem. Fix the ordering in linux?

Feng

On 02/18/2010 11:57 PM, Stefan Roese wrote:
> Hi Feng,
>
> On Thursday 18 February 2010 23:25:13 f...@amcc.com wrote:
>> From: Feng Kan<f...@amcc.com>
>>
>> This is to lock down the ordering in the correction routine against
>> the calculate routine. Otherwise, incorrect define would cause ECC errors.
>
> It was my impression that we (finally) had done this ordering correct. The
> last changes were upon your request:
>
> 68e74567cf317318df52dbcb2ac170ffc5e7758a:
>      ppc4xx: Fix ECC Correction bug with SMC ordering for NDFC driver
>
> I don't see how this patch should fix a potential problem. Please explain
> which problem exactly is fixed with this change. As Wolfgang already
> mentioned, CONFIG_MTD_NAND_ECC_SMC will not be set in this file.
>
> Cheers,
> Stefan
>
> --
> DENX Software Engineering GmbH,      MD: Wolfgang Denk&  Detlev Zundel
> HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: off...@denx.de

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to