diff options
| author | Zheng Bao <fishbaozi@gmail.com> | 2021-08-20 14:47:46 +0800 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2021-08-31 15:06:03 +0000 |
| commit | 481661e313ddc2efb151cbdb77eeb5237eba5609 (patch) | |
| tree | 50b8c7a6a76c60bf6653f5a74c46fd3a16902ebe /util | |
| parent | dac446165eab07216428fc660b3fd28603803f0f (diff) | |
| download | coreboot-481661e313ddc2efb151cbdb77eeb5237eba5609.tar.gz coreboot-481661e313ddc2efb151cbdb77eeb5237eba5609.tar.bz2 coreboot-481661e313ddc2efb151cbdb77eeb5237eba5609.zip | |
amdfwtool: Add flag for multi FW level to the struct amd_cb_config
This change can make the code be more flexible. And later we will use
amd_cb_config to transfer parameters.
Change-Id: Ic726aa9fc5f67803210af71d3e9cf2438b7e2a9b
Signed-off-by: Zheng Bao <fishbaozi@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/57062
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'util')
| -rw-r--r-- | util/amdfwtool/amdfwtool.c | 20 | ||||
| -rw-r--r-- | util/amdfwtool/amdfwtool.h | 1 |
2 files changed, 11 insertions, 10 deletions
diff --git a/util/amdfwtool/amdfwtool.c b/util/amdfwtool/amdfwtool.c index 4ec9edf00641..1c662bedfc21 100644 --- a/util/amdfwtool/amdfwtool.c +++ b/util/amdfwtool/amdfwtool.c @@ -1251,7 +1251,6 @@ int main(int argc, char **argv) uint8_t efs_spi_speed = 0xff; uint8_t efs_spi_micron_flag = 0xff; - int multi = 0; amd_cb_config cb_config; int debug = 0; int list_deps = 0; @@ -1261,6 +1260,7 @@ int main(int argc, char **argv) cb_config.use_secureos = 0; cb_config.load_mp2_fw = 0; cb_config.s0i3 = 0; + cb_config.multi_level = 0; while (1) { int optindex = 0; @@ -1287,7 +1287,7 @@ int main(int argc, char **argv) comboable = 1; break; case AMDFW_OPT_MULTILEVEL: - multi = 1; + cb_config.multi_level = 1; break; case AMDFW_OPT_UNLOCK: register_fw_token_unlock(); @@ -1560,18 +1560,18 @@ int main(int argc, char **argv) ctx.current = ALIGN(ctx.current, 0x10000U); /* TODO: is it necessary? */ - if (multi) { + if (cb_config.multi_level) { /* Do 2nd PSP directory followed by 1st */ - psp_directory_table *pspdir2 = new_psp_dir(&ctx, multi); + psp_directory_table *pspdir2 = new_psp_dir(&ctx, cb_config.multi_level); integrate_psp_firmwares(&ctx, pspdir2, 0, amd_psp_fw_table, PSPL2_COOKIE); - pspdir = new_psp_dir(&ctx, multi); + pspdir = new_psp_dir(&ctx, cb_config.multi_level); integrate_psp_firmwares(&ctx, pspdir, pspdir2, amd_psp_fw_table, PSP_COOKIE); } else { /* flat: PSP 1 cookie and no pointer to 2nd table */ - pspdir = new_psp_dir(&ctx, multi); + pspdir = new_psp_dir(&ctx, cb_config.multi_level); integrate_psp_firmwares(&ctx, pspdir, 0, amd_psp_fw_table, PSP_COOKIE); } @@ -1596,19 +1596,19 @@ int main(int argc, char **argv) if (have_bios_tables(amd_bios_table)) { bios_directory_table *biosdir; - if (multi) { + if (cb_config.multi_level) { /* Do 2nd level BIOS directory followed by 1st */ bios_directory_table *biosdir2 = - new_bios_dir(&ctx, multi); + new_bios_dir(&ctx, cb_config.multi_level); integrate_bios_firmwares(&ctx, biosdir2, 0, amd_bios_table, BDT2_COOKIE); - biosdir = new_bios_dir(&ctx, multi); + biosdir = new_bios_dir(&ctx, cb_config.multi_level); integrate_bios_firmwares(&ctx, biosdir, biosdir2, amd_bios_table, BDT1_COOKIE); } else { /* flat: BDT1 cookie and no pointer to 2nd table */ - biosdir = new_bios_dir(&ctx, multi); + biosdir = new_bios_dir(&ctx, cb_config.multi_level); integrate_bios_firmwares(&ctx, biosdir, 0, amd_bios_table, BDT1_COOKIE); } diff --git a/util/amdfwtool/amdfwtool.h b/util/amdfwtool/amdfwtool.h index 6db74f66e27b..d246af793222 100644 --- a/util/amdfwtool/amdfwtool.h +++ b/util/amdfwtool/amdfwtool.h @@ -224,6 +224,7 @@ typedef struct _amd_cb_config { uint8_t unlock_secure; uint8_t use_secureos; uint8_t load_mp2_fw; + uint8_t multi_level; uint8_t s0i3; } amd_cb_config; |