Re: [PATCH v6 082/102] x86: Move qemu CPU fixup function into its own file

On Sat, Dec 7, 2019 at 12:54 PM Simon Glass sjg@chromium.org wrote:

This function is specific to qemu so it seems best to keep it separate from the generic code.

Move it out to a new file and update the condition to use if() instead of #ifdef

Signed-off-by: Simon Glass sjg@chromium.org

Changes in v6:

• Add back '#ifdef' line to commit message
• Drop incorrect mention of coreboot in qfw_cpu.c

Changes in v5:

• Add a new patch to move qemu CPU fixup function into its own file

Changes in v4: None Changes in v3: None Changes in v2: None

arch/x86/cpu/Makefile | 1 + arch/x86/cpu/mp_init.c | 73 +++--------------------------------------- arch/x86/cpu/qfw_cpu.c | 73 ++++++++++++++++++++++++++++++++++++++++++ include/qfw.h | 8 +++++ 4 files changed, 87 insertions(+), 68 deletions(-) create mode 100644 arch/x86/cpu/qfw_cpu.c

Reviewed-by: Bin Meng bmeng.cn@gmail.com