[coreboot-gerrit] Change in coreboot[master]: [WIP] soc/amd/picasso/fsp_params: add asserts for descriptor count

[Felix Held (Code Review)]

Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43435 )

Change subject: [WIP] soc/amd/picasso/fsp_params: add asserts for descriptor count
......................................................................

[WIP] soc/amd/picasso/fsp_params: add asserts for descriptor count

Will likely need to fix some issues this uncovered before this can get
merged.

BUG=b:158695393

Change-Id: Ia220d5a9d4ff11707b795b04662ff7eead4e2888
Signed-off-by: Felix Held 
---
M src/soc/amd/picasso/fsp_params.c
1 file changed, 7 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/35/43435/1

diff --git a/src/soc/amd/picasso/fsp_params.c b/src/soc/amd/picasso/fsp_params.c
index 80d43fb..6fb3668 100644
--- a/src/soc/amd/picasso/fsp_params.c
+++ b/src/soc/amd/picasso/fsp_params.c
@@ -1,5 +1,6 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

+#include <assert.h>
 #include <device/pci.h>
 #include <soc/pci_devs.h>
 #include <soc/platform_descriptors.h>
@@ -60,6 +61,9 @@
 {
    size_t i;

+	ASSERT_MSG(num <= FSPS_UPD_DXIO_DESCRIPTOR_COUNT,
+			"Too many DXIO descriptors provided.");
+
    for (i = 0; i < num; i++) {
    	memcpy(scfg->dxio_descriptor[i], &descs[i], sizeof(scfg->dxio_descriptor[0]));
    }
@@ -70,6 +74,9 @@
 {
    size_t i;

+	ASSERT_MSG(num <= FSPS_UPD_DDI_DESCRIPTOR_COUNT,
+			"Too many DDI descriptors provided.");
+
    for (i = 0; i < num; i++) {
    	memcpy(&scfg->ddi_descriptor[i], &descs[i], sizeof(scfg->ddi_descriptor[0]));
    }