Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74742 )
Change subject: util/cbmem: Add REG_NEWLINE flag to fix matching pattern
......................................................................
Patch Set 2:
(1 comment)
File util/cbmem/cbmem.c:
https://review.coreboot.org/c/coreboot/+/74742/comment/61c13aa0_eb671ac7
PS2, Line 1010: match.rm_so + 1;
Why do we move the cursor to match.rm_so + 1 instead of match.rm_eo? […]
Never mind. I just realized that will skip the banner for the matching coreboot stage when using cbmem -1.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/74742
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9e924349ead0fa7eea8b9ad5161138a4c4946ade
Gerrit-Change-Number: 74742
Gerrit-PatchSet: 2
Gerrit-Owner: Konrad Adamczyk
konrada@google.com
Gerrit-Reviewer: Grzegorz Bernacki
Gerrit-Reviewer: Jakub Czapiga
jacz@semihalf.com
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Karthik Ramasubramanian
kramasub@google.com
Gerrit-Reviewer: Raul Rangel
rrangel@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Comment-Date: Thu, 27 Apr 2023 04:14:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Karthik Ramasubramanian
kramasub@google.com
Gerrit-MessageType: comment