[coreboot-gerrit] Change in coreboot[master]: amd/00730F01: Clean the Microcode updating

[Bao Zheng (Code Review)]

Hello build bot (Jenkins), Michał Żygowski, Patrick Georgi, Martin Roth, Marshall Dawson, Zheng Bao, Zheng Bao, Arthur Heymans, Felix Held,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/42094

to look at the new patch set (#10).

Change subject: amd/00730F01: Clean the Microcode updating
......................................................................

amd/00730F01: Clean the Microcode updating

According to the comments of
https://review.coreboot.org/c/coreboot/+/41719
, which is about Microcode patch for amd/picasso.
Change the code with the same way.

The changes include:
1. combine the microcode_xxx.c and update_microcode.c
   into one source.
2. Redefine the microcode updating function to eliminate
   the parameter. Get the revision ID in the black box.
   Reduce the depth of function calls.
3. Get the revision ID by bitwise calculation instead of
   lookup table.
4. Reduce the confusing type casts.
5. Squash some lines.

We do not change the way it used to be. The code assume
only one microcode is integrated in CBFS. If needed in future,
41719 is the example of integrating multiple binaries.

And, 41719 depends on the definition in this patch.

Change-Id: I8b0da99db0d3189058f75e199f05492c4e6c5881
Signed-off-by: Zheng Bao 
Signed-off-by: Zheng Bao 
---
M src/cpu/amd/pi/00730F01/Makefile.inc
D src/cpu/amd/pi/00730F01/microcode_fam16h.c
M src/cpu/amd/pi/00730F01/model_16_init.c
M src/cpu/amd/pi/00730F01/update_microcode.c
M src/include/cpu/amd/microcode.h
5 files changed, 118 insertions(+), 160 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/94/42094/10