[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/skylake: Add Sagv enum value definition
Patrick Georgi (Code Review)
gerrit at coreboot.org
Wed Dec 19 06:56:05 CET 2018
Patrick Georgi has submitted this change and it was merged. ( 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
Reviewed-on: https://review.coreboot.org/c/30305
Tested-by: build bot (Jenkins) <no-reply at coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi at google.com>
---
M src/soc/intel/skylake/chip.h
1 file changed, 6 insertions(+), 1 deletion(-)
Approvals:
build bot (Jenkins): Verified
Patrick Georgi: Looks good to me, approved
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: 2
Gerrit-Owner: PraveenX Hodagatta Pranesh <praveenx.hodagatta.pranesh at intel.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi at google.com>
Gerrit-Reviewer: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: PraveenX Hodagatta Pranesh <praveenx.hodagatta.pranesh at intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181219/f0aa812e/attachment.html>
More information about the coreboot-gerrit
mailing list