Riku Viitanen has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/86334?usp=email )
Change subject: mb/asus/p8h67-i_deluxe: Implement voltage settings ......................................................................
mb/asus/p8h67-i_deluxe: Implement voltage settings
Add NCT3933U chips and channel mappings to devicetree. This board can set DRAM, PCH, and VCCIO voltages, plus CPU and iGPU offsets (relative to what is requested by CPU via SVID).
Also enable automatic DRAM voltage setting from raminit. Tested with the following modules and a multimeter confirmed the voltages: - 2x HMT41GS6BFR8A-RD -> 1.35V - 2x HMT351S6CFR8C-PB -> 1.5V - HMT41GS6BFR8A-RD + M471B5273DH0-CH9 -> 1.5V
Change-Id: Ief2f02b0c2c2f6ba010df2ab4e6ecaef19e50a99 Signed-off-by: Riku Viitanen riku.viitanen@protonmail.com --- M src/mainboard/asus/h61-series/Kconfig M src/mainboard/asus/h61-series/variants/p8h67-i_deluxe/overridetree.cb 2 files changed, 27 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/86334/1
diff --git a/src/mainboard/asus/h61-series/Kconfig b/src/mainboard/asus/h61-series/Kconfig index 4c4fd7e..b4b3bf3 100644 --- a/src/mainboard/asus/h61-series/Kconfig +++ b/src/mainboard/asus/h61-series/Kconfig @@ -63,6 +63,7 @@ config BOARD_ASUS_P8H67_I_DELUXE select BOARD_ASUS_H61_SERIES select BOARD_ROMSIZE_KB_4096 + select DRIVERS_I2C_NCT3933U_DRAM select HAVE_CMOS_DEFAULT select HAVE_OPTION_TABLE select NO_UART_ON_SUPERIO @@ -90,6 +91,12 @@ default "P8H61-M PRO CM6630" if BOARD_ASUS_P8H61_M_PRO_CM6630 default "P8H67-I DELUXE" if BOARD_ASUS_P8H67_I_DELUXE
+config MAINBOARD_HW_MAXIMUM_DRAM_VOLTAGE + default 2135 if BOARD_ASUS_P8H67_I_DELUXE + +config MAINBOARD_HW_MINIMUM_DRAM_VOLTAGE + default 865 if BOARD_ASUS_P8H67_I_DELUXE + config OVERRIDE_DEVICETREE default "variants/$(CONFIG_VARIANT_DIR)/overridetree.cb"
diff --git a/src/mainboard/asus/h61-series/variants/p8h67-i_deluxe/overridetree.cb b/src/mainboard/asus/h61-series/variants/p8h67-i_deluxe/overridetree.cb index ef24784..13d595a7 100644 --- a/src/mainboard/asus/h61-series/variants/p8h67-i_deluxe/overridetree.cb +++ b/src/mainboard/asus/h61-series/variants/p8h67-i_deluxe/overridetree.cb @@ -92,6 +92,26 @@ register "sata_interface_speed_support" = "0x3" register "sata_port_map" = "0x1f" # 0-3: internal SATA, 4: eSATA end + device ref smbus on + device i2c 0x10 on + chip drivers/i2c/nct3933u + register "default_mv" = "{0, 0, 1050}" + register "step_uv" = "{0, 5000, 5000}" + device generic 1 off end # unknown + device generic 2 alias cpu_vcore_offset on end + device generic 3 alias vccio on end + end + end + device i2c 0x15 on + chip drivers/i2c/nct3933u + register "default_mv" = "{1500, 1050, 0}" + register "step_uv" = "{5000, 5000, 5000}" + device generic 1 alias v_dram on end + device generic 2 alias v_pch on end + device generic 3 alias igpu_vcore_offset on end + end + end + end end end end