[S] Change in coreboot[master]: ec/google/ec_cmd: Add EC command for pre-reset boot timestamps
Attention is currently required from: Bora Guvendik, Hannah Williams, Caveh Jalali, Paul Menzel, Boris Mittelberg.
Jay Patel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/72930 )
Change subject: ec/google/ec_cmd: Add EC command for pre-reset boot timestamps ......................................................................
Patch Set 9:
(6 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/72930/comment/e77d4e90_a1a39d4f PS4, Line 9: defines the command
no, this patch actually reverts a struct name to an […]
inconsistent name of the struct fixed.
Commit Message:
https://review.coreboot.org/c/coreboot/+/72930/comment/f4296211_3bfb5a04 PS7, Line 2: jaypatel256
Please configure your local git: […]
Done
https://review.coreboot.org/c/coreboot/+/72930/comment/bd5d80aa_89e75ddc PS7, Line 7: ec/google/ec_cmd: EC command for pre-reset boot timestamps
Please make it a statement by adding a verb (in imperative mood): Add EC command …
Done
https://review.coreboot.org/c/coreboot/+/72930/comment/7a5753eb_690be57e PS7, Line 11:
Please add a reasoning/motivation.
Done
https://review.coreboot.org/c/coreboot/+/72930/comment/d239bbff_7bc080ed PS7, Line 15: jaypatel256
Ditto.
Done
Patchset:
PS4:
- we just fixed the struct naming with […]
Ack
-
Jay Patel (Code Review)