[coreboot-gerrit] Change in coreboot[master]: lib/prog_loaders, soc/amd/: Make payload_preload use cbfs_preload

[Julius Werner (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Rob Barnes, Felix Held.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58962 )

Change subject: lib/prog_loaders, soc/amd/: Make payload_preload use cbfs_preload
......................................................................

Patch Set 1:

(1 comment)

File src/lib/prog_loaders.c:

https://review.coreboot.org/c/coreboot/+/58962/comment/af3316fa_8ba641ce 
PS1, Line 133: 		return;
> So the initial thought was that the preload methods are all no-ops when not enabled. […]

Hmm... yeah... I think I'm just worried it can be confusing to see `payload_preload()` called unconditionally like that and having to explicitly be aware of that this actually doesn't do anything if the Kconfig is not enabled. I know we have that pattern elsewhere in coreboot too but usually it's less invasive stuff (like timestamps). I don't know.