[coreboot-gerrit] [M] Change in coreboot[main]: region: Hide struct region members

[Nico Huber (Code Review)]

Attention is currently required from: Jakub Czapiga, Julius Werner, Maximilian Brune, Philipp Hug, ron minnich.

Nico Huber has posted comments on this change by Nico Huber. ( https://review.coreboot.org/c/coreboot/+/79907?usp=email )

Change subject: region: Hide struct region members
......................................................................

Patch Set 10:

(1 comment)

File src/arch/arm/fit_payload.c:

https://review.coreboot.org/c/coreboot/+/79907/comment/d60ea2e1_7d2fcd21?usp... :
PS8, Line 3: /* FIXME: should use the high-level region api */
> The main point of FIT is to be able to bundle a flattened device tree (FDT) together with a kernel i […]

It's um, a bit ugly: CB:84087. We don't have an API to manipulate regions
and the FIT code uses the struct to first only pass some offsets and sizes
around, patching them until it's a region. Not sure if we should extend the
API or live with it. For now, I've done everything with region_create().

Also not sure if the data is considered truted. In case of vboot, does the
CBFS API take care of verification?