[coreboot-gerrit] Change in ...coreboot[master]: [WIP] soc/amd/common: Use static allocation for params

[Richard Spiegel (Code Review)]

Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31489 )

Change subject: [WIP] soc/amd/common: Use static allocation for params
......................................................................

Patch Set 5: Code-Review-1

(1 comment)

y

https://review.coreboot.org/#/c/31489/5/src/soc/amd/common/block/pi/agesawra... 
File src/soc/amd/common/block/pi/agesawrapper.c:

https://review.coreboot.org/#/c/31489/5/src/soc/amd/common/block/pi/agesawra... 
PS5, Line 425: 
             : 
             : 
             : 
             : 
             : 
             : 
             :
> Do you have a platform to test? I'm not comfortable approving it until it's tested. […]

I finally had a chance to test it, and as I suspected it would, it hanged at amd_init_early. That section of code is needed and I'll block any change that remove it.