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

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31608 )

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

Patch Set 2:

(1 comment)

https://review.coreboot.org/#/c/31608/1/src/arch/x86/postcar_loader.c 
File src/arch/x86/postcar_loader.c:

https://review.coreboot.org/#/c/31608/1/src/arch/x86/postcar_loader.c@164 
PS1, Line 164: 	timestamp_add_now(TS_END_ROMSTAGE);
> Well patchset #2 is removing load_postcar_cbfs() from
> TS_END_ROMSTAGE.

[Subrata] I didn't get you. i don;t see any change in this file between patchset #1 and #2

https://review.coreboot.org/#/c/31608/1..2/src/arch/x86/postcar_loader.c
> I'd rather keep it in, you can provide a followup
> that adds individual stamps for postcar loader like the ones you
> refer to used in run_ramstage().

[Subrata] We can do that as well, i will push separate CL for that.