[S] Change in coreboot[main]: soc/amd: introduce SOC_AMD_COMMON_BLOCK_DATA_FABRIC_NP_REGION

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/77987?usp=email )

Change subject: soc/amd: introduce SOC_AMD_COMMON_BLOCK_DATA_FABRIC_NP_REGION
......................................................................

soc/amd: introduce SOC_AMD_COMMON_BLOCK_DATA_FABRIC_NP_REGION

Add a separate Kconfig option for adding np_region.c to the build. Only
the code for Picasso, Cezanne, Mendocino, Phoenix and Glinda call
data_fabric_set_mmio_np which is implemented in that file, so only
select the new SOC_AMD_COMMON_BLOCK_DATA_FABRIC_NP_REGION Kconfig option
for those.

Signed-off-by: Felix Held 
Change-Id: Ic49ce039462b52e2c593c7d2fef43efc50901905
Reviewed-on: https://review.coreboot.org/c/coreboot/+/77987
Reviewed-by: Matt DeVillier 
Reviewed-by: Nico Huber 
Tested-by: build bot (Jenkins) 
---
M src/soc/amd/cezanne/Kconfig
M src/soc/amd/common/block/data_fabric/Kconfig
M src/soc/amd/common/block/data_fabric/Makefile.inc
M src/soc/amd/glinda/Kconfig
M src/soc/amd/mendocino/Kconfig
M src/soc/amd/phoenix/Kconfig
M src/soc/amd/picasso/Kconfig
7 files changed, 14 insertions(+), 1 deletion(-)

Approvals:
  Matt DeVillier: Looks good to me, approved
  Nico Huber: Looks good to me, but someone else must approve
  build bot (Jenkins): Verified

diff --git a/src/soc/amd/cezanne/Kconfig b/src/soc/amd/cezanne/Kconfig
index 832c779..160bc65 100644
--- a/src/soc/amd/cezanne/Kconfig
+++ b/src/soc/amd/cezanne/Kconfig
@@ -41,6 +41,7 @@
    select SOC_AMD_COMMON_BLOCK_CPUFREQ_FAM17H_19H
    select SOC_AMD_COMMON_BLOCK_DATA_FABRIC
    select SOC_AMD_COMMON_BLOCK_DATA_FABRIC_DOMAIN
+	select SOC_AMD_COMMON_BLOCK_DATA_FABRIC_NP_REGION
    select SOC_AMD_COMMON_BLOCK_EMMC
    select SOC_AMD_COMMON_BLOCK_GRAPHICS
    select SOC_AMD_COMMON_BLOCK_HAS_ESPI
diff --git a/src/soc/amd/common/block/data_fabric/Kconfig b/src/soc/amd/common/block/data_fabric/Kconfig
index add2374..da35dae 100644
--- a/src/soc/amd/common/block/data_fabric/Kconfig
+++ b/src/soc/amd/common/block/data_fabric/Kconfig
@@ -4,6 +4,14 @@
      Select this option to add data fabric configuration related
      functionality to the build.

+config SOC_AMD_COMMON_BLOCK_DATA_FABRIC_NP_REGION
+	bool
+	depends on SOC_AMD_COMMON_BLOCK_DATA_FABRIC
+	help
+	  Select this option to include the code to make sure that there's a
+	  non-posted MMIO region configured in the data fabric registers that
+	  covers the FCH MMIO from the HPET up to right below the LAPIC.
+
 config SOC_AMD_COMMON_BLOCK_DATA_FABRIC_DOMAIN
    bool
    depends on SOC_AMD_COMMON_BLOCK_DATA_FABRIC
diff --git a/src/soc/amd/common/block/data_fabric/Makefile.inc b/src/soc/amd/common/block/data_fabric/Makefile.inc
index fea3759..b0684f4 100644
--- a/src/soc/amd/common/block/data_fabric/Makefile.inc
+++ b/src/soc/amd/common/block/data_fabric/Makefile.inc
@@ -2,7 +2,7 @@

ramstage-$(CONFIG_SOC_AMD_COMMON_BLOCK_DATA_FABRIC) += data_fabric_helper.c
 ramstage-$(CONFIG_SOC_AMD_COMMON_BLOCK_DATA_FABRIC_DOMAIN) += domain.c
