Patchwork [U-Boot,01/17] arm: Add new bootstage step for the main loop

login
register
mail settings
Submitter Simon Glass
Date Nov. 3, 2012, 12:27 a.m.
Message ID <1351902453-27956-2-git-send-email-sjg@chromium.org>
Download mbox | patch
Permalink /patch/196781/
State Superseded, archived
Delegated to: Tom Rini
Headers show

Comments

Simon Glass - Nov. 3, 2012, 12:27 a.m.
Mark when we get to the main loop.

Signed-off-by: Simon Glass <sjg@chromium.org>
---
 common/main.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
Wolfgang Denk - Nov. 3, 2012, 3:12 p.m.
Dear Simon Glass,

In message <1351902453-27956-2-git-send-email-sjg@chromium.org> you wrote:
> Mark when we get to the main loop.
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>  common/main.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/common/main.c b/common/main.c
> index 9507cec..ed1da24 100644
> --- a/common/main.c
> +++ b/common/main.c
> @@ -299,6 +299,8 @@ void main_loop (void)
>  	char bcs_set[16];
>  #endif /* CONFIG_BOOTCOUNT_LIMIT */
>  
> +	bootstage_mark_name(BOOTSTAGE_ID_MAIN_LOOP, "main_loop");
> +
>  #ifdef CONFIG_BOOTCOUNT_LIMIT
>  	bootcount = bootcount_load();
>  	bootcount++;

In which way is this patch ARM specific?


Best regards,

Wolfgang Denk
Simon Glass - Nov. 7, 2012, 12:54 a.m.
Hi Wolfgang,

On Sat, Nov 3, 2012 at 8:12 AM, Wolfgang Denk <wd@denx.de> wrote:
> Dear Simon Glass,
>
> In message <1351902453-27956-2-git-send-email-sjg@chromium.org> you wrote:
>> Mark when we get to the main loop.
>>
>> Signed-off-by: Simon Glass <sjg@chromium.org>
>> ---
>>  common/main.c |    2 ++
>>  1 files changed, 2 insertions(+), 0 deletions(-)
>>
>> diff --git a/common/main.c b/common/main.c
>> index 9507cec..ed1da24 100644
>> --- a/common/main.c
>> +++ b/common/main.c
>> @@ -299,6 +299,8 @@ void main_loop (void)
>>       char bcs_set[16];
>>  #endif /* CONFIG_BOOTCOUNT_LIMIT */
>>
>> +     bootstage_mark_name(BOOTSTAGE_ID_MAIN_LOOP, "main_loop");
>> +
>>  #ifdef CONFIG_BOOTCOUNT_LIMIT
>>       bootcount = bootcount_load();
>>       bootcount++;
>
> In which way is this patch ARM specific?

It is not - I fixed it up and forgot to remove the tag. I will
resubmit once I sort out the rest of the series.

Regards,
Simon

>
>
> 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@denx.de
> Do not underestimate the value of print statements for debugging.

Patch

diff --git a/common/main.c b/common/main.c
index 9507cec..ed1da24 100644
--- a/common/main.c
+++ b/common/main.c
@@ -299,6 +299,8 @@  void main_loop (void)
 	char bcs_set[16];
 #endif /* CONFIG_BOOTCOUNT_LIMIT */
 
+	bootstage_mark_name(BOOTSTAGE_ID_MAIN_LOOP, "main_loop");
+
 #ifdef CONFIG_BOOTCOUNT_LIMIT
 	bootcount = bootcount_load();
 	bootcount++;