soc/amd/include/msr: factor out P state MSR enable bit to cpu/amd/msr.h
The bit position of the P state enable bit in the 8 P state MSRs is identical for all AMD chips including the family 16h model 30h APU that lives outside of soc/amd. The other bits in those 8 MSRs are more or less family- and model-specific. Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Change-Id: Ia69c33e28e2a91ff9a9bfe95859c1fd454921b77 Reviewed-on: https://review.coreboot.org/c/coreboot/+/73506 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com> Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
This commit is contained in:
		| @@ -41,6 +41,9 @@ | ||||
| #define PS_STS_REG			0xC0010063 | ||||
| #define PSTATE_0_MSR			0xC0010064 | ||||
| #define PSTATE_MSR(pstate)		(PSTATE_0_MSR + (pstate)) | ||||
| #define  PSTATE_DEF_HI_ENABLE_SHIFT	31 | ||||
| #define  PSTATE_DEF_HI_ENABLE_MASK	(0x1 << PSTATE_DEF_HI_ENABLE_SHIFT) | ||||
|  | ||||
| #define MSR_PATCH_LOADER		0xC0010020 | ||||
|  | ||||
| #define MSR_COFVID_STS			0xC0010071 | ||||
|   | ||||
| @@ -4,8 +4,6 @@ | ||||
| #define AMD_CEZANNE_MSR_H | ||||
|  | ||||
| /* MSRC001_00[6B:64] P-state [7:0] bit definitions */ | ||||
| #define PSTATE_DEF_HI_ENABLE_SHIFT	31 | ||||
| #define PSTATE_DEF_HI_ENABLE_MASK	(0x1 << PSTATE_DEF_HI_ENABLE_SHIFT) | ||||
| #define PSTATE_DEF_LO_CUR_DIV_SHIFT	30 | ||||
| #define PSTATE_DEF_LO_CUR_DIV_MASK	(0x3 << PSTATE_DEF_LO_CUR_DIV_SHIFT) | ||||
| #define PSTATE_DEF_LO_CUR_VAL_SHIFT	22 | ||||
|   | ||||
| @@ -7,7 +7,6 @@ | ||||
| #include <console/console.h> | ||||
| #include <cpu/amd/msr.h> | ||||
| #include <cpu/x86/msr.h> | ||||
| #include <soc/msr.h> | ||||
| #include <types.h> | ||||
|  | ||||
| /* | ||||
|   | ||||
| @@ -6,8 +6,6 @@ | ||||
| #define AMD_GLINDA_MSR_H | ||||
|  | ||||
| /* MSRC001_00[6B:64] P-state [7:0] bit definitions */ | ||||
| #define PSTATE_DEF_HI_ENABLE_SHIFT	31 | ||||
| #define PSTATE_DEF_HI_ENABLE_MASK	(0x1 << PSTATE_DEF_HI_ENABLE_SHIFT) | ||||
| #define PSTATE_DEF_LO_CUR_DIV_SHIFT	30 | ||||
| #define PSTATE_DEF_LO_CUR_DIV_MASK	(0x3 << PSTATE_DEF_LO_CUR_DIV_SHIFT) | ||||
| #define PSTATE_DEF_LO_CUR_VAL_SHIFT	22 | ||||
|   | ||||
| @@ -4,8 +4,6 @@ | ||||
| #define AMD_MENDOCINO_MSR_H | ||||
|  | ||||
| /* MSRC001_00[6B:64] P-state [7:0] bit definitions */ | ||||
| #define PSTATE_DEF_HI_ENABLE_SHIFT	31 | ||||
| #define PSTATE_DEF_HI_ENABLE_MASK	(0x1 << PSTATE_DEF_HI_ENABLE_SHIFT) | ||||
| #define PSTATE_DEF_LO_CUR_DIV_SHIFT	30 | ||||
| #define PSTATE_DEF_LO_CUR_DIV_MASK	(0x3 << PSTATE_DEF_LO_CUR_DIV_SHIFT) | ||||
| #define PSTATE_DEF_LO_CUR_VAL_SHIFT	22 | ||||
|   | ||||
| @@ -6,8 +6,6 @@ | ||||
| #define AMD_PHOENIX_MSR_H | ||||
|  | ||||
| /* MSRC001_00[6B:64] P-state [7:0] bit definitions */ | ||||
| #define PSTATE_DEF_HI_ENABLE_SHIFT	31 | ||||
| #define PSTATE_DEF_HI_ENABLE_MASK	(0x1 << PSTATE_DEF_HI_ENABLE_SHIFT) | ||||
| #define PSTATE_DEF_LO_CUR_DIV_SHIFT	30 | ||||
| #define PSTATE_DEF_LO_CUR_DIV_MASK	(0x3 << PSTATE_DEF_LO_CUR_DIV_SHIFT) | ||||
| #define PSTATE_DEF_LO_CUR_VAL_SHIFT	22 | ||||
|   | ||||
| @@ -8,8 +8,6 @@ | ||||
| #define AMD_PICASSO_MSR_H | ||||
|  | ||||
| /* MSRC001_00[6B:64] P-state [7:0] bit definitions */ | ||||
| #define PSTATE_DEF_HI_ENABLE_SHIFT	31 | ||||
| #define PSTATE_DEF_HI_ENABLE_MASK	(0x1 << PSTATE_DEF_HI_ENABLE_SHIFT) | ||||
| #define PSTATE_DEF_LO_CUR_DIV_SHIFT	30 | ||||
| #define PSTATE_DEF_LO_CUR_DIV_MASK	(0x3 << PSTATE_DEF_LO_CUR_DIV_SHIFT) | ||||
| #define PSTATE_DEF_LO_CUR_VAL_SHIFT	22 | ||||
|   | ||||
		Reference in New Issue
	
	Block a user