-ramstage-$(CONFIG_SOC_AMD_COMMON_BLOCK_DATA_FABRIC) += np_region.c
+ramstage-$(CONFIG_SOC_AMD_COMMON_BLOCK_DATA_FABRIC_NP_REGION) += np_region.c

ifeq ($(CONFIG_SOC_AMD_COMMON_BLOCK_DATA_FABRIC_MULTI_PCI_SEGMENT),y)
 ramstage-$(CONFIG_SOC_AMD_COMMON_BLOCK_DATA_FABRIC_DOMAIN) += pci_segment_multi.c
diff --git a/src/soc/amd/glinda/Kconfig b/src/soc/amd/glinda/Kconfig
index 98761a8..ae4b190 100644
--- a/src/soc/amd/glinda/Kconfig
+++ b/src/soc/amd/glinda/Kconfig
@@ -44,6 +44,7 @@
    select SOC_AMD_COMMON_BLOCK_DATA_FABRIC
    select SOC_AMD_COMMON_BLOCK_DATA_FABRIC_DOMAIN
    select SOC_AMD_COMMON_BLOCK_DATA_FABRIC_MULTI_PCI_SEGMENT
+	select SOC_AMD_COMMON_BLOCK_DATA_FABRIC_NP_REGION
    select SOC_AMD_COMMON_BLOCK_ESPI_EXTENDED_DECODE_RANGES # TODO: Check if this is still correct
    select SOC_AMD_COMMON_BLOCK_GRAPHICS		# TODO: Check if this is still correct
    select SOC_AMD_COMMON_BLOCK_HAS_ESPI		# TODO: Check if this is still correct
diff --git a/src/soc/amd/mendocino/Kconfig b/src/soc/amd/mendocino/Kconfig
index ad0ea4c..d179f35 100644
--- a/src/soc/amd/mendocino/Kconfig
+++ b/src/soc/amd/mendocino/Kconfig
@@ -45,6 +45,7 @@
    select SOC_AMD_COMMON_BLOCK_CPU_SYNC_PSP_ADDR_MSR
    select SOC_AMD_COMMON_BLOCK_DATA_FABRIC
    select SOC_AMD_COMMON_BLOCK_DATA_FABRIC_DOMAIN
+	select SOC_AMD_COMMON_BLOCK_DATA_FABRIC_NP_REGION
    select SOC_AMD_COMMON_BLOCK_ESPI_EXTENDED_DECODE_RANGES
    select SOC_AMD_COMMON_BLOCK_GRAPHICS
    select SOC_AMD_COMMON_BLOCK_HAS_ESPI
diff --git a/src/soc/amd/phoenix/Kconfig b/src/soc/amd/phoenix/Kconfig
index 7f6f879..18ed58f 100644
--- a/src/soc/amd/phoenix/Kconfig
+++ b/src/soc/amd/phoenix/Kconfig
@@ -45,6 +45,7 @@
    select SOC_AMD_COMMON_BLOCK_DATA_FABRIC
    select SOC_AMD_COMMON_BLOCK_DATA_FABRIC_DOMAIN
    select SOC_AMD_COMMON_BLOCK_DATA_FABRIC_MULTI_PCI_SEGMENT
+	select SOC_AMD_COMMON_BLOCK_DATA_FABRIC_NP_REGION
    select SOC_AMD_COMMON_BLOCK_ESPI_EXTENDED_DECODE_RANGES
    select SOC_AMD_COMMON_BLOCK_GRAPHICS		# TODO: Check if this is still correct
    select SOC_AMD_COMMON_BLOCK_HAS_ESPI
diff --git a/src/soc/amd/picasso/Kconfig b/src/soc/amd/picasso/Kconfig
index f4b356c..79f60ce 100644
--- a/src/soc/amd/picasso/Kconfig
+++ b/src/soc/amd/picasso/Kconfig
@@ -36,6 +36,7 @@
    select SOC_AMD_COMMON_BLOCK_CPUFREQ_FAM17H_19H
    select SOC_AMD_COMMON_BLOCK_DATA_FABRIC
    select SOC_AMD_COMMON_BLOCK_DATA_FABRIC_DOMAIN
+	select SOC_AMD_COMMON_BLOCK_DATA_FABRIC_NP_REGION
    select SOC_AMD_COMMON_BLOCK_GRAPHICS
    select SOC_AMD_COMMON_BLOCK_HAS_ESPI
    select SOC_AMD_COMMON_BLOCK_HDA