Dera Wolfgang Denk,

On 01.09.2012 14:02, Wolfgang Denk wrote:
Dear Andreas,

In message <20120901101031.2da15205...@gemini.denx.de> I wrote:

i. e. this has the offset 2 added twice, so it is missing the first
two real entries, env writing beyond the end of the array.

Please ignore me. Seems my brain is still in vacation mode.

However, I really wonder why you say this code would be causing
runtime errors for you:

        mbinptr *p = &av_[2];
        int i;

        for (i = 2; i < ARRAY_SIZE(av_); ++i)
                *p = (mbinptr)((ulong)*p + gd->reloc_off);

It looks good to me.

Well, I did not investigate it further, but I can do this weekend (after merging/testing/pull request the atmel arm stuff). Hopefully we get a solution for the annoying alias warning in upcoming release.

Best regards

Andreas Bießmann

Can you please add some debug printf()s to find out how the old code
and this one differ in their results?

Best regards,

Wolfgang Denk

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

Reply via email to