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

Bin Meng bmeng.cn at gmail.com
Fri Jun 22 04:13:26 UTC 2018


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(-)
>

Regards,
Bin


More information about the U-Boot mailing list