diff options
| author | Subrata Banik <subrata.banik@intel.com> | 2018-04-24 13:45:30 +0530 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2018-05-14 08:39:42 +0000 |
| commit | 3337497d2a1995614afd4a554747d86e4bcd8d31 (patch) | |
| tree | d633ef07da222f54c90057d0e9b179c785f2a948 /src/soc/amd | |
| parent | 223fb436fe89a265f4ddbe7f8fe38a35b47a8253 (diff) | |
| download | coreboot-3337497d2a1995614afd4a554747d86e4bcd8d31.tar.xz | |
cpu/x86: Add support to run function with argument over APs
This patch ensures that user can pass a function with given argument
list to execute over APs.
BUG=b:74436746
BRANCH=none
TEST=Able to run functions over APs with argument.
Change-Id: I668b36752f6b21cb99cd1416c385d53e96117213
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-on: https://review.coreboot.org/25725
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'src/soc/amd')
| -rw-r--r-- | src/soc/amd/common/block/pi/def_callouts.c | 6 | ||||
| -rw-r--r-- | src/soc/amd/stoneyridge/finalize.c | 4 |
2 files changed, 5 insertions, 5 deletions
diff --git a/src/soc/amd/common/block/pi/def_callouts.c b/src/soc/amd/common/block/pi/def_callouts.c index fc3a88b949..0afa82517d 100644 --- a/src/soc/amd/common/block/pi/def_callouts.c +++ b/src/soc/amd/common/block/pi/def_callouts.c @@ -192,7 +192,7 @@ static struct agesa_data { * BSP deploys APs to callout_ap_entry(), which calls * agesawrapper_amdlaterunaptask with the agesadata. */ -static void callout_ap_entry(void) +static void callout_ap_entry(void *unused) { AGESA_STATUS Status = AGESA_UNSUPPORTED; @@ -219,7 +219,7 @@ AGESA_STATUS agesa_RunFuncOnAp(UINT32 Func, UINTN Data, VOID *ConfigPtr) agesadata.Func = Func; agesadata.Data = Data; agesadata.ConfigPtr = ConfigPtr; - mp_run_on_aps(callout_ap_entry, 100 * USECS_PER_MSEC); + mp_run_on_aps(callout_ap_entry, NULL, 100 * USECS_PER_MSEC); return AGESA_SUCCESS; } @@ -231,7 +231,7 @@ AGESA_STATUS agesa_RunFcnOnAllAps(UINT32 Func, UINTN Data, VOID *ConfigPtr) agesadata.Func = Func; agesadata.Data = Data; agesadata.ConfigPtr = ConfigPtr; - mp_run_on_aps(callout_ap_entry, 100 * USECS_PER_MSEC); + mp_run_on_aps(callout_ap_entry, NULL, 100 * USECS_PER_MSEC); return AGESA_SUCCESS; } diff --git a/src/soc/amd/stoneyridge/finalize.c b/src/soc/amd/stoneyridge/finalize.c index 67f9e67df4..21d203cdd2 100644 --- a/src/soc/amd/stoneyridge/finalize.c +++ b/src/soc/amd/stoneyridge/finalize.c @@ -20,7 +20,7 @@ #include <timer.h> #include <console/console.h> -static void per_core_finalize(void) +static void per_core_finalize(void *unused) { msr_t hwcr, mask; @@ -44,7 +44,7 @@ static void finalize_cores(void) int r; printk(BIOS_SPEW, "Lock SMM configuration\n"); - r = mp_run_on_all_cpus(per_core_finalize, 10 * USECS_PER_MSEC); + r = mp_run_on_all_cpus(per_core_finalize, NULL, 10 * USECS_PER_MSEC); if (r) printk(BIOS_WARNING, "Failed to finalize all cores\n"); } |