From dc666f50c7bd479caf9d9b91a1769686c15cfdf9 Mon Sep 17 00:00:00 2001 From: Ronak Kanabar Date: Mon, 14 Jan 2019 21:47:56 +0530 Subject: soc/intel/cannonlake: Change in SaGv options CNL,WHL and CFL all are not using midfixed option in SaGv so keeping it for CNL only and removing it for others. Change-Id: I754515c2f8e249479c603872c61ac9a006e962ff Signed-off-by: Ronak Kanabar Reviewed-on: https://review.coreboot.org/c/30917 Reviewed-by: Lijian Zhao Reviewed-by: Subrata Banik Reviewed-by: Aamir Bohra Reviewed-by: Maulik V Vaghela Tested-by: build bot (Jenkins) --- src/soc/intel/cannonlake/chip.h | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/soc/intel/cannonlake/chip.h b/src/soc/intel/cannonlake/chip.h index 947cd65718..a877ec1a90 100644 --- a/src/soc/intel/cannonlake/chip.h +++ b/src/soc/intel/cannonlake/chip.h @@ -100,12 +100,18 @@ struct soc_intel_cannonlake_config { uint16_t FreqSaGvMid; /* System Agent dynamic frequency support. Only effects ULX/ULT CPUs. + * for CNL options are as following + * When enabled memory will be training at three different frequencies. + * 0:Disabled, 1:FixedLow, 2:FixedMid, 3:FixedHigh, 4:Enabled + * for WHL/CFL options are as following * When enabled memory will be training at two different frequencies. - * 0:Disabled, 1:FixedLow, 2:FixedMid, 3:FixedHigh, 4:Enabled */ + * 0:Disabled, 1:FixedLow, 2:FixedHigh, 3:Enabled*/ enum { SaGv_Disabled, SaGv_FixedLow, +#if IS_ENABLED(CONFIG_SOC_INTEL_CANNONLAKE) SaGv_FixedMid, +#endif SaGv_FixedHigh, SaGv_Enabled, } SaGv; -- cgit v1.2.3