commit 37e160efb5973ebe7b8b3f5e445d3bdf33c35dd5
author Felix Held <felix.held@amd.corp-partner.google.com> Wed Oct 20 23:43:12 2021 +0200
committer Felix Held <felix-coreboot@felixheld.de> Fri Oct 22 01:27:48 2021 +0000
tree 800f6f62b1f60dedd1690726c5f3ad0afd14e57b
parent 4dd7d1196545f1dff71b8e70b7a9b80b8bcd4041

soc/intel/baytrail: use mp_cpu_bus_init

Implement mp_init_cpus and use mp_cpu_bus_init as init function in
cpu_bus_ops.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I057ae8d95bdc510e9e7afb144b692531107fa45d
Reviewed-on: https://review.coreboot.org/c/coreboot/+/58506
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Raul Rangel <rrangel@chromium.org>

src/soc/intel/baytrail/chip.c

diff --git a/src/soc/intel/baytrail/chip.c b/src/soc/intel/baytrail/chip.c
index c7eb149..1f72dc4 100644
--- a/src/soc/intel/baytrail/chip.c
+++ b/src/soc/intel/baytrail/chip.c
@@ -17,7 +17,7 @@
 static struct device_operations cpu_bus_ops = {
 	.read_resources   = noop_read_resources,
 	.set_resources    = noop_set_resources,
-	.init             = baytrail_init_cpus,
+	.init             = mp_cpu_bus_init,
 };
 
 static void enable_dev(struct device *dev)

src/soc/intel/baytrail/cpu.c

diff --git a/src/soc/intel/baytrail/cpu.c b/src/soc/intel/baytrail/cpu.c
index 98400d1..1dbc3d7 100644
--- a/src/soc/intel/baytrail/cpu.c
+++ b/src/soc/intel/baytrail/cpu.c
@@ -12,6 +12,7 @@
 #include <cpu/x86/msr.h>
 #include <cpu/x86/mtrr.h>
 #include <cpu/x86/smm.h>
+#include <device/device.h>
 #include <reg_script.h>
 #include <soc/iosf.h>
 #include <soc/msr.h>
@@ -193,10 +194,8 @@
 	.post_mp_init        = post_mp_init,
 };
 
-void baytrail_init_cpus(struct device *dev)
+void mp_init_cpus(struct bus *cpu_bus)
 {
-	struct bus *cpu_bus = dev->link_list;
-
 	/* TODO: Handle mp_init_with_smm failure? */
 	mp_init_with_smm(cpu_bus, &mp_ops);
 }

src/soc/intel/baytrail/include/soc/ramstage.h

diff --git a/src/soc/intel/baytrail/include/soc/ramstage.h b/src/soc/intel/baytrail/include/soc/ramstage.h
index 51189cf..8b4727c 100644
--- a/src/soc/intel/baytrail/include/soc/ramstage.h
+++ b/src/soc/intel/baytrail/include/soc/ramstage.h
@@ -9,7 +9,6 @@
 /* The baytrail_init_pre_device() function is called prior to device
  * initialization, but it's after console and cbmem has been reinitialized. */
 void baytrail_init_pre_device(struct soc_intel_baytrail_config *config);
-void baytrail_init_cpus(struct device *dev);
 void southcluster_enable_dev(struct device *dev);
 /* The baytrail_run_reference_code() function is provided by either
  * native refcode or the ChromeBook refcode wrapper */