Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/81387?usp=email )
Change subject: soc/intel/alderlake: Attach timestamp around eSOL call ......................................................................
soc/intel/alderlake: Attach timestamp around eSOL call
This patch adds timestamp start/end entries around the eSOL implementation to track the panel initialization time while rendering the eSOL screen.
TEST=Able to build and boot google/omnigul.
555: started early sign-off life (eSOL) notification 643,694 (40) 556: finished early sign-off life (eSOL) notification 1,072,143 (428,449)
Signed-off-by: Subrata Banik subratabanik@google.com Change-Id: I51c04fc4bd2540b3f42e2f896178521d297ef246 Reviewed-on: https://review.coreboot.org/c/coreboot/+/81387 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Eric Lai ericllai@google.com --- M src/soc/intel/alderlake/romstage/ux.c 1 file changed, 7 insertions(+), 1 deletion(-)
Approvals: Eric Lai: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/src/soc/intel/alderlake/romstage/ux.c b/src/soc/intel/alderlake/romstage/ux.c index af8c783..77439ab 100644 --- a/src/soc/intel/alderlake/romstage/ux.c +++ b/src/soc/intel/alderlake/romstage/ux.c @@ -3,6 +3,7 @@ #include <console/console.h> #include <intelblocks/early_graphics.h> #include <pc80/vga.h> +#include <timestamp.h> #include <ux_locales.h>
#include "ux.h" @@ -11,9 +12,13 @@
bool ux_inform_user_of_update_operation(const char *name) { + timestamp_add_now(TS_ESOL_START); + if (!CONFIG(MAINBOARD_HAS_EARLY_LIBGFXINIT) || - !early_graphics_init()) + !early_graphics_init()) { + timestamp_add_now(TS_ESOL_END); return false; + }
printk(BIOS_INFO, "Informing user on-display of %s.\n", name);
@@ -26,5 +31,6 @@ vga_write_text(VGA_TEXT_CENTER, VGA_TEXT_HORIZONTAL_MIDDLE, (const unsigned char *)text); ux_locales_unmap(); + timestamp_add_now(TS_ESOL_END); return true; }