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

Reply via email to