[coreboot-gerrit] Change in ...coreboot[master]: prog_loader: Associate TS_END_ROMSTAGE timestamp with postcar if exist

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/31608 )

Change subject: prog_loader: Associate TS_END_ROMSTAGE timestamp with postcar if exist
......................................................................

prog_loader: Associate TS_END_ROMSTAGE timestamp with postcar if exist

This patch adds timestamp for "end of romstage" with postcar if platform
has selected postcar as dedicated stage.

If postcar stage doesn't exist then "end of romstage" timestamp will get
call while starting of ramstage as exist today.

TEST=It's been observed that "end of romstage" timestamp doesn't appear
in "cbmem -t" log when ramstage is not getting executed. As part of this fix
"end of romstage" timestamp is showing in "cbmem -t" log on Intel platform
where POSTCAR is a dedicated stage.

Change-Id: I17fd89296354b66a5538f85737c79145232593d3
Signed-off-by: Subrata Banik 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/31608
Tested-by: build bot (Jenkins) 
Reviewed-by: Aaron Durbin 
---
M src/arch/x86/postcar_loader.c
M src/lib/prog_loaders.c
2 files changed, 7 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Aaron Durbin: Looks good to me, approved

diff --git a/src/arch/x86/postcar_loader.c b/src/arch/x86/postcar_loader.c
index 141e8d2..a36b900 100644
--- a/src/arch/x86/postcar_loader.c
+++ b/src/arch/x86/postcar_loader.c
@@ -23,6 +23,7 @@
 #include <rmodule.h>
 #include <romstage_handoff.h>
 #include <stage_cache.h>
+#include <timestamp.h>

static inline void stack_push(struct postcar_frame *pcf, uint32_t val)
 {
@@ -171,5 +172,8 @@
    } else
    	load_postcar_cbfs(&prog, pcf);

+	/* As postcar exist, it's end of romstage here */
+	timestamp_add_now(TS_END_ROMSTAGE);
+
    prog_run(&prog);
 }
diff --git a/src/lib/prog_loaders.c b/src/lib/prog_loaders.c
index da021f1..7319811 100644
--- a/src/lib/prog_loaders.c
+++ b/src/lib/prog_loaders.c
@@ -131,7 +131,9 @@
    if (ENV_POSTCAR)
    	timestamp_add_now(TS_END_POSTCAR);

-	timestamp_add_now(TS_END_ROMSTAGE);
+	/* Call "end of romstage" here if postcar stage doesn't exist */
+	if (ENV_ROMSTAGE)
+		timestamp_add_now(TS_END_ROMSTAGE);

/*
     * Only x86 systems using ramstage stage cache currently take the same