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

[Bin Meng]

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.

Regards,
Bin