Hi all,

I've been trying to stop my initrd being relocated duing a bootm
process, and ended up at this bit of code in common/image.c (starting
line 1135):


        if ((s = getenv("initrd_high")) != NULL) {
                /* a value of "no" or a similar string will act like 0,
                 * turning the "load high" feature off. This is intentional.
                 */
                initrd_high = simple_strtoul(s, NULL, 16);
                if (initrd_high == ~0)
                        initrd_copy_to_ram = 0;
        } else {
                initrd_high = getenv_bootm_mapsize() + getenv_bootm_low();
        }

The check - "if (initrd_high == ~0)" seems wrong - based on the comment
an value of "no" for initrd_high gets interpreted as 0.  But the test
is effectively for 0xffffffff (or 0xffffffffffffffff).   Assuming the
comment is correct, shouldn't the check be against 0 rather than ~0 ?

Thanks


-- 
Peter Howard <p...@northern-ridge.com.au>

Attachment: signature.asc
Description: This is a digitally signed message part

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

Reply via email to