[coreboot-gerrit] Change in coreboot[master]: amdfwtool: Add support for AMD's BIOS A/B recovery feature

[Felix Held (Code Review)]

Attention is currently required from: Bao Zheng, Jason Glenesk, Raul Rangel, Martin Roth, Marshall Dawson, Zheng Bao.
Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56773 )

Change subject: amdfwtool: Add support for AMD's BIOS A/B recovery feature
......................................................................

Patch Set 32:

(2 comments)

File util/amdfwtool/amdfwtool.c:

https://review.coreboot.org/c/coreboot/+/56773/comment/2b6bda5c_744ea72b 
PS32, Line 237:  
not directly related to this patch: would be good to have the .subprog = 0 assignment explicitly for consistency

https://review.coreboot.org/c/coreboot/+/56773/comment/c0e35255_3bfef837 
PS32, Line 271: 	{ .type = AMD_FW_PSP_SMU_FIRMWARE, .subprog = 1, .level = PSP_BOTH },
              : 	{ .type = AMD_FW_PSP_SMU_FIRMWARE2, .subprog = 1, .level = PSP_BOTH },
not directly related to this patch: these two entries seem to be duplicates of entries above. is this needed on any platform?