[coreboot-gerrit] [S] Change in coreboot[main]: cbmem.h: Change signature of cbmem_get_region

[Angel Pons (Code Review)]

Attention is currently required from: Arthur Heymans, Christian Walter, Jakub Czapiga, Johnny Lin, Jonathan Zhang, Jérémy Compostella, Lean Sheng Tan, Patrick Rudolph, Shuo Liu, Tim Chu.

Angel Pons has posted comments on this change by Arthur Heymans. ( https://review.coreboot.org/c/coreboot/+/84039?usp=email )

Change subject: cbmem.h: Change signature of cbmem_get_region
......................................................................

Patch Set 4:

(2 comments)

File src/lib/fit_payload.c:

https://review.coreboot.org/c/coreboot/+/84039/comment/f99d64c0_1bdcddc0?usp... :
PS4, Line 144: 		die("Could not find cbmem region");
Is it necessary to promote the warning to a fatal error?

File src/lib/imd_cbmem.c:

https://review.coreboot.org/c/coreboot/+/84039/comment/6fbb2f5b_a19ca0c1?usp... :
PS4, Line 199: 	if (cbmem_get_region(&baseptr, &size)) {
I wonder, why only use braces here? It feels inconsistent