[coreboot-gerrit] Change in coreboot[master]: lib/prog_loaders.c: Add prog_locate_hook()

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30811 )

Change subject: lib/prog_loaders.c: Add prog_locate_hook()
......................................................................

Patch Set 10:

(1 comment)

https://review.coreboot.org/c/coreboot/+/30811/10/src/include/program_loadin... 
File src/include/program_loading.h:

https://review.coreboot.org/c/coreboot/+/30811/10/src/include/program_loadin... 
PS10, Line 142: if identified program might run.
That's not true in absolute terms. This is a generic hook, and it could be used for such purposes. Please update the comment to describe the generic functionality and what it could be used for. e.g. when the function is called and what is valid. Current implementation is very early in prog_locate() w/o having many fields filled in yet as cbfs location hasn't been done.