commit 0e556326619ece4c7847ddc6ffccb19b02e22da4
author Aaron Durbin <adurbin@chromium.org> Fri Apr 29 23:15:12 2016 -0500
committer Aaron Durbin <adurbin@chromium.org> Mon May 02 20:07:25 2016 +0200
tree 0c3f0a5fb491ddd5f39c37c9d0924b8b09909d44
parent ddf4fa0cc35e6fc65d347b8c9eb4acbe0cba51b9

cpu/x86/mp_init: remove unused callback arguments

The BSP and AP callback declarations both had an optional argument
that could be passed. In practice that functionality was never used
so drop it.

Change-Id: I47fa814a593b6c2ee164c88d255178d3fb71e8ce
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/14556
Tested-by: build bot (Jenkins)
Reviewed-by: Werner Zeh <werner.zeh@siemens.com>
Reviewed-by: Leroy P Leahy <leroy.p.leahy@intel.com>
Reviewed-by: Furquan Shaikh <furquan@google.com>

src/soc/intel/fsp_baytrail/cpu.c

diff --git a/src/soc/intel/fsp_baytrail/cpu.c b/src/soc/intel/fsp_baytrail/cpu.c
index 1fed554..3856a06 100644
--- a/src/soc/intel/fsp_baytrail/cpu.c
+++ b/src/soc/intel/fsp_baytrail/cpu.c
@@ -33,18 +33,18 @@
 #if IS_ENABLED(CONFIG_HAVE_SMI_HANDLER)
 #include <soc/smm.h>
 
-static void smm_relocate(void *unused);
-static void enable_smis(void *unused);
+static void smm_relocate(void);
+static void enable_smis(void);
 
 static struct mp_flight_record mp_steps[] = {
-	MP_FR_BLOCK_APS(smm_relocate, NULL, smm_relocate, NULL),
-	MP_FR_BLOCK_APS(mp_initialize_cpu, NULL, mp_initialize_cpu, NULL),
+	MP_FR_BLOCK_APS(smm_relocate, smm_relocate),
+	MP_FR_BLOCK_APS(mp_initialize_cpu, mp_initialize_cpu),
 	/* Wait for APs to finish initialization before proceeding. */
-	MP_FR_BLOCK_APS(NULL, NULL, enable_smis, NULL),
+	MP_FR_BLOCK_APS(NULL, enable_smis),
 };
 #else /* CONFIG_HAVE_SMI_HANDLER */
 static struct mp_flight_record mp_steps[] = {
-	MP_FR_BLOCK_APS(mp_initialize_cpu, NULL, mp_initialize_cpu, NULL),
+	MP_FR_BLOCK_APS(mp_initialize_cpu, mp_initialize_cpu),
 };
 #endif
 
@@ -252,7 +252,7 @@
 	return 0;
 }
 
-static void smm_relocate(void *unused)
+static void smm_relocate(void)
 {
 
 	/* Load relocation and permanent handler. */
@@ -268,7 +268,7 @@
 	smm_initiate_relocation();
 }
 
-static void enable_smis(void *unused)
+static void enable_smis(void)
 {
 	southcluster_smm_enable_smi();
 }