vendorcode/eltan/security: Cleanup prog_locate_hook

Cleanup of the prog_locate_hook routine so the actual coreboot flow is
more clearly reflected in the code.
Remove logging that is not really needed.

BUG=N/A
TEST=tested on fbg1701

Change-Id: Iab6c75beac35d043d296336021c0bce1f828cf34
Signed-off-by: Wim Vervoorn <wvervoorn@eltan.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/36846
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Wim Vervoorn 2019-11-14 10:03:25 +01:00 committed by Patrick Georgi
parent e05dc17d4b
commit f4a304722a
1 changed files with 12 additions and 15 deletions

View File

@ -330,28 +330,25 @@ int verified_boot_should_run_oprom(struct rom_header *rom_header)
int prog_locate_hook(struct prog *prog)
{
if (ENV_BOOTBLOCK) {
printk(BIOS_SPEW, "%s: bootblock\n", __func__);
if (ENV_BOOTBLOCK)
verified_boot_bootblock_check();
}
if (ENV_ROMSTAGE) {
static int prepare_romstage = 0;
printk(BIOS_SPEW, "%s: romstage\n", __func__);
if (!prepare_romstage) {
if (prog->type == PROG_REFCODE)
verified_boot_early_check();
prepare_romstage = 1;
}
if (CONFIG(POSTCAR_STAGE) && prog->type == PROG_POSTCAR)
process_verify_list(postcar_verify_list);
if (!CONFIG(POSTCAR_STAGE) && prog->type == PROG_RAMSTAGE)
process_verify_list(ramstage_verify_list);
}
if (ENV_POSTCAR) {
printk(BIOS_SPEW, "%s: postcar\n", __func__);
process_verify_list(postcar_verify_list);
}
if (ENV_POSTCAR && prog->type == PROG_RAMSTAGE)
process_verify_list(ramstage_verify_list);
if (ENV_RAMSTAGE) {
printk(BIOS_SPEW, "%s: ramstage\n", __func__);
if (ENV_RAMSTAGE && prog->type == PROG_PAYLOAD)
process_verify_list(payload_verify_list);
}
return 0;
}