[coreboot-gerrit] Change in coreboot[master]: soc/mediatek: Register coreboot table for ATF

[Hung-Te Lin (Code Review)]

Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32541 )

Change subject: soc/mediatek: Register coreboot table for ATF
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/32541/4/src/soc/mediatek/common/bl3... 
File src/soc/mediatek/common/bl31_plat_params.c:

https://review.coreboot.org/c/coreboot/+/32541/4/src/soc/mediatek/common/bl3... 
PS4, Line 33: 	if (!cbtable_param.value) {
> This is following rk3399 implementation. […]

To make this easier to read, it's also possible to introduce one flag:

static int cbtable_injected = 0;

if (ctable_injected)
    return plat_params;

cbtable_param.value = (uint64_t)cbmem_find(CBMEM_ID_CBTABLE);
 if (cbtable_param.value) {
    register_bl31_param(&cbtable_param.h);
    cbtable_injected = 1;
 }

return plat_params;

Do you prefer this version?