Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/59583
to look at the new patch set (#2).
Change subject: amdfwtool: replace strcmp to switch-case ......................................................................
amdfwtool: replace strcmp to switch-case
Since we have separate function for filename to fw_type conversion, when we determine whether we should include the firmware or not we have fw_type.
Replace strcmp in if-else to switch-case so it looks more clear. However PSPBTLDR_WL_FILE and PSPBTLDR_AB_FILE have same fw_type so we should use strmp for them.
BUG=b:206909680 TEST=build guybrush and confirm amdfw and coreboot.rom are identical
Signed-off-by: Kangheui Won khwon@chromium.org Change-Id: Icf9f3465a36d83cf79f9f4bd1ec824b546a71aaf --- M util/amdfwtool/data_parse.c 1 file changed, 29 insertions(+), 34 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/83/59583/2