[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso/cpu: move set_cstate_io_addr to common code

[Felix Held (Code Review)]

Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/50407 )

Change subject: soc/amd/picasso/cpu: move set_cstate_io_addr to common code
......................................................................

soc/amd/picasso/cpu: move set_cstate_io_addr to common code

Signed-off-by: Felix Held 
Change-Id: I3b8a38ea39d8dc56ff1249a3212fe352b3e805ce
---
M src/soc/amd/common/block/cpu/noncar/cpu.c
M src/soc/amd/common/block/include/amdblocks/cpu.h
M src/soc/amd/picasso/cpu.c
3 files changed, 13 insertions(+), 9 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/07/50407/1

diff --git a/src/soc/amd/common/block/cpu/noncar/cpu.c b/src/soc/amd/common/block/cpu/noncar/cpu.c
index 7c1daf9..98926bf 100644
--- a/src/soc/amd/common/block/cpu/noncar/cpu.c
+++ b/src/soc/amd/common/block/cpu/noncar/cpu.c
@@ -2,8 +2,20 @@

#include <amdblocks/cpu.h>
 #include <cpu/cpu.h>
+#include <cpu/x86/msr.h>
+#include <cpu/amd/msr.h>
+#include <soc/iomap.h>

int get_cpu_count(void)
 {
    return 1 + (cpuid_ecx(0x80000008) & 0xff);
 }
+
+void set_cstate_io_addr(void)
+{
+	msr_t cst_addr;
+
+	cst_addr.hi = 0;
+	cst_addr.lo = ACPI_CPU_CONTROL;
+	wrmsr(MSR_CSTATE_ADDRESS, cst_addr);
+}
diff --git a/src/soc/amd/common/block/include/amdblocks/cpu.h b/src/soc/amd/common/block/include/amdblocks/cpu.h
index 50300d5..0627bc1 100644
--- a/src/soc/amd/common/block/include/amdblocks/cpu.h
+++ b/src/soc/amd/common/block/include/amdblocks/cpu.h
@@ -4,6 +4,7 @@
 #define AMD_BLOCK_CPU_H

int get_cpu_count(void);
+void set_cstate_io_addr(void);
 void write_resume_eip(void);

#endif /* AMD_BLOCK_CPU_H */
diff --git a/src/soc/amd/picasso/cpu.c b/src/soc/amd/picasso/cpu.c
index b18a3b4..9460485 100644
--- a/src/soc/amd/picasso/cpu.c
+++ b/src/soc/amd/picasso/cpu.c
@@ -42,15 +42,6 @@
    x86_mtrr_check();
 }

-static void set_cstate_io_addr(void)
-{
-	msr_t cst_addr;
-
-	cst_addr.hi = 0;
-	cst_addr.lo = ACPI_CPU_CONTROL;
-	wrmsr(MSR_CSTATE_ADDRESS, cst_addr);
-}
-
 static void fill_in_relocation_params(struct smm_relocation_params *params)
 {
    uintptr_t tseg_base;