Change in coreboot[master]: commonlib/mem_pool: Allow configuring the alignment - coreboot-gerrit

15 Nov 2021


      Attention is currently required from: Raul Rangel.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56580 )
Change subject: commonlib/mem_pool: Allow configuring the alignment
......................................................................
Patch Set 13:
(1 comment)
File src/commonlib/mem_pool.c:
https://review.coreboot.org/c/coreboot/+/56580/comment/964a642e_13b9dd96 
PS13, Line 11: 		return NULL;
nit: not sure why this is still here when you assert that alignment can't be 0?
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/56580
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8d77ffe4411f86c54450305320c9f52ab41a3075
Gerrit-Change-Number: 56580
Gerrit-PatchSet: 13
Gerrit-Owner: Raul Rangel rrangel@chromium.org
Gerrit-Reviewer: Furquan Shaikh furquan.m.shaikh@gmail.com
Gerrit-Reviewer: Julius Werner jwerner@chromium.org
Gerrit-Reviewer: Karthik Ramasubramanian kramasub@google.com
Gerrit-Reviewer: Patrick Georgi patrick@coreboot.org
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-CC: Angel Pons th3fanbus@gmail.com
Gerrit-CC: Paul Menzel paulepanter@mailbox.org
Gerrit-Attention: Raul Rangel rrangel@chromium.org
Gerrit-Comment-Date: Mon, 15 Nov 2021 19:48:09 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment