Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45303 )
Change subject: drivers/elog: Enable ELOG_PRERAM for all boards ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45303/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/45303/1//COMMIT_MSG@7 PS1, Line 7: Enable ELOG_PRERAM for all boards
I think it would be better to just drop CONFIG_ELOG_PRERAM completely and use CONFIG_ELOG instead to […]
At the time when I added ELOG_PRERAM, I wasn't sure if all the boards wanted to enable it by default. Hence added a separate config.
If we want to enable it for all boards, then +1 to what Furquan mentioned. Besides that, I don't see any issues with enabling ELOG_PRERAM for all boards.