public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] v1.3.4-rc1 released
Date: Thu, 17 Jul 2008 08:41:03 +0200	[thread overview]
Message-ID: <487EE97F.8010603@denx.de> (raw)

Hello Wolfgang,

Wolfgang Denk wrote:
> Dear Heiko,
>
> in message <487E4C16.5050603@invitel.hu> you wrote:
>> diff --git a/include/configs/sc3.h b/include/configs/sc3.h
>> index f6e40de..0ff889a 100644
>> --- a/include/configs/sc3.h
>> +++ b/include/configs/sc3.h
>> @@ -132,7 +132,8 @@
>>
>>  #if 1	/* feel free to disable for development */
>>  #define CONFIG_AUTOBOOT_KEYED		/* Enable password protection	*/
>> -#define CONFIG_AUTOBOOT_PROMPT		"\nSC3 - booting... stop with ENTER\n"
>> +#define CONFIG_AUTOBOOT_PROMPT		"\nSC3 - booting in %d seconds ..." \
>> +					" stop with ENTER\n"
>>  #define CONFIG_AUTOBOOT_DELAY_STR	"\r"	/* 1st "password"	*/
>>  #define CONFIG_AUTOBOOT_DELAY_STR2	"\n"	/* 1st "password"	*/
>>  #endif

Didnt know this ... so we must change something in common/main.c :-(

Whats with something like this?

diff --git a/common/main.c b/common/main.c
index 79ad291..a8d05f4 100644
--- a/common/main.c
+++ b/common/main.c
@@ -87,6 +87,13 @@ int do_mdm_init = 0;
 extern void mdm_init(void); /* defined in board.c */
 #endif

+void __autoboot_prompt(int bootdelay)
+{
+	return;
+}
+void inline autoboot_prompt(int bootdelay)
+                __attribute__((weak, alias("__autoboot_prompt")));
+
 /***************************************************************************
  * Watch for 'delay' seconds for autoboot stop or autoboot delay string.
  * returns: 0 -  no key string, allow autoboot
@@ -115,9 +122,7 @@ static __inline__ int abortboot(int bootdelay)
 	u_int presskey_max = 0;
 	u_int i;

-#  ifdef CONFIG_AUTOBOOT_PROMPT
-	printf(CONFIG_AUTOBOOT_PROMPT, bootdelay);
-#  endif
+	autoboot_prompt(bootdelay);

 #  ifdef CONFIG_AUTOBOOT_DELAY_STR
 	if (delaykey[0].str == NULL)


and of course, we must also fix all boards which use CONFIG_AUTOBOOT_PROMPT :-(

bye
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

             reply	other threads:[~2008-07-17  6:41 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-17  6:41 Heiko Schocher [this message]
2008-07-17  8:24 ` [U-Boot-Users] v1.3.4-rc1 released Wolfgang Denk
2008-07-17  8:59   ` Heiko Schocher
2008-07-17 10:21   ` Jerry Van Baren
2008-07-17 11:48     ` Jerry Van Baren
2008-07-17 12:53       ` Heiko Schocher
     [not found] <mailman.767415.1216216547.5026.u-boot-users@lists.sourceforge.net>
2008-07-16 19:29 ` Heiko Schocher
2008-07-16 21:14   ` Wolfgang Denk
  -- strict thread matches above, loose matches on Subject: below --
2008-07-16 11:18 Wolfgang Denk
2008-07-16 13:32 ` Jerry Van Baren
2008-07-23 19:12 ` Scott Wood
2008-07-29 14:37   ` Wolfgang Denk
2008-07-29 16:48     ` Scott Wood
2008-07-31  8:31       ` Wolfgang Denk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=487EE97F.8010603@denx.de \
    --to=hs@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox