[coreboot-gerrit] Patch set updated for coreboot: src/include: Capitalize APIC and SMM

HAOUAS Elyes (ehaouas@noos.fr) gerrit at coreboot.org
Sun Aug 21 13:14:59 CEST 2016


HAOUAS Elyes (ehaouas at noos.fr) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16278

-gerrit

commit 05d5f57af4d5fa0c2c2ed7d312e63d983ba55891
Author: Elyes HAOUAS <ehaouas at noos.fr>
Date:   Sun Aug 21 10:49:19 2016 +0200

    src/include: Capitalize APIC and SMM
    
    Change-Id: I9b3a2cce6c6bb85791d5cde076d5de95ef0e8790
    Signed-off-by: Elyes HAOUAS <ehaouas at noos.fr>
---
 src/include/cpu/x86/lapic.h | 2 +-
 src/include/cpu/x86/smm.h   | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/include/cpu/x86/lapic.h b/src/include/cpu/x86/lapic.h
index 16bc42d..dfcb4da 100644
--- a/src/include/cpu/x86/lapic.h
+++ b/src/include/cpu/x86/lapic.h
@@ -6,7 +6,7 @@
 #include <halt.h>
 #include <smp/node.h>
 
-/* See if I need to initialize the local apic */
+/* See if I need to initialize the local APIC */
 #if CONFIG_SMP || CONFIG_IOAPIC
 #  define NEED_LAPIC 1
 #else
diff --git a/src/include/cpu/x86/smm.h b/src/include/cpu/x86/smm.h
index c1051ad..8ea66ba 100644
--- a/src/include/cpu/x86/smm.h
+++ b/src/include/cpu/x86/smm.h
@@ -528,7 +528,7 @@ void *smm_get_save_state(int cpu);
  * - per_cpu_stack_size - stack size per CPU for smm modules.
  * - num_concurrent_stacks - number of concurrent cpus in handler needing stack
  *                           optional for setting up relocation handler.
- * - per_cpu_save_state_size - the smm save state size per cpu
+ * - per_cpu_save_state_size - the SMM save state size per cpu
  * - num_concurrent_save_states - number of concurrent cpus needing save state
  *                                space
  * - handler - optional handler to call. Only used during SMM relocation setup.
@@ -537,8 +537,8 @@ void *smm_get_save_state(int cpu);
  *                 the address of the module's parameters (if present).
  * - runtime - this field is a result only. The SMM runtime location is filled
  *             into this field so the code doing the loading can manipulate the
- *             runtime's assumptions. e.g. updating the apic id to CPU map to
- *             handle sparse apic id space.
+ *             runtime's assumptions. e.g. updating the APIC id to CPU map to
+ *             handle sparse APIC id space.
  */
 struct smm_loader_params {
 	void *stack_top;



More information about the coreboot-gerrit mailing list