[coreboot-gerrit] [S] Change in coreboot[master]: amdfwtool: Add entry RIB whose subprog equals 1

[Felix Held (Code Review)]

Attention is currently required from: Bao Zheng, Jason Glenesk, ritul guru, Zheng Bao, Fred Reitberger.

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/72507 )

Change subject: amdfwtool: Add entry RIB whose subprog equals 1
......................................................................

Patch Set 3:

(1 comment)

File util/amdfwtool/data_parse.c:

https://review.coreboot.org/c/coreboot/+/72507/comment/8ca8e86d_48557f59 
PS2, Line 414: 	} else if (strcmp(fw_name, "PSP_RIB_FILE") == 0) {
> Does this SUB1 often happen? […]

from document #55758 Rev. 2.02, Table 21 Sub-Programming Encoding i'd say that family 19h model 70h uses PSP_RIB_FILE_SUB0 and family 19h model 78h PSP_RIB_FILE_SUB1. we'll need to support both, so for consistency i'd use PSP_RIB_FILE_SUB0 instead of PSP_RIB_FILE to be consistent with PSP_RIB_FILE_SUB1