[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/skylake: Add Sagv enum value definition

[PraveenX Hodagatta Pranesh (Code Review)]

PraveenX Hodagatta Pranesh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/30305


Change subject: soc/intel/skylake: Add Sagv enum value definition
......................................................................

soc/intel/skylake: Add Sagv enum value definition

SaGv(system Agent Dynamic Frequency) have 4 settings
Disabled, Fixedlow, Fixedhigh, Enabled.
This patch add all 4 settings in enum definition and
used in devicetree.

BUG=None

Signed-off-by: Praveen hodagatta pranesh <praveenx.hodagatta.pranesh at intel.com>
Change-Id: I8f3b56f4d2bea1836373cc505ef5147144100b95
---
M src/soc/intel/skylake/chip.h
1 file changed, 6 insertions(+), 1 deletion(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/05/30305/1

diff --git a/src/soc/intel/skylake/chip.h b/src/soc/intel/skylake/chip.h
index 7014a2e..39e5056 100644
--- a/src/soc/intel/skylake/chip.h
+++ b/src/soc/intel/skylake/chip.h
@@ -152,7 +152,12 @@
 	 * 2 = FixedHigh
 	 * 3 = Enabled
 	*/
-	u8 SaGv;
+	enum {
+		SaGv_Disabled,
+		SaGv_FixedLow,
+		SaGv_FixedHigh,
+		SaGv_Enabled,
+	} SaGv;
 
 	/* Enable/disable Rank Margin Tool */
 	u8 Rmt;

-- 
To view, visit https://review.coreboot.org/c/coreboot/+/30305
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8f3b56f4d2bea1836373cc505ef5147144100b95
Gerrit-Change-Number: 30305
Gerrit-PatchSet: 1
Gerrit-Owner: PraveenX Hodagatta Pranesh <praveenx.hodagatta.pranesh at intel.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181219/2060fcec/attachment.html>