Change in coreboot[master]: soc/intel/alderlake: Allow devicetree override to leave some VR setti...

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/58807 )

Change subject: soc/intel/alderlake: Allow devicetree override to leave some VR settings as default
......................................................................

soc/intel/alderlake: Allow devicetree override to leave some VR settings as default

Allow devicetree override to leave ac_loadline, dc_loadline and icc_max as default.

Test=Boot to OS

Signed-off-by: Bora Guvendik 
Change-Id: I715345d5ea83aed9ee929b2a4e13921c9d8895b1
Reviewed-on: https://review.coreboot.org/c/coreboot/+/58807
Tested-by: build bot (Jenkins) 
Reviewed-by: Tim Wawrzynczak 
---
M src/soc/intel/alderlake/vr_config.c
1 file changed, 6 insertions(+), 3 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Tim Wawrzynczak: Looks good to me, approved

diff --git a/src/soc/intel/alderlake/vr_config.c b/src/soc/intel/alderlake/vr_config.c
index 2180fc5..99caacb 100644
--- a/src/soc/intel/alderlake/vr_config.c
+++ b/src/soc/intel/alderlake/vr_config.c
@@ -87,9 +87,12 @@
    if (chip_cfg->vr_config_enable) {
    	cfg = chip_cfg;

-		s_cfg->AcLoadline[domain] = cfg->ac_loadline;
-		s_cfg->DcLoadline[domain] = cfg->dc_loadline;
-		s_cfg->IccMax[domain] = cfg->icc_max;
+		if (cfg->ac_loadline)
+			s_cfg->AcLoadline[domain] = cfg->ac_loadline;
+		if (cfg->dc_loadline)
+			s_cfg->DcLoadline[domain] = cfg->dc_loadline;
+		if (cfg->icc_max)
+			s_cfg->IccMax[domain] = cfg->icc_max;
    	s_cfg->TdcTimeWindow[domain] = cfg->tdc_timewindow;
    	s_cfg->TdcCurrentLimit[domain] = cfg->tdc_currentlimit;
    } else {