Hi,
> Signed-off-by: Gerd Hoffmann <kra...@redhat.com>
No, I didn't signed this patch off.
And I will not because it is broken.
> for (cnt = (maxsize / sizeof(long)) >> 1; cnt > 0; cnt >>= 1) {
> addr = base + cnt; /* pointer arith! */
> @@ -44,6 +44,7 @@ long get_ram_size(long *base, long maxsize)
> addr = base;
> sync();
> save[i] = *addr;
base[0] might not be the original value any more at this point ...
> + addr = base;
> + sync();
> + *addr = save[last];
> return (maxsize);
... so this may corrupt memory.
cheers,
Gerd
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
