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

[Paul Menzel (Code Review)]

Paul Menzel 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) {
Why is this check useful? Can’t it be removed and the assignment always happen?