[coreboot-gerrit] Change in coreboot[master]: vc/cavium/bdk/libdram: Print unknown voltages

[Jacob Garber (Code Review)]

Jacob Garber has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/34576 )

Change subject: vc/cavium/bdk/libdram: Print unknown voltages
......................................................................

vc/cavium/bdk/libdram: Print unknown voltages

volt_str is used to print information about the RAM configuration
in report_common_dimm(), so let's print out "unknown voltage" if the
voltage isn't recognized rather than a garbage value.

Change-Id: I8e85917fd682e166172fbf10597bde4a8a11dfc7
Signed-off-by: Jacob Garber 
Found-by: Coverity CID 1393958, 1393982
---
M src/vendorcode/cavium/bdk/libdram/dram-spd.c
1 file changed, 2 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/76/34576/1

diff --git a/src/vendorcode/cavium/bdk/libdram/dram-spd.c b/src/vendorcode/cavium/bdk/libdram/dram-spd.c
index 84df69a..894a060 100644
--- a/src/vendorcode/cavium/bdk/libdram/dram-spd.c
+++ b/src/vendorcode/cavium/bdk/libdram/dram-spd.c
@@ -424,7 +424,7 @@
                       int dram_width, int dimm_size_mb)
 {
     int spd_voltage;
-    const char *volt_str;
+    const char *volt_str = "unknown voltage";

spd_voltage = read_spd(node, dimm_config, DDR3_SPD_NOMINAL_VOLTAGE);
     if ((spd_voltage == 0) || (spd_voltage & 3))
@@ -464,7 +464,7 @@
                       int dram_width, int dimm_size_mb)
 {
     int spd_voltage;
-    const char *volt_str;
+    const char *volt_str = "unknown voltage";

spd_voltage = read_spd(node, dimm_config, DDR4_SPD_MODULE_NOMINAL_VOLTAGE);
     if ((spd_voltage == 0x01) || (spd_voltage & 0x02))