erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

soc/intel/common/cpu: Remove the address-of (&) operator usage

Browse source 
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>
This commit is contained in:
Subrata Banik 2022-08-12 17:14:43 +05:30
commit 5a9b7aa8e3
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/soc/intel/common/block/cpu/mp_init.c
View file

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