[coreboot-gerrit] Change in coreboot[master]: intel/common/systemagent: Add APIs to get SOC specific PRMRR base and...

[Pratikkumar V Prajapati (Code Review)]

Pratikkumar V Prajapati has uploaded this change for review. ( https://review.coreboot.org/21243


Change subject: intel/common/systemagent: Add APIs to get SOC specific PRMRR base and mask
......................................................................

intel/common/systemagent: Add APIs to get SOC specific PRMRR base and mask

Implement weak definitions and SOC specific code would implement
actual definitions.

Change-Id: I5e787a2603aaa475cb5c61558cc31ec0afcb4a8b
Signed-off-by: Pratik Prajapati <pratikkumar.v.prajapati at intel.com>
---
M src/soc/intel/common/block/include/intelblocks/systemagent.h
M src/soc/intel/common/block/systemagent/systemagent.c
2 files changed, 16 insertions(+), 0 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/43/21243/1

diff --git a/src/soc/intel/common/block/include/intelblocks/systemagent.h b/src/soc/intel/common/block/include/intelblocks/systemagent.h
index 5f04e36..a1c551f 100644
--- a/src/soc/intel/common/block/include/intelblocks/systemagent.h
+++ b/src/soc/intel/common/block/include/intelblocks/systemagent.h
@@ -16,6 +16,7 @@
 #ifndef SOC_INTEL_COMMON_BLOCK_SA_H
 #define SOC_INTEL_COMMON_BLOCK_SA_H
 
+#include <cpu/x86/msr.h>
 #include <device/device.h>
 #include <soc/iomap.h>
 #include <stddef.h>
@@ -96,4 +97,7 @@
  */
 void soc_add_fixed_mmio_resources(struct device *dev, int *resource_cnt);
 
+/* SoC specific APIs to get UNCORE PRMRR base and mask values */
+int soc_get_uncore_prmmr_base(msr_t *prmrr_base);
+int soc_get_uncore_prmmr_mask(msr_t *prmrr_mask);
 #endif	/* SOC_INTEL_COMMON_BLOCK_SA_H */
diff --git a/src/soc/intel/common/block/systemagent/systemagent.c b/src/soc/intel/common/block/systemagent/systemagent.c
index cb7af41..188548a 100644
--- a/src/soc/intel/common/block/systemagent/systemagent.c
+++ b/src/soc/intel/common/block/systemagent/systemagent.c
@@ -36,6 +36,18 @@
 	/* no-op */
 }
 
+__attribute__((weak)) int soc_get_uncore_prmmr_base(msr_t *prmrr_base)
+{
+	/* return failure for this dummy API */
+	return -1;
+}
+
+__attribute__((weak)) int soc_get_uncore_prmmr_mask(msr_t *prmrr_mask)
+{
+	/* return failure for this dummy API */
+	return -1;
+}
+
 /*
  * Add all known fixed MMIO ranges that hang off the host bridge/memory
  * controller device.

-- 
To view, visit https://review.coreboot.org/21243
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5e787a2603aaa475cb5c61558cc31ec0afcb4a8b
Gerrit-Change-Number: 21243
Gerrit-PatchSet: 1
Gerrit-Owner: Pratikkumar V Prajapati <pratikkumar.v.prajapati at intel.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170828/6cc76e5e/attachment.html>