Felix Held submitted this change.

View Change


Approvals: build bot (Jenkins): Verified Felix Held: Looks good to me, approved ritul guru: Looks good to me, but someone else must approve Fred Reitberger: Looks good to me, approved
amdfwtool: Remove the duplicated entry RIB

It should be PSP_RIB_FILE which is already there.

Change-Id: Ie7471489bd34554e357510b04473102d002f9988
Signed-off-by: Zheng Bao <fishbaozi@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/72506
Reviewed-by: ritul guru <ritul.bits@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
---
M util/amdfwtool/data_parse.c
1 file changed, 17 insertions(+), 3 deletions(-)

diff --git a/util/amdfwtool/data_parse.c b/util/amdfwtool/data_parse.c
index 793fa9b..8a336d9 100644
--- a/util/amdfwtool/data_parse.c
+++ b/util/amdfwtool/data_parse.c
@@ -278,9 +278,6 @@
} else if (strcmp(fw_name, "PSP_C20MP_FILE") == 0) {
fw_type = AMD_FW_C20_MP;
subprog = 0;
- } else if (strcmp(fw_name, "RIB_FILE") == 0) {
- fw_type = AMD_RIB;
- subprog = 0;
} else if (strcmp(fw_name, "AMF_SRAM_FILE") == 0) {
fw_type = AMD_FW_AMF_SRAM;
subprog = 0;

To view, visit change 72506. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie7471489bd34554e357510b04473102d002f9988
Gerrit-Change-Number: 72506
Gerrit-PatchSet: 2
Gerrit-Owner: Bao Zheng <fishbaozi@gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: Fred Reitberger <reitbergerfred@gmail.com>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk@gmail.com>
Gerrit-Reviewer: Zheng Bao
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Reviewer: ritul guru <ritul.bits@gmail.com>
Gerrit-MessageType: merged