[coreboot-gerrit] [S] Change in coreboot[main]: ec/google/chromeec: Add debug timestamp for host EC commands

[Julius Werner (Code Review)]

Attention is currently required from: Caveh Jalali, Forest Mittelberg, Jayvik Desai, Kapil Porwal, Subrata Banik.

Julius Werner has posted comments on this change by Jayvik Desai. ( https://review.coreboot.org/c/coreboot/+/85326?usp=email )

Change subject: ec/google/chromeec: Add debug timestamp for host EC commands
......................................................................

Patch Set 9:

(1 comment)

File src/ec/google/chromeec/ec_lpc.c:

https://review.coreboot.org/c/coreboot/+/85326/comment/d7ba54e3_41a8f99b?usp... :
PS9, Line 286: printk
> > Please call stopwatch_tick() before accessing sw.current. […]

The absolute timestamp isn't really meant to be accessed at all. Is there a reason to print it here (rather than just the duration)? coreboot timers aren't guaranteed to start at 0 at boot, so absolute timer values are basically meaningless anyway.