diff options
| author | Jon Murphy <jpmurphy@google.com> | 2022-03-16 11:03:38 -0600 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-03-17 17:39:25 +0000 |
| commit | fcccff35f0964b02d9e083aa79a9424bbac8cce6 (patch) | |
| tree | 6b121ce0302bd2032d50597d8d5b4f9de967e612 | |
| parent | cd93e8e67f44caa78d2a99d22dd8a5d29489fa50 (diff) | |
soc/amd/cezanne: Add counter initializers
Some counters are not being initialized and are relying on mainboards to
set their values. If the mainboards have not implemented these
functions it leads to indeterminate behavior.
BUG=b:224987813
TEST=builds
Signed-off-by: Jon Murphy <jpmurphy@google.com>
Change-Id: I254e26080319478b1b5b1f5c353a7966cfac63b3
Reviewed-on: https://review.coreboot.org/c/coreboot/+/62872
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
| -rw-r--r-- | src/soc/amd/cezanne/fsp_m_params.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/soc/amd/cezanne/fsp_m_params.c b/src/soc/amd/cezanne/fsp_m_params.c index 92debe3b08..8c9e5a15cb 100644 --- a/src/soc/amd/cezanne/fsp_m_params.c +++ b/src/soc/amd/cezanne/fsp_m_params.c @@ -45,10 +45,10 @@ static void fill_ddi_descriptors(FSP_M_CONFIG *mcfg, static void fsp_fill_pcie_ddi_descriptors(FSP_M_CONFIG *mcfg) { - const fsp_dxio_descriptor *fsp_dxio; - const fsp_ddi_descriptor *fsp_ddi; - size_t num_dxio; - size_t num_ddi; + const fsp_dxio_descriptor *fsp_dxio = NULL; + const fsp_ddi_descriptor *fsp_ddi = NULL; + size_t num_dxio = 0; + size_t num_ddi = 0; mainboard_get_dxio_ddi_descriptors(&fsp_dxio, &num_dxio, &fsp_ddi, &num_ddi); |