[coreboot-gerrit] [S] Change in coreboot[master]: amdfwtool: Remove the duplicated entry RIB

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/72506 )

Change subject: amdfwtool: Remove the duplicated entry RIB
......................................................................

amdfwtool: Remove the duplicated entry RIB

It should be PSP_RIB_FILE which is already there.

Change-Id: Ie7471489bd34554e357510b04473102d002f9988
Signed-off-by: Zheng Bao 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/72506
Reviewed-by: ritul guru 
Tested-by: build bot (Jenkins) 
Reviewed-by: Fred Reitberger 
Reviewed-by: Felix Held 
---
M util/amdfwtool/data_parse.c
1 file changed, 17 insertions(+), 3 deletions(-)

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

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;