Dear Otavio Salvador, In message <1402879613-21362-5-git-send-email-ota...@ossystems.com.br> you wrote: > When debugging initramfs failures it is quite useful to known where it > is being loaded from.
Agreed. > + > + printf(" ramdisk load start = 0x%08lx, ramdisk load end = > 0x%08lx\n", > + *initrd_start, *initrd_end); > } else { > *initrd_start = 0; > *initrd_end = 0; > } > - debug(" ramdisk load start = 0x%08lx, ramdisk load end = 0x%08lx\n", > - *initrd_start, *initrd_end); As you write yourself, this is a debug feature. There is no need to pollute normal output with such information. It just makes booting slower. So debug() is appropriate here. And I don't see why you would NOT want to have the information in the else case. I don't think this change is a good idea. It should not be applied. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de The past is not dead. In fact, it's not even past. - William Faulkner _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot