Re: [U-Boot] [PATCH 5/6] Print ramdisk load address if it is in use

Dear Otavio Salvador,

In message 1402879613-21362-5-git-send-email-otavio@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