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

[Julius Werner (Code Review)]

Julius Werner 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) {
> Let’s wait for Julius’ answer first.

Yes, the goal was just to make this function idempotent as a precaution in case it's ever called twice somewhere. Please just submit it like this (following existing practice) for now... I have changes in the pipe that will unify all of this in generic code anyway, they're just taking a while to go through on the Trusted Firmware side.