soc/amd: introduce and use common amd_cpu_bus_ops struct

The device operations for the CPU bus are identical for all AMD SoCs, so
introduce a common device operations struct for this and use it in all
AMD SoC's chipset devicetrees as ops for the CPU cluster.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: Id32f89b8a33db8dbb747b917eeac3009fbae6631
Reviewed-on: https://review.coreboot.org/c/coreboot/+/71998
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Elyes Haouas <ehaouas@noos.fr>
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>

src/soc/amd/cezanne/chip.c

@@ -12,13 +12,6 @@
 #include <types.h>
 #include "chip.h"
 
-struct device_operations cezanne_cpu_bus_ops = {
-	.read_resources	= noop_read_resources,
-	.set_resources	= noop_set_resources,
-	.init		= mp_cpu_bus_init,
-	.acpi_fill_ssdt	= generate_cpu_entries,
-};
-
 static const char *soc_acpi_name(const struct device *dev)
 {
 	if (dev->path.type == DEVICE_PATH_DOMAIN)

src/soc/amd/cezanne/chipset.cb

@@ -1,6 +1,6 @@
 chip soc/amd/cezanne
 	device cpu_cluster 0 on
-		ops cezanne_cpu_bus_ops
+		ops amd_cpu_bus_ops
 	end
 	device domain 0 on
 		ops cezanne_pci_domain_ops

src/soc/amd/common/block/cpu/cpu.c

@@ -1,9 +1,18 @@
 /* SPDX-License-Identifier: GPL-2.0-only */
 
+#include <acpi/acpi.h>
 #include <amdblocks/cpu.h>
 #include <cpu/cpu.h>
+#include <device/device.h>
 
 int get_cpu_count(void)
 {
 	return 1 + (cpuid_ecx(0x80000008) & 0xff);
 }
+
+struct device_operations amd_cpu_bus_ops = {
+	.read_resources	= noop_read_resources,
+	.set_resources	= noop_set_resources,
+	.init		= mp_cpu_bus_init,
+	.acpi_fill_ssdt	= generate_cpu_entries,
+};

src/soc/amd/glinda/chip.c

@@ -14,13 +14,6 @@
 #include <types.h>
 #include "chip.h"
 
-struct device_operations glinda_cpu_bus_ops = {
-	.read_resources	= noop_read_resources,
-	.set_resources	= noop_set_resources,
-	.init		= mp_cpu_bus_init,
-	.acpi_fill_ssdt	= generate_cpu_entries,
-};
-
 static const char *soc_acpi_name(const struct device *dev)
 {
 	if (dev->path.type == DEVICE_PATH_DOMAIN)

src/soc/amd/glinda/chipset.cb

@@ -2,7 +2,7 @@
 
 chip soc/amd/glinda
 	device cpu_cluster 0 on
-		ops glinda_cpu_bus_ops
+		ops amd_cpu_bus_ops
 	end
 	device domain 0 on
 		ops glinda_pci_domain_ops

src/soc/amd/mendocino/chip.c

@@ -14,13 +14,6 @@
 #include <types.h>
 #include "chip.h"
 
-struct device_operations mendocino_cpu_bus_ops = {
-	.read_resources	= noop_read_resources,
-	.set_resources	= noop_set_resources,
-	.init		= mp_cpu_bus_init,
-	.acpi_fill_ssdt	= generate_cpu_entries,
-};
-
 static const char *soc_acpi_name(const struct device *dev)
 {
 	if (dev->path.type == DEVICE_PATH_DOMAIN)

src/soc/amd/mendocino/chipset_mendocino.cb

@@ -1,6 +1,6 @@
 chip soc/amd/mendocino
 	device cpu_cluster 0 on
-		ops mendocino_cpu_bus_ops
+		ops amd_cpu_bus_ops
 	end
 	device domain 0 on
 		ops mendocino_pci_domain_ops

src/soc/amd/mendocino/chipset_rembrandt.cb

@@ -1,6 +1,6 @@
 chip soc/amd/mendocino
 	device cpu_cluster 0 on
-		ops mendocino_cpu_bus_ops
+		ops amd_cpu_bus_ops
 	end
 	device domain 0 on
 		ops mendocino_pci_domain_ops

src/soc/amd/phoenix/chip.c

@@ -14,13 +14,6 @@
 #include <types.h>
 #include "chip.h"
 
-struct device_operations phoenix_cpu_bus_ops = {
-	.read_resources	= noop_read_resources,
-	.set_resources	= noop_set_resources,
-	.init		= mp_cpu_bus_init,
-	.acpi_fill_ssdt	= generate_cpu_entries,
-};
-
 static const char *soc_acpi_name(const struct device *dev)
 {
 	if (dev->path.type == DEVICE_PATH_DOMAIN)

src/soc/amd/phoenix/chipset.cb

@@ -2,7 +2,7 @@
 
 chip soc/amd/phoenix
 	device cpu_cluster 0 on
-		ops phoenix_cpu_bus_ops
+		ops amd_cpu_bus_ops
 	end
 	device domain 0 on
 		ops phoenix_pci_domain_ops

src/soc/amd/picasso/chip.c

@@ -13,13 +13,6 @@
 #include "chip.h"
 #include <fsp/api.h>
 
-struct device_operations picasso_cpu_bus_ops = {
-	.read_resources	  = noop_read_resources,
-	.set_resources	  = noop_set_resources,
-	.init		  = mp_cpu_bus_init,
-	.acpi_fill_ssdt   = generate_cpu_entries,
-};
-
 static const char *soc_acpi_name(const struct device *dev)
 {
 	if (dev->path.type == DEVICE_PATH_DOMAIN)

src/soc/amd/picasso/chipset.cb

@@ -2,7 +2,7 @@
 
 chip soc/amd/picasso
 	device cpu_cluster 0 on
-		ops picasso_cpu_bus_ops
+		ops amd_cpu_bus_ops
 	end
 	device domain 0 on
 		ops picasso_pci_domain_ops

src/soc/amd/stoneyridge/chip.c

@@ -18,13 +18,6 @@
 
 #include "chip.h"
 
-struct device_operations stoneyridge_cpu_bus_ops = {
-	.read_resources	  = noop_read_resources,
-	.set_resources	  = noop_set_resources,
-	.init		  = mp_cpu_bus_init,
-	.acpi_fill_ssdt   = generate_cpu_entries,
-};
-
 const char *soc_acpi_name(const struct device *dev)
 {
 	if (dev->path.type == DEVICE_PATH_DOMAIN)

src/soc/amd/stoneyridge/chipset_cz.cb

@@ -2,7 +2,7 @@
 
 chip soc/amd/stoneyridge
 	device cpu_cluster 0 on
-		ops stoneyridge_cpu_bus_ops
+		ops amd_cpu_bus_ops
 	end
 	device domain 0 on
 		ops stoneyridge_pci_domain_ops

src/soc/amd/stoneyridge/chipset_st.cb

@@ -2,7 +2,7 @@
 
 chip soc/amd/stoneyridge
 	device cpu_cluster 0 on
-		ops stoneyridge_cpu_bus_ops
+		ops amd_cpu_bus_ops
 	end
 	device domain 0 on
 		ops stoneyridge_pci_domain_ops