[coreboot-gerrit] [S] Change in coreboot[main]: mb/asus/p8h67-i_deluxe: Implement voltage settings

[Riku Viitanen (Code Review)]

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 
---
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