[S] Change in coreboot[master]: soc/intel/meteorlake: Skip crashlog region with metadata tag

[Pratikkumar V Prajapati (Code Review)]

Attention is currently required from: Jakub Czapiga, Kapil Porwal, Pratikkumar Prajapati, Subrata Banik, Tarun Tuli.

Hello Pratikkumar Prajapati,

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

https://review.coreboot.org/c/coreboot/+/76834?usp=email

to review the following change.

Change subject: soc/intel/meteorlake: Skip crashlog region with metadata tag
......................................................................

soc/intel/meteorlake: Skip crashlog region with metadata tag

Region with metadata tag contains information about BDF entry for
SOC PMC SRAM and IOE SRAM. We don't need to parse this as we already
define BDFs in soc/pci_devs.h for these SRAMs. Also we need to skip
to region as it does not contain any crashlog data.

BUG=b:262501347
TEST=Able to build REX. Able to trigger crashlog and decode correctly.

Change-Id: Id8ed40b865cde8e89045f5c9e713398fcbff5890
Signed-off-by: Pratikkumar Prajapati 
---
M src/soc/intel/meteorlake/crashlog.c
1 file changed, 12 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/76834/1

diff --git a/src/soc/intel/meteorlake/crashlog.c b/src/soc/intel/meteorlake/crashlog.c
index e55b214..ce8ad54 100644
--- a/src/soc/intel/meteorlake/crashlog.c
+++ b/src/soc/intel/meteorlake/crashlog.c
@@ -125,8 +125,19 @@

if (!descriptor_table.regions[i].bits.size)
    		continue;
-
+		/*
+		 * Region with metadata TAG contains information about BDF entry for SOC PMC SRAM
+		 * and IOE SRAM. We don't need to parse this as we already define BDFs in
+		 * soc/pci_devs.h for these SRAMs. Also we need to skip to region as it does not
+		 * contain any crashlog data
+		 */
    	if (descriptor_table.regions[i].bits.assign_tag ==
+				CRASHLOG_DESCRIPTOR_TABLE_TAG_META) {
+			pmc_crashLog_size -= descriptor_table.regions[i].bits.size *
+						sizeof(u32);
+			printk(BIOS_DEBUG, "PMC crashlog size adjusted to: 0x%x\n",
+					pmc_crashLog_size);
+		} else if (descriptor_table.regions[i].bits.assign_tag ==
    			CRASHLOG_DESCRIPTOR_TABLE_TAG_SOC) {

if (cl_copy_data_from_sram(pmc_sram_base,