[coreboot-gerrit] Change in coreboot[master]: amd/picasso: Load x86 microcode from CBFS modules

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41719 )

Change subject: amd/picasso: Load x86 microcode from CBFS modules
......................................................................

Patch Set 20:

(2 comments)

https://review.coreboot.org/c/coreboot/+/41719/20/src/soc/amd/picasso/update... 
File src/soc/amd/picasso/update_microcode.c:

https://review.coreboot.org/c/coreboot/+/41719/20/src/soc/amd/picasso/update... 
PS20, Line 52: 		(new_patch_id == m->patch_id) ? "success" : "fail");
microcode: Updating to patch id = 0x%08x %s\n … succeeded/failed

The success message level should be `BIOS_INFO`, the failure message level should be `BIOS_ERR`.

https://review.coreboot.org/c/coreboot/+/41719/20/src/soc/amd/picasso/update... 
PS20, Line 69: 		m++) {
This fits on the line above.