Bora Guvendik has uploaded this change for review.

View Change

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)

To view, visit change 37940. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I90659722aaca6ced1e1cbc3db4180b0811205e95
Gerrit-Change-Number: 37940
Gerrit-PatchSet: 1
Gerrit-Owner: Bora Guvendik <bora.guvendik@intel.com>
Gerrit-MessageType: newchange