[coreboot] Patch set updated for coreboot: 44ed94b Fix CONFIG_MAX_CPU set to 1 CPU build problem

Ronald G. Minnich (rminnich@gmail.com) gerrit at coreboot.org
Tue Nov 13 02:12:56 CET 2012


Ronald G. Minnich (rminnich at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1819

-gerrit

commit 44ed94b54d7e7e5d83dcea4954d6c89a6700cc78
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Mon Nov 12 15:17:24 2012 -0800

    Fix CONFIG_MAX_CPU set to 1 CPU build problem
    
    There are some function dependancies that didn't work
    when MAX_CPU was set to 1 and the build would fail.
    
    Change-Id: I033a42056f7b48a40316e03772ed89ad9cb013fe
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
---
 src/cpu/intel/model_206ax/model_206ax_init.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/cpu/intel/model_206ax/model_206ax_init.c b/src/cpu/intel/model_206ax/model_206ax_init.c
index f34c589..64c32a0 100644
--- a/src/cpu/intel/model_206ax/model_206ax_init.c
+++ b/src/cpu/intel/model_206ax/model_206ax_init.c
@@ -513,12 +513,14 @@ static void intel_cores_init(device_t cpu)
 		       cpu->path.apic.apic_id,
 		       new->path.apic.apic_id);
 
+#if CONFIG_SMP && CONFIG_MAX_CPUS > 1
 		/* Start the new cpu */
 		if (!start_cpu(new)) {
 			/* Record the error in cpu? */
 			printk(BIOS_ERR, "CPU %u would not start!\n",
 			       new->path.apic.apic_id);
 		}
+#endif
 	}
 }
 




More information about the coreboot mailing list