[coreboot-gerrit] Change in coreboot[master]: soc/intel/cannonlake: Fix coverity report

Lijian Zhao (Code Review) gerrit at coreboot.org
Tue Sep 18 21:12:27 CEST 2018


Lijian Zhao has uploaded this change for review. ( https://review.coreboot.org/28664


Change subject: soc/intel/cannonlake: Fix coverity report
......................................................................

soc/intel/cannonlake: Fix coverity report

Remove const define for spd_smbus_address to fix error reported from
coverity.

BUG=CID 1395725
TEST=N/A

Change-Id: Ib933ed872e9f85087bb3cd76a1f1e29cca75cd54
Signed-off-by: Lijian Zhao <lijian.zhao at intel.com>
---
M src/soc/intel/cannonlake/include/soc/cnl_memcfg_init.h
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/64/28664/1

diff --git a/src/soc/intel/cannonlake/include/soc/cnl_memcfg_init.h b/src/soc/intel/cannonlake/include/soc/cnl_memcfg_init.h
index 7a76eb0..245d2cf 100644
--- a/src/soc/intel/cannonlake/include/soc/cnl_memcfg_init.h
+++ b/src/soc/intel/cannonlake/include/soc/cnl_memcfg_init.h
@@ -46,7 +46,7 @@
 		int spd_index;
 		struct spd_by_pointer spd_data_ptr_info;
 	} spd_spec;
-	const uint8_t spd_smbus_address[4];
+	uint8_t spd_smbus_address[4];
 };
 
 /* Board-specific memory dq mapping information */

-- 
To view, visit https://review.coreboot.org/28664
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib933ed872e9f85087bb3cd76a1f1e29cca75cd54
Gerrit-Change-Number: 28664
Gerrit-PatchSet: 1
Gerrit-Owner: Lijian Zhao <lijian.zhao at intel.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180918/95e29050/attachment.html>


More information about the coreboot-gerrit mailing list