[U-Boot] [PATCH v3] x86: Use microcode update from device tree for all processors

[Bin Meng]

On Fri, Jun 29, 2018 at 5:06 AM, Ivan Gorinov <ivan.gorinov at intel.com> wrote:
> On Tue, Jun 26, 2018 at 03:47:49PM +0800, Bin Meng wrote:
>> Hi Ivan,
>>
>> On Fri, Jun 22, 2018 at 12:13 PM, Bin Meng <bmeng.cn at gmail.com> wrote:
>> > Hi Ivan,
>> >
>> > On Fri, Jun 22, 2018 at 12:16 PM, Bin Meng <bmeng.cn at gmail.com> wrote:
>> >> From: Ivan Gorinov <ivan.gorinov at intel.com>
>> >>
>> >> Built without a ROM image with FSP (u-boot.rom), the U-Boot loader applies
>> >> the microcode update data block encoded in Device Tree to the bootstrap
>> >> processor but not passed to the other CPUs when multiprocessing is enabled.
>> >>
>> >> If the bootstrap processor successfully performs a microcode update
>> >> from Device Tree, use the same data block for the other processors.
>> >>
>> >> Signed-off-by: Ivan Gorinov <ivan.gorinov at intel.com>
>> >> Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
>> >> [bmeng: fixed build errors on edison and qemu-x86]
>> >> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>> >>
>> >> ---
>> >>
>> >> Changes in v3:
>> >> - don't change arch/x86/cpu/i386/cpu.c to fix build errors on edison and
>> >>   qemu-x86
>> >>
>> >
>> > I don't think the update to arch/x86/cpu/i386/cpu.c in previous
>> > version is needed as mp_params.microcode_pointer is not needed during
>> > the MP boot. Please test this patch to see if it works on your board.
>> >
>> >>  arch/x86/cpu/intel_common/car.S       |  2 ++
>> >>  arch/x86/cpu/intel_common/microcode.c | 10 +++++++---
>> >>  arch/x86/include/asm/microcode.h      |  1 +
>> >>  arch/x86/lib/fsp/fsp_car.S            |  4 +++-
>> >>  4 files changed, 13 insertions(+), 4 deletions(-)
>>
>> Did you get a chance to test this? If no issue, I will take this patch
>> for v2018.07.
>
> No issues. Thank you!
>

applied to u-boot-x86, thanks!