[M] Change in coreboot[main]: soc/amd/common: Support sbin ucode files

[Patrick Rudolph (Code Review)]

Attention is currently required from: Felix Held, Fred Reitberger, Jason Glenesk, Martin Roth, Matt DeVillier, Maximilian Brune, Raul Rangel.

Patrick Rudolph has posted comments on this change by Fred Reitberger. ( https://review.coreboot.org/c/coreboot/+/68122?usp=email )

Change subject: soc/amd/common: Support sbin ucode files
......................................................................

Patch Set 9:

(4 comments)

File src/soc/amd/common/block/cpu/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/68122/comment/c6794507_331b6537?usp... :
PS2, Line 21: hexdump
> I tried `dd` and `printf` in the shell and couldn't get it to work, but I did find another solution  […]

Done

File src/soc/amd/common/block/cpu/Makefile.mk:

https://review.coreboot.org/c/coreboot/+/68122/comment/5510864d_44eaebd1?usp... :
PS5, Line 20: ucode
> This kind of shell code (if else stuff) is arguable harder to read if directly included inside the M […]

Done

https://review.coreboot.org/c/coreboot/+/68122/comment/f3eb6cef_f3b1cfbd?usp... :
PS5, Line 41: $(amd_microcode_bins)
> Doesn't that make it unnecessarily complicated? We would still need 2 loops to filter sbin and bin. […]

Done

https://review.coreboot.org/c/coreboot/+/68122/comment/9c9c61a4_a6b673fe?usp... :
PS5, Line 43: $(amd_microcode_sbins)
> $(filter %. […]

Done