[U-Boot-Users] v1.3.4-rc1 released
Heiko Schocher
hs at denx.de
Thu Jul 17 14:53:33 CEST 2008
Hello Jerry,
Jerry Van Baren wrote:
> Jerry Van Baren wrote:
>> Wolfgang Denk wrote:
>>> In message <487EE97F.8010603 at denx.de> you wrote:
>>>> Didnt know this ... so we must change something in common/main.c :-(
>>> Yes.
>>>
>>>> Whats with something like this?
>>> Looks way too complicated to me. What do you think about the PATCH /
>>> RFC I posted yesterday?
>>>
>>> Best regards,
>>>
>>> Wolfgang Denk
>>
>> I presume you considered and discarded using a special #define (e.g.
>> CONFIG_AUTOBOOT_PROMPT_0) for the no value case. It only hurts our
>> #ifdef hellcount a little and changes our "touch" count from <all> to
>> 5 (4 if I fix sacsng).
>>
>> diff --git a/common/main.c b/common/main.c
>> index 79ad291..09e6033 100644
>> --- a/common/main.c
>> +++ b/common/main.c
>> @@ -115,8 +115,10 @@ static __inline__ int abortboot(int bootdelay)
>> u_int presskey_max = 0;
>> u_int i;
>>
>> -# ifdef CONFIG_AUTOBOOT_PROMPT
>> +# if defined(CONFIG_AUTOBOOT_PROMPT)
>> printf(CONFIG_AUTOBOOT_PROMPT, bootdelay);
>> +# elif defined(CONFIG_AUTOBOOT_PROMPT_0)
>> + c
>
> Arrr, missed the <ctl> key and other sorts of evil
> + printf(CONFIG_AUTOBOOT_PROMPT_0);
>
Hmm.. and if somebody wants to print more variables we make more defines,
maybe for every board +1 define? I vote for making this in a weak function,
and each board code can print what he wants ...
bye
Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
More information about the U-Boot
mailing list