[U-Boot] [PATCH] Provide more useful debugging when an initcall fails

Simon Glass sjg at chromium.org
Tue Apr 16 07:10:20 CEST 2013


Hi Wolfgang,

On Sun, Apr 14, 2013 at 1:34 AM, Wolfgang Denk <wd at denx.de> wrote:
> Dear Simon Glass,
>
> In message <1365863234-22906-1-git-send-email-sjg at chromium.org> you wrote:
>> The debug() which prints out the current call is not very useful, since if
>> it is called early enough (such as before the console is ready in the
>> pre-relocation board_init_f() sequence) it can hang the board.
>
> This should not happen (and if it does, that wouldbe a bug that needs
> fixing).

Yes.

>
>> It is more useful to print a message when a call fails, and in this case
>> the non-relocated symbol address provides a way for the offending function
>> to be located in System.map.
>
> No.  it is a long styanding rule that for this tyoe of actions the
> begin of the message gets printed before even attempting to call the
> respective function, so if the systemn hangs you can see the last
> operation which was about to be run.  I think we should follow this
> principle here, too.

OK fair enough, makes sense. I think. It is certainly true that any of
these calls could hang the board.

Let's drop this patch.

>
> 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: wd at denx.de
> "Love is an ideal thing, marriage a real thing; a  confusion  of  the
> real with the ideal never goes unpunished."                  - Goethe

Regards,
Simon


More information about the U-Boot mailing list