Change in coreboot[master]: /cpu/x86/mp_init: Rename c_handler to ap_handler

[Patrick Rudolph (Code Review)]

Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/47713 )

Change subject: /cpu/x86/mp_init: Rename c_handler to ap_handler
......................................................................

/cpu/x86/mp_init: Rename c_handler to ap_handler

Change-Id: I434fba031581b8acb685c82c5219238fe5b5cab2
Signed-off-by: Patrick Rudolph 
---
M src/cpu/x86/mp_init.c
M src/cpu/x86/sipi_vector.S
M src/cpu/x86/sipi_vector_c_handler.c
M src/include/cpu/x86/mp.h
4 files changed, 5 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/13/47713/1

diff --git a/src/cpu/x86/mp_init.c b/src/cpu/x86/mp_init.c
index b8d2008..0eee001 100644
--- a/src/cpu/x86/mp_init.c
+++ b/src/cpu/x86/mp_init.c
@@ -324,7 +324,7 @@
    	sp->microcode_lock = 0;
    else
    	sp->microcode_lock = ~0;
-	sp->c_handler = (uintptr_t)&ap_init;
+	sp->ap_handler = (uintptr_t)&ap_init;
    ap_count = &sp->ap_count;
    atomic_set(ap_count, 0);

diff --git a/src/cpu/x86/sipi_vector.S b/src/cpu/x86/sipi_vector.S
index cb5641a..394c03a 100644
--- a/src/cpu/x86/sipi_vector.S
+++ b/src/cpu/x86/sipi_vector.S
@@ -13,7 +13,7 @@
  * The SIPI vector is responsible for initializing the APs in the system. It
  * sets up protected mode and assigns a stack to each AP before it calls into
  * C code. The C handler loads microcode, sets up MSRs, and enables caching
- * before calling into the MP init provided 'c_handler'
+ * before calling into the MP init provided 'ap_handler'
  */
 #if ENV_X86_64
 #define uintptr_t quad
@@ -41,7 +41,7 @@
 .uintptr_t 0
 msr_count:
 .long 0
-c_handler:
+ap_handler:
 .uintptr_t 0
 ap_count:
 .long 0
diff --git a/src/cpu/x86/sipi_vector_c_handler.c b/src/cpu/x86/sipi_vector_c_handler.c
index 47cd11a..25e4587 100644
--- a/src/cpu/x86/sipi_vector_c_handler.c
+++ b/src/cpu/x86/sipi_vector_c_handler.c
@@ -108,6 +108,6 @@
    sipi_enable_caches(params);
    sipi_enable_sse(params);

-	void (*doit)(int) = (void (*)(int))params->c_handler;
+	void (*doit)(int) = (void (*)(int))params->ap_handler;
    doit(cpu_num);
 }
diff --git a/src/include/cpu/x86/mp.h b/src/include/cpu/x86/mp.h
index ed6e91f..26c3e05 100644
--- a/src/include/cpu/x86/mp.h
+++ b/src/include/cpu/x86/mp.h
@@ -155,7 +155,7 @@
    uintptr_t microcode_ptr;
    uintptr_t msr_table_ptr;
    uint32_t msr_count;
-	uintptr_t c_handler;
+	uintptr_t ap_handler;
    atomic_t ap_count;
 } __packed;

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47713 )

Change subject: /cpu/x86/mp_init: Rename c_handler to ap_handler
......................................................................

Patch Set 1: Code-Review+2

[Patrick Rudolph (Code Review)]

Patrick Rudolph has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/47713 )

Change subject: /cpu/x86/mp_init: Rename c_handler to ap_handler
......................................................................

Abandoned

patch series replaced by CB:49303