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

[Felix Held (Code Review)]

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

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

Patch Set 6: Code-Review+1

(1 comment)

The rest looks good to me; would be good if Michał could verify that the microcode update still works on the APU2 board with the patch applied

https://review.coreboot.org/c/coreboot/+/42094/4/src/cpu/amd/pi/00730F01/upd... 
File src/cpu/amd/pi/00730F01/update_microcode.c:

https://review.coreboot.org/c/coreboot/+/42094/4/src/cpu/amd/pi/00730F01/upd... 
PS4, Line 104: m = (struct microcode *)c;
             : 
             : 	if (m->processor_rev_id == equivalent_processor_rev_id)
             : 		apply_microcode_patch(m);
> Currently only one blob is in the CBFS. I dont change the original code flow.

I'm ok with keeping the code in it's current form, but please add a comment about this issue, because this really is unexpected behaviour