diff options
| author | Subrata Banik <subratabanik@google.com> | 2022-08-12 17:14:43 +0530 |
|---|---|---|
| committer | Subrata Banik <subratabanik@google.com> | 2022-08-16 08:05:49 +0000 |
| commit | 5a9b7aa8e30fd68ff2178e11e5ba9d6935aba63f (patch) | |
| tree | 4e1276a4c5a5fd496b8f16434f6da458bc2c7e9f | |
| parent | 4b864e5c308fd0e4efdc1a99b7216c89f5ef0570 (diff) | |
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 <subratabanik@google.com>
Change-Id: I2c77959a76d2240ad1bfb7a9d7b9db7e8aee42f7
Reviewed-on: https://review.coreboot.org/c/coreboot/+/66685
Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
| -rw-r--r-- | src/soc/intel/common/block/cpu/mp_init.c | 4 |
1 files 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(); |