Attention is currently required from: Hsuan-ting Chen, Julius Werner. Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63115 )
Change subject: libpayload: Add CONFIG_LP_ELOG and its dependency ......................................................................
Patch Set 9:
(1 comment)
File payloads/libpayload/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/63115/comment/ac2f90ad_25de9b7c PS9, Line 51: classes-$(CONFIG_LP_ELOG) += elog
It seems overkill to create a Kconfig and a whole extra library for such a small file. […]
Rocco did that in PS3. However, in my opinion the elog utility doesn't seem to belong to libc (and any existing libs). If there are more stuff like elog in the future, do we want to add them all to libc? I wonder if we can create a separate lib `libutil` for all these small utilities. We can keep the `elog` class name for now, but I'll rename it when there are more utilities coming.
-
Yu-Ping Wu (Code Review)