Bora Guvendik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37940 )
Change subject: dram-spd: Remove free() ......................................................................
dram-spd: Remove free()
free() is not needed since the memory is not dynamically allocated.
Change-Id: I90659722aaca6ced1e1cbc3db4180b0811205e95 Signed-off-by: Bora Guvendik bora.guvendik@intel.com --- M src/vendorcode/cavium/bdk/libdram/dram-spd.c 1 file changed, 0 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/40/37940/1
diff --git a/src/vendorcode/cavium/bdk/libdram/dram-spd.c b/src/vendorcode/cavium/bdk/libdram/dram-spd.c index 894a060..8aec41c 100644 --- a/src/vendorcode/cavium/bdk/libdram/dram-spd.c +++ b/src/vendorcode/cavium/bdk/libdram/dram-spd.c @@ -105,7 +105,6 @@ int64_t data = bdk_twsix_read_ia(node, bus, address, i, 4, 1); if (data < 0) { - free(spd_buf); bdk_error("Failed to read SPD data at 0x%x\n", i + (bank << 8)); /* Restore the bank to zero */ if (bank)