diff mbox

[U-Boot,1/3] bootstage: Dont print reset entry separately

Message ID 0f47cf62b1af3dac5d2a1b9da65a4bf5364b6765.1496146930.git.michal.simek@xilinx.com
State Deferred
Delegated to: Tom Rini
Headers show

Commit Message

Michal Simek May 30, 2017, 12:22 p.m. UTC 
From: Siva Durga Prasad Paladugu <siva.durga.paladugu@xilinx.com>

Printing the first entry reset separately is no longer
needed as it now prints the entries with valid name and
timestamp zero. This removes duplicate printing of the reset
record.

Signed-off-by: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 common/bootstage.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Tom Rini June 9, 2017, 1 a.m. UTC | #1 
On Tue, May 30, 2017 at 02:22:11PM +0200, Michal Simek wrote:

> From: Siva Durga Prasad Paladugu <siva.durga.paladugu@xilinx.com>
> 
> Printing the first entry reset separately is no longer
> needed as it now prints the entries with valid name and
> timestamp zero. This removes duplicate printing of the reset
> record.
> 
> Signed-off-by: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>

Can you please rebase this and the rest of the series on top of master
again?  And please cc Simon when you repost, thanks!
Michal Simek June 21, 2017, 10:51 a.m. UTC | #2 
On 9.6.2017 03:00, Tom Rini wrote:
> On Tue, May 30, 2017 at 02:22:11PM +0200, Michal Simek wrote:
> 
>> From: Siva Durga Prasad Paladugu <siva.durga.paladugu@xilinx.com>
>>
>> Printing the first entry reset separately is no longer
>> needed as it now prints the entries with valid name and
>> timestamp zero. This removes duplicate printing of the reset
>> record.
>>
>> Signed-off-by: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
>> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> 
> Can you please rebase this and the rest of the series on top of master
> again?  And please cc Simon when you repost, thanks!
> 

There are new patches from Simon. Siva will look at it and try to
validate them on our platforms.

Thanks,
Michal
diff mbox

Patch

diff --git a/common/bootstage.c b/common/bootstage.c
index 35bce3d881a5..c8080dcab5d7 100644
--- a/common/bootstage.c
+++ b/common/bootstage.c
@@ -271,7 +271,7 @@  void bootstage_report(void)
 	/* Fake the first record - we could get it from early boot */
 	rec->name = "reset";
 	rec->time_us = 0;
-	prev = print_time_record(BOOTSTAGE_ID_AWAKE, rec, 0);
+	prev = 0;
 
 	/* Sort records by increasing time */
 	qsort(record, ARRAY_SIZE(record), sizeof(*rec), h_compare_record);