Jamie Ryu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46644 )
Change subject: soc/intel/tigerlake: Add support for calling microcode update API ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/46644/3/src/soc/intel/tigerlake/uco... File src/soc/intel/tigerlake/ucode_update.c:
https://review.coreboot.org/c/coreboot/+/46644/3/src/soc/intel/tigerlake/uco... PS3, Line 1: /* SPDX-License-Identifier: GPL-2.0-only */
None of this is tigerlake specific. […]
Hi Arthur, this is now moved to the common code by https://review.coreboot.org/c/coreboot/+/27369/60. Thanks for your comments.
https://review.coreboot.org/c/coreboot/+/46644/3/src/soc/intel/tigerlake/uco... PS3, Line 15: INTEL_TOP_SWAP_MULTI_FIT_UCODE_UPDATE
This is already been taken care of by the makefile inclusion
This is now moved to the common code - https://review.coreboot.org/c/coreboot/+/27369/60 and updated to remove the config check. Since the implementation is moved to the common code and this patch is not needed anymore, the patch will be abandoned. Thanks.