[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block/systemagent: Add choice option for PCIEX_LENGTH

[Subrata Banik (Code Review)]

Hello Usha P,

I'd like you to do a code review. Please visit

https://review.coreboot.org/c/coreboot/+/40379

to review the following change.

Change subject: soc/intel/common/block/systemagent: Add choice option for PCIEX_LENGTH
......................................................................

soc/intel/common/block/systemagent: Add choice option for PCIEX_LENGTH

This patch adds choice option for PCIEX_LENGTH related Kconfig to avoid
multiple selection from SoC Kconfig.

Change-Id: Icb61e9a0263c058726cc07442af1985a96bf37c2
Signed-off-by: Usha P 
---
M src/soc/intel/common/block/systemagent/Kconfig
1 file changed, 12 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/79/40379/1

diff --git a/src/soc/intel/common/block/systemagent/Kconfig b/src/soc/intel/common/block/systemagent/Kconfig
index 6dd1f3b..ea75d5b 100644
--- a/src/soc/intel/common/block/systemagent/Kconfig
+++ b/src/soc/intel/common/block/systemagent/Kconfig
@@ -18,14 +18,23 @@
    help
      This option allows you to select length of PCIEX region.

+choice
+	prompt "Length of PCI Express Base Address Region"
+	default PCIEX_LENGTH_256MB
+	help
+	  This is to provide new kconfig option that can be used to
+	  select PCI Express Base Address Length.
+
 config PCIEX_LENGTH_256MB
-	bool
+	bool "256 MiB"

config PCIEX_LENGTH_128MB
-	bool
+	bool "128 MiB"

config PCIEX_LENGTH_64MB
-	bool
+	bool "64 MiB"
+
+endchoice

config SA_ENABLE_IMR
    bool