build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30083 )
Change subject: [RFC] commonlib/region: Check `xlate` regions early ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/30083/1/src/commonlib/include/commonlib/regi... File src/commonlib/include/commonlib/region.h:
https://review.coreboot.org/#/c/30083/1/src/commonlib/include/commonlib/regi... PS1, Line 88: int (*region_ok)(const struct region_device *, const struct region *); function definition argument 'const struct region_device *' should also have an identifier name
https://review.coreboot.org/#/c/30083/1/src/commonlib/include/commonlib/regi... PS1, Line 88: int (*region_ok)(const struct region_device *, const struct region *); function definition argument 'const struct region *' should also have an identifier name