Improving BKDG implementation of P-states,
CPU and northbridge frequency and voltage handling for Fam 10 in SVI mode. I didn't understand quite why it did that iwth F3xA0 (Power Control Misc Register) so I moved Pll Lock time to rules in defaults.h and reimplemented F3xA0 programming. A later patch will remove a part I don't know what's mean to do. Signed-off-by: Xavi Drudis Ferran <xdrudis@tinet.cat> Acked-by: Marc Jones <marcj303@gmail.com> git-svn-id: svn://svn.coreboot.org/coreboot/trunk@6396 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
This commit is contained in:
parent
adb23a51f5
commit
0e5d3e16b4
|
@ -68,7 +68,7 @@ static const struct {
|
|||
1 << 24, 0x00000000,
|
||||
1 << 24, 0x00000000 }, /* Erratum #261 [DIS_PIGGY_BACK_SCRUB]=1 */
|
||||
|
||||
{ LS_CFG, AMD_FAM10_GT_B0, AMD_PTYPE_ALL,
|
||||
{ LS_CFG, AMD_DR_GT_B0, AMD_PTYPE_ALL,
|
||||
0 << 1, 0x00000000,
|
||||
1 << 1, 0x00000000 }, /* IDX_MATCH_ALL=0 */
|
||||
|
||||
|
@ -261,8 +261,11 @@ static const struct {
|
|||
{ 3, 0xA0, AMD_FAM10_ALL, AMD_PTYPE_MOB | AMD_PTYPE_DSK,
|
||||
0x00000080, 0x00000080 }, /* [7] PSIVidEnable */
|
||||
|
||||
{ 3, 0xA0, AMD_FAM10_ALL, AMD_PTYPE_ALL,
|
||||
0x00001800, 0x000003800 }, /* [13:11] PllLockTime = 3 */
|
||||
{ 3, 0xA0, AMD_DR_Bx, AMD_PTYPE_ALL,
|
||||
0x00002800, 0x000003800 }, /* [13:11] PllLockTime = 5 */
|
||||
|
||||
{ 3, 0xA0, (AMD_FAM10_ALL & ~(AMD_DR_Bx)), AMD_PTYPE_ALL,
|
||||
0x00000800, 0x000003800 }, /* [13:11] PllLockTime = 1 */
|
||||
|
||||
/* Reported Temp Control Register */
|
||||
{ 3, 0xA4, AMD_FAM10_ALL, AMD_PTYPE_ALL,
|
||||
|
|
|
@ -275,19 +275,18 @@ static void config_clk_power_ctrl_reg0(int node, u32 cpuRev, u8 procPkg) {
|
|||
|
||||
}
|
||||
|
||||
static void config_power_ctrl_misc_reg(device_t dev) {
|
||||
static void config_power_ctrl_misc_reg(device_t dev,u32 cpuRev, u8 procPkg) {
|
||||
/* check PVI/SVI */
|
||||
u32 dword = pci_read_config32(dev, 0xA0);
|
||||
|
||||
/* BKDG r31116 2010-04-22 2.4.1.7 step b F3xA0[VSSlamVidMod] */
|
||||
/* PllLockTime and PsiVidEn set in ruleset in defaults.h */
|
||||
if (dword & PVI_MODE) { /* PVI */
|
||||
/* set slamVidMode to 0 for PVI */
|
||||
dword &= VID_SLAM_OFF | PLLLOCK_OFF;
|
||||
dword |= PLLLOCK_DFT_L;
|
||||
pci_write_config32(dev, 0xA0, dword);
|
||||
dword &= VID_SLAM_OFF ;
|
||||
} else { /* SVI */
|
||||
/* set slamVidMode to 1 for SVI */
|
||||
dword &= PLLLOCK_OFF;
|
||||
dword |= PLLLOCK_DFT_L | VID_SLAM_ON;
|
||||
pci_write_config32(dev, 0xA0, dword);
|
||||
dword |= VID_SLAM_ON;
|
||||
|
||||
u32 dtemp = dword;
|
||||
|
||||
|
@ -299,7 +298,27 @@ static void config_power_ctrl_misc_reg(device_t dev) {
|
|||
else
|
||||
dword &= PWR_PLN_OFF;
|
||||
pci_write_config32(dev, 0xD8, dword);
|
||||
|
||||
dword = dtemp;
|
||||
}
|
||||
/* set the rest of A0 since we're at it... */
|
||||
|
||||
if (cpuRev & (AMD_DA_Cx | AMD_RB_C3 )) {
|
||||
dword |= NB_PSTATE_FORCE_ON;
|
||||
} // else should we clear it ?
|
||||
|
||||
|
||||
if ((procPkg == AMD_PKGTYPE_G34) || (procPkg == AMD_PKGTYPE_C32) ) {
|
||||
dword |= BP_INS_TRI_EN_ON ;
|
||||
}
|
||||
|
||||
/* TODO: look into C1E state and F3xA0[IdleExitEn]*/
|
||||
#if CONFIG_SVI_HIGH_FREQ
|
||||
if (cpuRev & AMD_FAM10_C3) {
|
||||
dword |= SVI_HIGH_FREQ_ON;
|
||||
}
|
||||
#endif
|
||||
pci_write_config32(dev, 0xA0, dword);
|
||||
}
|
||||
|
||||
static void config_nb_syn_ptr_adj(device_t dev) {
|
||||
|
@ -344,8 +363,7 @@ static void prep_fid_change(void)
|
|||
|
||||
config_clk_power_ctrl_reg0(i,cpuRev,procPkg);
|
||||
|
||||
config_power_ctrl_misc_reg(dev);
|
||||
|
||||
config_power_ctrl_misc_reg(dev,cpuRev,procPkg);
|
||||
config_nb_syn_ptr_adj(dev);
|
||||
|
||||
config_acpi_pwr_state_ctrl_regs(dev);
|
||||
|
|
|
@ -112,4 +112,12 @@ if DIMM_DDR3
|
|||
endif
|
||||
endif
|
||||
|
||||
config SVI_HIGH_FREQ
|
||||
bool
|
||||
default n
|
||||
depends on NORTHBRIDGE_AMD_AMDFAM10
|
||||
help
|
||||
Select this for boards with a Voltage Regulator able to operate
|
||||
at 3.4 MHz in SVI mode. Ignored unless the AMD CPU is rev C3.
|
||||
|
||||
source src/northbridge/amd/amdfam10/root_complex/Kconfig
|
||||
|
|
|
@ -198,11 +198,19 @@
|
|||
#define PVI_MODE 0x100 /* PviMode bit mask */
|
||||
#define VID_SLAM_OFF 0x0dfffffff /* set VidSlamMode OFF */
|
||||
#define VID_SLAM_ON 0x020000000 /* set VidSlamMode ON */
|
||||
#define NB_PSTATE_FORCE_ON 0x010000000 /* set Northbridge P-state
|
||||
force on next LDTSTOP
|
||||
assertion on, in F3xA0 */
|
||||
#define BP_INS_TRI_EN_ON 0x00004000 /* breakpoint pins tristate
|
||||
enable in F3xA0 */
|
||||
#define PLLLOCK_OFF 0x0ffffc7ff /* PllLockTime Mask OFF */
|
||||
#define PLLLOCK_DFT 0x00001800 /* PllLockTime default value = 011b */
|
||||
#define PLLLOCK_DFT_L 0x00002800 /* PllLockTime long value = 101b */
|
||||
|
||||
/* P-state Specification register base in PCI sapce */
|
||||
#define SVI_HIGH_FREQ_ON 0x00000200 /* F3xA0[SviHighFreqSel] for
|
||||
3.4 MHz SVI in rev. C3 */
|
||||
|
||||
/* P-state Specification register base in PCI space */
|
||||
#define PS_SPEC_REG 0x1e0 /* PS Spec register base address */
|
||||
#define PCI_REG_LEN 4 /* PCI register length */
|
||||
#define NB_DID_MASK 0x10000 /* NbDid bit mask */
|
||||
|
|
|
@ -64,7 +64,9 @@
|
|||
#define AMD_DR_ALL (AMD_DR_Bx)
|
||||
#define AMD_FAM10_ALL (AMD_DR_ALL | AMD_RB_C2 | AMD_HY_D0 | AMD_DA_C3 | AMD_DA_C2 | AMD_RB_C3 )
|
||||
#define AMD_FAM10_GT_B0 (AMD_FAM10_ALL & ~(AMD_DR_B0))
|
||||
#define AMD_DR_Cx (AMD_RB_C2 | AMD_DA_C2 | AMD_RB_C3 | AMD_DA_C3)
|
||||
#define AMD_DA_Cx (AMD_DA_C2 | AMD_DA_C3)
|
||||
#define AMD_DR_Cx (AMD_RB_C2 | AMD_RB_C3 | AMD_DA_Cx)
|
||||
#define AMD_FAM10_C3 (AMD_RB_C3 | AMD_DA_C3)
|
||||
#define AMD_DR_Dx (AMD_HY_D0)
|
||||
#define AMD_DRBH_Cx (AMD_DR_Cx | AMD_HY_D0 )
|
||||
#define AMD_DRBA23_RBC2 (AMD_DR_BA | AMD_DR_B2 | AMD_DR_B3 | AMD_RB_C2 )
|
||||
|
|
Loading…
Reference in New Issue