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

[Julius Werner (Code Review)]

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/07e2a106_46131c26 
PS13, Line 11: 		return NULL;
> Only mem_pool_init asserts. Using MEM_POOL_INIT it's still possible (and desirable) to set a 0.

Only when size is 0 too though, right? So then it would drop out in line 18 anyway. (In fact, would probably not hurt to have a _Static_assert(!size == !alignment) in MEM_POOL_INIT().)