[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:

(1 comment)

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: 
             : 
             : 
             : 
             : 
             : 
             : 
             :
> The point of this entire commit is that the lifetime of the parameter struct is the duration of the  […]

Do you have a platform to test? I'm not comfortable approving it until it's tested. If you don't have a platform, I can do it, but it'll have to wait an opening on my schedule (2 or 3 days).