diff options
| author | Zheng Bao <fishbaozi@gmail.com> | 2022-08-18 15:26:39 +0800 |
|---|---|---|
| committer | Fred Reitberger <reitbergerfred@gmail.com> | 2023-03-01 13:25:13 +0000 |
| commit | fd51af6286d939a8308ff65260f0fa7ac948507d (patch) | |
| tree | 062ed988e874556b7e2e229063a49b33e405d108 | |
| parent | c91867af13896ceddae00cb1e8d32614b2158b45 (diff) | |
amdfwtool:combo: Move the filling of field "lookup" into function
This filling does not need to be done separately.
Change-Id: I53051349923dce40f4fc3f747ab41a93a3798823
Signed-off-by: Zheng Bao <fishbaozi@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/66853
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
| -rw-r--r-- | util/amdfwtool/amdfwtool.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/util/amdfwtool/amdfwtool.c b/util/amdfwtool/amdfwtool.c index 0d5acee111..19a9438ddc 100644 --- a/util/amdfwtool/amdfwtool.c +++ b/util/amdfwtool/amdfwtool.c @@ -632,8 +632,9 @@ static void fill_dir_header(void *directory, uint32_t count, uint32_t cookie, co switch (cookie) { case PSP2_COOKIE: - /* caller is responsible for lookup mode */ cdir->header.cookie = cookie; + /* lookup mode is hardcoded for now. */ + cdir->header.lookup = 1; cdir->header.num_entries = count; cdir->header.reserved[0] = 0; cdir->header.reserved[1] = 0; @@ -2604,7 +2605,6 @@ int main(int argc, char **argv) combo_dir->entries[0].lvl2_addr = BUFF_TO_RUN_MODE(ctx, pspdir, AMD_ADDR_REL_BIOS); - combo_dir->header.lookup = 1; fill_dir_header(combo_dir, 1, PSP2_COOKIE, &ctx); } |