Attention is currently required from: Martin L Roth, Julius Werner.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69893 )
Change subject: commonlib/bsd: Add zstd support
......................................................................
Patch Set 1:
(1 comment)
File src/commonlib/bsd/zstd_wrapper.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-164274):
https://review.coreboot.org/c/coreboot/+/69893/comment/b64040a4_bce8730f
PS1, Line 27: printk(BIOS_ERR, "Bogus free called\n");
Prefer using '"%s...", __func__' to using 'free', this function's name, in a string
--
To view, visit
https://review.coreboot.org/c/coreboot/+/69893
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic1b1f53327c598d07bd83d4391e8012d41696a16
Gerrit-Change-Number: 69893
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Martin L Roth
gaumless@gmail.com
Gerrit-CC: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Martin L Roth
gaumless@gmail.com
Gerrit-Attention: Julius Werner
jwerner@chromium.org
Gerrit-Comment-Date: Tue, 22 Nov 2022 15:52:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment