Attention is currently required from: Mattias Nissler.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62639 )
Change subject: util/cbmem: add an option to append timestamp
......................................................................
Patch Set 2:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/62639/comment/5ec51b9a_66d686fe
PS2, Line 16: invoke new cbmem option
Please give an example.
File util/cbmem/cbmem.c:
https://review.coreboot.org/c/coreboot/+/62639/comment/c95bfea5_a3618670
PS2, Line 88: char *v = mapping->virt;
Excuse my ignorance, why is the const removed?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/62639
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic99e5a11d8cc3f9fffae8eaf2787652105cf4842
Gerrit-Change-Number: 62639
Gerrit-PatchSet: 2
Gerrit-Owner: Mattias Nissler
mnissler@chromium.org
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Mattias Nissler
mnissler@chromium.org
Gerrit-Comment-Date: Fri, 18 Mar 2022 05:21:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment