GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/amd/common/block/cpu/smm/finalize: simplify finalize_cores 

The local variable int r isn't needed, so remove it. This is a
preparation to change the return type of mp_run_on_all_cpus from int to
enum cb_err which will be done in a follow-up patch.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: Ie4c454cbfcc581be41ea3463ea6f852a72886128
Reviewed-on: https://review.coreboot.org/c/coreboot/+/58493
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
This commit is contained in:
Felix Held 2021-10-20 20:45:28 +02:00 committed by Felix Held
parent 30f7031488
commit 250988d943
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/soc/amd/common/block/cpu/smm/finalize.c
View File

@ -29,11 +29,9 @@ static void per_core_finalize(void *unused)
static void finalize_cores(void) static void finalize_cores(void)
{ {
int r;
printk(BIOS_SPEW, "Lock SMM configuration\n"); printk(BIOS_SPEW, "Lock SMM configuration\n");
r = mp_run_on_all_cpus(per_core_finalize, NULL); if (mp_run_on_all_cpus(per_core_finalize, NULL))
if (r)
printk(BIOS_WARNING, "Failed to finalize all cores\n"); printk(BIOS_WARNING, "Failed to finalize all cores\n");
} }