From 5a9b7aa8e30fd68ff2178e11e5ba9d6935aba63f Mon Sep 17 00:00:00 2001 From: Subrata Banik Date: Fri, 12 Aug 2022 17:14:43 +0530 Subject: [PATCH] soc/intel/common/cpu: Remove the address-of (`&`) operator usage This patch drops explicit usage of the address-of operator ('&') while passing the function pointer (argument 0) to the `mp_run_on_all_cpus` API. Note: It's just cosmetic change without any real difference in the operation. TEST=Able to build and boot Google/Kano where CPU feature programming is successful on all logical processors. Signed-off-by: Subrata Banik Change-Id: I2c77959a76d2240ad1bfb7a9d7b9db7e8aee42f7 Reviewed-on: https://review.coreboot.org/c/coreboot/+/66685 Reviewed-by: Lean Sheng Tan Reviewed-by: Angel Pons Tested-by: build bot (Jenkins) Reviewed-by: Eric Lai --- src/soc/intel/common/block/cpu/mp_init.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/soc/intel/common/block/cpu/mp_init.c b/src/soc/intel/common/block/cpu/mp_init.c index 8ec6c6660d..754c485ba5 100644 --- a/src/soc/intel/common/block/cpu/mp_init.c +++ b/src/soc/intel/common/block/cpu/mp_init.c @@ -193,10 +193,10 @@ void before_post_cpus_init(void) if (!CONFIG(USE_COREBOOT_MP_INIT)) return; - if (mp_run_on_all_cpus(&wrapper_init_core_prmrr, NULL) != CB_SUCCESS) + if (mp_run_on_all_cpus(wrapper_init_core_prmrr, NULL) != CB_SUCCESS) printk(BIOS_ERR, "core PRMRR sync failure\n"); - if (mp_run_on_all_cpus(&wrapper_set_bios_done, NULL) != CB_SUCCESS) + if (mp_run_on_all_cpus(wrapper_set_bios_done, NULL) != CB_SUCCESS) printk(BIOS_ERR, "Set BIOS Done failure\n"); intel_reload_microcode();